[wp-trac] [WordPress Trac] #25466: Hook Docs: wp-includes/vars.php

[WordPress Trac]

#25466: Hook Docs: wp-includes/vars.php
--------------------------+------------------------------
 Reporter:  tmtoy         |       Owner:  DrewAPicture
     Type:  defect (bug)  |      Status:  reviewing
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Inline Docs   |     Version:
 Severity:  normal        |  Resolution:
 Keywords:  needs-patch   |
--------------------------+------------------------------
Changes (by DrewAPicture):

 * keywords:  has-patch => needs-patch
 * type:  enhancement => defect (bug)


Comment:

 Hi, thanks for the patch. Some notes on [attachment:vars.diff]
 * "Filter whether to use google chrome frame." is incorrect. The filters
 serves more as whether to identify the HTTP_USER_AGENT as Google Chrome.
 * Once you've fixed the short description, match that to the parameter
 description. Something like, "Whether to allow the HTTP_USER_AGENT to be
 identified as Google Chrome."
 * Specify the default of the boolean value (in this case, `true`) at the
 end of the parameter description.
 * Per [18036], `@since` should be 3.2.0.

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