[wp-trac] [WordPress Trac] #13961: search query_vars bug when using search base
WordPress Trac
wp-trac at lists.automattic.com
Thu Jul 5 18:54:59 UTC 2012
#13961: search query_vars bug when using search base
---------------------------------+--------------------------
Reporter: shidouhikari | Owner: markjaquith
Type: defect (bug) | Status: reopened
Priority: normal | Milestone: 3.5
Component: Query | Version: 3.0
Severity: normal | Resolution:
Keywords: has-patch 3.5-early |
---------------------------------+--------------------------
Comment (by nacin):
Replying to [comment:28 markjaquith]:
> Replying to [comment:27 nacin]:
> > Should this be $this->is_main_query()? Or should we just always
decode?
>
> If someone is manually doing a new WP_Query, I don't think we want to
decode.
Sure. But right now this will always fire on new WP_Query, unless
query_posts() was used, as is_main_query() will just return true. This
needs to be $this->is_main_query().
Kinda want to deprecate the is_main_query() function and leave just the
method. It's confusing.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/13961#comment:29>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list