[wp-trac] [WordPress Trac] #16622: [17246] Breaks category exclusion
WordPress Trac
wp-trac at lists.automattic.com
Wed Feb 23 21:42:59 UTC 2011
#16622: [17246] Breaks category exclusion
------------------------------------+--------------------
Reporter: markjaquith | Owner:
Type: defect (bug) | Status: new
Priority: high | Milestone: 3.1.1
Component: Query | Version: 3.1
Severity: critical | Resolution:
Keywords: has-patch dev-feedback |
------------------------------------+--------------------
Comment (by ryan):
pre_get_posts is an unreliable place to do set(). You are setting a query
var after the query vars are parsed. It happens to work for a few things
in 3.0, but it isn't uniformly reliable. It also don't set the query
flags to match the query. But, I guess these things have become
features. :-)
Anyhow, the patch seems a good approach. The query vars array is usually
pretty small so I don't think the serialize will be a big hit. Going
forward, we might want to
--
Ticket URL: <http://core.trac.wordpress.org/ticket/16622#comment:5>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list