[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