Skip to:

Opened 14 years ago

Closed 14 years ago

#1843 closed defect (bug) (fixed)

Remove delete activity link for user 0

Reported by: boonebgorges's profile boonebgorges Owned by:
Milestone: 1.2.1 Priority: minor
Severity: Version:
Component: Keywords: activity, delete, non-logged-in, has-patch, needs-testing


bp_activity_content_filter() in bp-activity-templatetags.php checks to see whether it should include the Delete Activity link based (among other things) on whether $bp->loggedin_user->id == $activities_template->activity->user_id. But when the activity's user_id is 0, either because of a plugin like or or because of an error, the non-logged-in viewers of the activity stream can see the Delete button (though it doesn't work).

I've attached a patch that checks to make sure user_id != 0.

Attachments (1)

no-activity-delete-for-user-0.patch (961 bytes) - added by boonebgorges 14 years ago.

Download all attachments as: .zip

Change History (3)

#1 @boonebgorges
14 years ago

  • Keywords has-patch needs-testing added

#2 @johnjamesjacoby
14 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [2718]) Only show delete link for logged in users. Fixes #1843. props boonebgorges

Note: See TracTickets for help on using tickets.