[wp-trac] [WordPress Trac] #21195: get_avatar_url

WordPress Trac noreply at wordpress.org
Thu Jan 8 00:29:46 UTC 2015


#21195: get_avatar_url
-----------------------------+------------------
 Reporter:  pathawks         |       Owner:
     Type:  feature request  |      Status:  new
 Priority:  normal           |   Milestone:  4.2
Component:  General          |     Version:  2.5
 Severity:  normal           |  Resolution:
 Keywords:  has-patch        |     Focuses:
-----------------------------+------------------

Comment (by pento):

 So, I realised the purpose of the `apply_filters_ref_array()` - to allow
 filters to change the `$args`.

 Because references make everyone sad, [attachment:21195.13.diff] shuffles
 all of the `$args` processing into a new `get_avatar_data()` function,
 with filters specifically for filtering the data. `get_avatar_url()` is
 now just a little wrapper around this function.

 This has the added bonus of allowing us to remove the `$processed_args`
 references, as `get_avatar()` is able to call `get_avatar_data()`, and get
 all of the processed args directly.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/21195#comment:55>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list