[buddypress-trac] [BuddyPress Trac] #6612: xprofile_avatar_uploaded action hook

buddypress-trac noreply at wordpress.org
Tue Sep 22 08:25:41 UTC 2015


#6612: xprofile_avatar_uploaded action hook
------------------------------+--------------------
 Reporter:  landwire          |       Owner:
     Type:  defect (bug)      |      Status:  new
 Priority:  normal            |   Milestone:  2.3.4
Component:  API - Avatars     |     Version:  2.3.0
 Severity:  normal            |  Resolution:
 Keywords:  has-patch commit  |
------------------------------+--------------------

Comment (by imath):

 Thanks r-a-y, i was about to commit it when after some other tests, i've
 noticed the "changed their avatar" activity was not created when a user is
 setting his avatar using the wp-admin/extended profile screen.

 That's because `bp_xprofile_new_avatar_activity()` is using
 `bp_displayed_user_id()` to get the user id and in the wp-admin/extended
 profile screen the displayed user id is not set.

 So i think we should also add a new parameter to "transport" the user id
 in the `xprofile_avatar_uploaded` action.

 I've tested 6612.03.patch (Admin/front) and it's fixing the issue, if no
 objections i will commit it soon.

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


More information about the buddypress-trac mailing list