Bug #5630
closedSystem Action logic for canvas and avatar update activities not accounting for removal of uploaded images
Description
As reported in this thread:
With image upload approvals off,
When someone uploads a new image for canvas or avatar and then uploads a second one the first image is removed (standard CB practice).
This breaks the activity display in the stream as the old image url is no longer available.
Updated by nant about 9 years ago
Update - the system action looks like it also has bad logic if approval is turned on - perhaps uploading an image that needs approval should not be tracked as it just displayes the pending approval image.
Updated by krileon about 9 years ago
- Status changed from Assigned to Rejected
- % Done changed from 0 to 100
Nothing to do with any of that. The activity is static. It's not being parsed on display and pulling the avatar field. It's logged once and the HTML stored is directly output. CB Activity needs built in trigger handling for internal activity like that. Basically the same way GJ has built in handling of its own activity.
Rejecting this ticket and linking it to the ticket in CB Activity to hardcode core activity parsing.
Updated by krileon about 9 years ago
- Related to Feature proposal #5407: Hardcode support for core activity added
Updated by krileon about 9 years ago
- Status changed from Rejected to Assigned
- Priority changed from High to Normal
- Target version set to 6.3.4
- % Done changed from 100 to 0
Re-opening as this activity should only maintain 1 activity entry like some other activity. Meaning it should edit the old entry and update it with new information and a new date or delete the old entry first. Even hardcoding the activity output won't really help with this as the activity just isn't valid anymore.
Updated by krileon about 9 years ago
- Status changed from Assigned to Resolved
- % Done changed from 0 to 100
Old activity entries will now delete before a new activity entry is logged for canvas and avatar.