Project

General

Profile

Actions

Bug #5630

closed

System Action logic for canvas and avatar update activities not accounting for removal of uploaded images

Added by nant over 8 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Start date:
01 December 2015
Due date:
% Done:

100%

Estimated time:

Description

As reported in this thread:

https://www.joomlapolis.com/forum/153-professional-member-support/231962-error-in-image-canvas-into-activity

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.


Related issues 1 (0 open1 closed)

Related to CB Activity - Feature proposal #5407: Hardcode support for core activityClosedkrileon17 August 2015

Actions
Actions #1

Updated by nant over 8 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.

Actions #2

Updated by krileon over 8 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.

Actions #3

Updated by krileon over 8 years ago

Actions #4

Updated by krileon over 8 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.

Actions #5

Updated by krileon over 8 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.

Actions #6

Updated by krileon over 8 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF