Make WordPress Core


Ignore:
Timestamp:
10/21/2015 04:53:51 PM (8 years ago)
Author:
boonebgorges
Message:

Prevent non-public taxonomies from registering aquery var.

[34247] made the 'public' paramater of register_taxonomy() work by blocking
requests for non-public taxonomy archives during parse_request(). Blocking
taxonomy archive requests this late means that it's impossible to register an
independent query var that matches the slug of a non-public taxonomy. By
moving the block to register_taxonomy() - not allowing these taxonomies to
register their query vars in the first place - we free up the slug for other
use. In addition, we free up a bit of processing (no need to look for the query
var in parse_request() and better parallel the way non-public post types
work. See register_post_type().

Non-public taxonomy archives that are requested using ?taxonomy=tax_name are
still blocked during parse_request. It's only custom query vars -
?tax_name=term - that are affected by this change.

Props mboynes.
Fixes #21949.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/class-wp.php

    r35205 r35333  
    312312        if ( ! is_admin() ) {
    313313            foreach ( get_taxonomies( array( 'public' => false ), 'objects' ) as $taxonomy => $t ) {
    314                 // Check first for taxonomy-specific query_var.
    315                 if ( $t->query_var && isset( $this->query_vars[ $t->query_var ] ) ) {
    316                     unset( $this->query_vars[ $t->query_var ] );
    317                 }
    318 
    319                 // Next, check the 'taxonomy' query_var.
     314                /*
     315                 * Disallow when set to the 'taxonomy' query var.
     316                 * Non-public taxonomies cannot register custom query vars. See register_taxonomy().
     317                 */
    320318                if ( isset( $this->query_vars['taxonomy'] ) && $taxonomy === $this->query_vars['taxonomy'] ) {
    321319                    unset( $this->query_vars['taxonomy'], $this->query_vars['term'] );
Note: See TracChangeset for help on using the changeset viewer.