[buddypress-trac] [BuddyPress Trac] #5900: bp_activity_user_can_delete() doesn't work for non-admins

buddypress-trac noreply at wordpress.org
Thu Sep 25 17:16:06 UTC 2014


#5900: bp_activity_user_can_delete() doesn't work for non-admins
------------------------------+---------------------
 Reporter:  r-a-y             |       Owner:  r-a-y
     Type:  defect (bug)      |      Status:  closed
 Priority:  normal            |   Milestone:  2.1.1
Component:  Activity          |     Version:  2.1
 Severity:  normal            |  Resolution:  fixed
 Keywords:  has-patch commit  |
------------------------------+---------------------
Changes (by r-a-y):

 * owner:   => r-a-y
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"9042"]:
 {{{
 #!CommitTicketReference repository="" revision="9042"
 Activity: Make sure a non-admin can delete their own activity.

 Changes in r8697 broke this functionality due to a strict type check. The
 activity user ID is a string, while the logged-in user is an integer.

 Commit fixes this by typecasting the activity user ID as an integer and
 adds a unit test.

 Fixes #5900 (2.1-branch).
 }}}

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5900#comment:3>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list