[wp-trac] [WordPress Trac] #21195: get_avatar_url
WordPress Trac
noreply at wordpress.org
Wed Oct 30 03:59:12 UTC 2013
#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 dd32):
> `function get_avatar( $id_or_email, $args = null )`
I don't think we can change the signature like that since it's a pluggable
function, adding optional parameters is OK if the calling code can deal
with it not being respected, but we should attempt to limit any in/out
behaviour of the existing function.
Moving the entire contents of the function to a `get_avatar_url()`
function is perfectly fine though, just have to remember that anyone who
has a pluggable `get_avatar()` at present will suddenly have their
functions alterations not respected.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/21195#comment:31>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list