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

[WordPress Trac]

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

Comment (by pathawks):

 Replying to [comment:15 cliffseal]:
 > Replying to [comment:14 pathawks]:
 > > If `get_avatar` has been plugged with a version that does not support
 returning just the URL when given an extra paramater, it will break
 anything that relies on that behavior.
 >
 > It doesn't occur in any of my tests

 Of course it doesn't; you're working with a modified version of
 `get_avatar` :)

 The problem comes when a plugin has replaced `get_avatar` completely. In
 that case, no updates that have been made to core's `get_avatar` will be
 reflected.
 In this case, `get_avatar` would never return just a URL, because it was
 merely designed to replace the legacy `get_avatar`.

 Now, the problem comes when a plugin expects to be able to retrieve a URL
 from `get_avatar`. As you can see, it wouldn't be possible in this
 senario. If the new plugin is expecting just a URL to be returned and
 instead is given an `<img>` tag, Bad Things will happen. If the plugged
 `get_avatar` is passed more parameters than it knows what to do with, Bad
 Things might happen.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/21195#comment:16>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software