[buddypress-trac] [BuddyPress] #3434: $secondary_item_id is no longer optional in delete_for_user_by_item_id

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Thu Aug 4 10:50:50 UTC 2011


#3434: $secondary_item_id is no longer optional in delete_for_user_by_item_id
-------------------------+------------------
 Reporter:  enderandrew  |       Owner:
     Type:  defect       |      Status:  new
 Priority:  normal       |   Milestone:  1.5
Component:  Core         |     Version:  1.5
 Severity:  normal       |  Resolution:
 Keywords:               |
-------------------------+------------------
Changes (by boonebgorges):

 * milestone:  Awaiting Review => 1.5


Comment:

 I'm not seeing where it was every $secondary_item_id = false
 https://buddypress.trac.wordpress.org/browser/trunk/bp-core/bp-core-
 classes.php?annotate=blame&rev=4917#L773

 However, there is a bit of logic in the function that accounts for the
 case where it's not passed. So it makes sense to put a default value in
 there.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/3434#comment:1>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list