[buddypress-trac] [BuddyPress] #4333: Allow User avatar functions to pass through arguments to bp_core_fetch_avatar

buddypress-trac noreply at wordpress.org
Thu Dec 13 10:00:32 UTC 2012


#4333: Allow User avatar functions to pass through arguments to
bp_core_fetch_avatar
-------------------------------------------------+-------------------------
 Reporter:  cklosows                             |       Owner:
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  1.8
Component:  Core                                 |     Version:  1.6-beta
 Severity:  normal                               |  Resolution:
 Keywords:  needs-testing dev-feedback needs-    |
  patch has-patch                                |
-------------------------------------------------+-------------------------
Changes (by cnorris23):

 * keywords:  needs-testing dev-feedback needs-patch => needs-testing dev-
     feedback needs-patch has-patch


Comment:

 First off, ignore the 4333.04 varieties. Guess I got a little trigger
 happy with the agree and upload button, so it's a duplicate upload, plus
 they won't apply correctly for some reason.

 Attached patch includes the following:

 1) allows relevant functions to accept all bp_core_fetch_avatar arguments
 (except those specific to the function's, uh, function)
 2) removes use of extract (TODO: bp_core_fetch_avatar(). Big under taking
 and I need to get some shut eye)
 3) removes redeclaration of default bp_core_fetch_avatar arguments where
 possible

 I couldn't get any errors to appear, but it was also limited testing. Will
 definitely need some eyes.

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


More information about the buddypress-trac mailing list