[buddypress-trac] [BuddyPress] #1843: Remove delete activity link for user 0

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Sun Feb 7 21:24:26 UTC 2010


#1843: Remove delete activity link for user 0
--------------------------------------------+-------------------------------
Reporter:  boonebgorges                     |       Owner:       
    Type:  defect                           |      Status:  new  
Priority:  minor                            |   Milestone:  1.2.1
Keywords:  activity, delete, non-logged-in  |  
--------------------------------------------+-------------------------------
 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 http://wordpress.org/extend/plugins
 /buddypress-group-twitter/ or http://teleogistic.net/bp-include-non-
 member-comments 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.

-- 
Ticket URL: <http://trac.buddypress.org/ticket/1843>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list