[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