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

WordPress Trac noreply at wordpress.org
Thu Oct 2 23:52:36 UTC 2014


#21195: get_avatar_url
----------------------------------------+------------------
 Reporter:  pathawks                    |       Owner:
     Type:  feature request             |      Status:  new
 Priority:  normal                      |   Milestone:  4.1
Component:  General                     |     Version:  2.5
 Severity:  normal                      |  Resolution:
 Keywords:  has-patch needs-unit-tests  |     Focuses:
----------------------------------------+------------------
Changes (by pento):

 * keywords:  has-patch => has-patch needs-unit-tests


Comment:

 That was a fun walk down memory lane. [attachment:21195.8.diff]:

 * Updates the patch to apply cleanly against trunk
 * Re-adds some filter PHPDocs that had been removed
 * Adds new PHPDocs for new filters
 * Adds PHPDocs for new parameters to old filters
 * Fixes a bunch of space/tab indent issues
 * Fixes some minor coding style issues

 It wouldn't kill us to put some unit tests on this, too. I'll do that
 sometime later today, if someone doesn't beat me to it.

 @DrewAPicture - I couldn't find anything in the doc standards about
 marking when a new parameter was added, just a couple of examples that I
 managed to grep for the source. I went with the `(since 4.1.0)` style,
 please correct me if there's a better way. :-)

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


More information about the wp-trac mailing list