[wp-trac] [WordPress Trac] #25514: Hook Docs: wp-includes/query.php

[WordPress Trac]

#25514: Hook Docs: wp-includes/query.php
--------------------------+------------------------------
 Reporter:  dougwollison  |       Owner:  DrewAPicture
     Type:  enhancement   |      Status:  reviewing
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Inline Docs   |     Version:
 Severity:  normal        |  Resolution:
 Keywords:  needs-patch   |
--------------------------+------------------------------

Comment (by DrewAPicture):

 @dougwoolison: You do good work :)

 [attachment:WP_Query-hook-docs.5.diff]:
 `the_post` filter:
 * "retreived"

 `pre_get_posts` hook:
 The long description note isn't really correct and that's my badness. It's
 not that they aren't available, it's that conditional tags are testing the
 `$wp_query` global and not the main query, which is what is passed to this
 hook.

 After some discussion with @helen, I'd like to change that to something
 like:
 "Note: WP_Query member methods should be used to test the main query here,
 rather than global conditional functions. For instance, use
 WP_Query::is_main_query() instead of is_main_query()."

 Have a better suggested wording?

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