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

WordPress Trac noreply at wordpress.org
Sun Jan 11 02:07:38 UTC 2015


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

Comment (by nacin):

 In [18513]'s case, `get_user_by()` is also pluggable. Presumably, if one
 is plugging the user_by functions, they're plugging all of them.

 Frankly, I care very little about the promise of most pluggable functions.
 The auth ones, sure. But then there's `get_avatar()`, a fairly high-level
 function that, if I had to take a guess, is commonly overridden to use
 something other than Gravatar. I wish it were never pluggable and that it
 simply had just the filter (which was introduced at the same time).

 I don't know — maybe we can get away with ditching it, deprecating it, and
 just using a new `wp_get_avatar()`, and anyone who is currently overriding
 `get_avatar()` just gets screwed and told they should have been using the
 filter.

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


More information about the wp-trac mailing list