WordPress.org
Get WordPress

Making WordPress.org

Changeset 7319


Ignore:
Timestamp:
06/19/2018 09:44:33 AM (8 years ago)
Author:
obenland
Message:

Developer: Use query_vars correctly.

Props grapplerulrich, johnbillion, SergeyBiryukov.
Fixes #2882.

Location:
sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/extras.php

    r5598 r7319  
    9797 */
    9898function wporg_filter_archive_excerpt( $excerpt ) {
    99         if ( ! is_single() && ! get_query_var( 'is_handbook' ) && 'command' !== get_query_var( 'post_type' ) ) {
     99        if ( ! is_single() && ! $GLOBALS['wp_query']->is_handbook && 'command' !== get_query_var( 'post_type' ) ) {
    100100
    101101                $post_id = get_the_ID();
    … …  
    201201}
    202202add_filter( 'loop_pagination', 'wporg_loop_pagination' );
    203 

  • sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/handbooks.php

    r5681 r7319  
    6363         */
    6464        public static function add_query_vars( $public_query_vars ) {
    65                 $public_query_vars['is_handbook'] = false;
    66                 $public_query_vars['current_handbook'] = false;
    67                 $public_query_vars['current_handbook_home_url'] = false;
    68                 $public_query_vars['current_handbook_name'] = '';
     65                $public_query_vars[] = 'current_handbook';
     66                $public_query_vars[] = 'current_handbook_home_url';
     67                $public_query_vars[] = 'current_handbook_name';
    6968
    7069                return $public_query_vars;
    … …  
    7776         */
    7877        public static function pre_get_posts( $query ) {
    79                 $is_handbook = function_exists( 'wporg_is_handbook' ) ? wporg_is_handbook() : false;
    80                 $query->set( 'is_handbook', $is_handbook );
    81 
    82                 $current_handbook = function_exists( 'wporg_get_current_handbook' ) ? wporg_get_current_handbook() : false;
     78                $query->is_handbook = function_exists( 'wporg_is_handbook' ) && wporg_is_handbook();
     79
     80                $current_handbook = function_exists( 'wporg_get_current_handbook' ) ? (string) wporg_get_current_handbook() : '';
    8381                $query->set( 'current_handbook', $current_handbook );
    8482
    85                 $current_handbook_home_url = function_exists( 'wporg_get_current_handbook_home_url' ) ? wporg_get_current_handbook_home_url() : false;
     83                $current_handbook_home_url = function_exists( 'wporg_get_current_handbook_home_url' ) ? (string) wporg_get_current_handbook_home_url() : '';
    8684                $query->set( 'current_handbook_home_url', $current_handbook_home_url );
    8785
    88                 $current_handbook_name = function_exists( 'wporg_get_current_handbook_name' ) ? wporg_get_current_handbook_name() : '';
     86                $current_handbook_name = function_exists( 'wporg_get_current_handbook_name' ) ? (string) wporg_get_current_handbook_name() : '';
    8987                $query->set( 'current_handbook_name', $current_handbook_name );
    9088        }

  • sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/redirects.php

    r4709 r7319  
    3131        /**
    3232         * Redirects a search query with only one result directly to that result.
     33         *
     34         * @globals \WP_Query $wp_query Global WP_Query instance.
    3335         */
    3436        public static function redirect_single_search_match() {
    35                 if ( is_search() && ! get_query_var( 'is_handbook' ) && 1 == $GLOBALS['wp_query']->found_posts ) {
     37                global $wp_query;
     38
     39                if ( is_search() && ! $wp_query->is_handbook && 1 == $wp_query->found_posts ) {
    3640                        wp_redirect( get_permalink( get_post() ) );
    3741                        exit();

  • sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/search.php

    r5432 r7319  
    2525                add_filter( 'posts_orderby', array( __CLASS__, 'search_posts_orderby' ), 10, 2 );
    2626                add_filter( 'the_posts',     array( __CLASS__, 'rerun_empty_exact_search' ), 10, 2 );
    27                 add_filter( 'query_vars',    array( __CLASS__, 'default_qv_empty_post_type_search' ) );
    28         }
    29 
    30         /**
    31          * Add query var to indicate if no post type filters were explicitly used for
    32          * a search.
    33          *
    34          * Defaults the query var 'empty_post_type_search' to false. It is potentially
    35          * set to true elsewhere.
    36          *
    37          * @param array $public_query_vars The array of whitelisted query variables.
    38          * @return array
    39          */
    40         public static function default_qv_empty_post_type_search( $public_query_vars ) {
    41                 $public_query_vars['empty_post_type_search'] = false;
    42                 return $public_query_vars;
    4327        }
    4428
    … …  
    6044                // Separates searches for handbook pages from non-handbook pages depending on
    6145                // whether the search was performed within context of a handbook page or not.
    62                 if ( get_query_var( 'is_handbook' ) ) {
     46                if ( $query->is_handbook ) {
    6347                        // Search only in current handbook post type.
    6448                        // Just to make sure. post type should already be set.
    … …  
    8367                if ( ! $qv_post_types ) {
    8468                        // Record the fact no post types were explicitly supplied.
    85                         $query->set( 'empty_post_type_search', true );
     69                        $query->is_empty_post_type_search = true;
    8670
    8771                        // Not a handbook page, or exact search, or filters used.

  • sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/search.php

    r3380 r7319  
    1414                        <span><a href="<?php echo home_url( '/' ); ?>"><?php _e( 'Home', 'wporg' ); ?></a></span>
    1515                        <span class="sep">/</span>
    16                         <?php if ( get_query_var( 'is_handbook' ) ) : ?>
     16                        <?php if ( $GLOBALS['wp_query']->is_handbook ) : ?>
    1717                        <span><a href="<?php echo esc_url( get_query_var( 'current_handbook_home_url' ) ); ?>"><?php echo get_query_var( 'current_handbook_name' ); ?></a></span>
    1818                        <?php elseif ( $reference_page = get_page_by_path( 'reference' ) ) : ?>
    … …  
    3232                        <?php while ( have_posts() ) : the_post(); ?>
    3333
    34                                 <?php get_template_part( 'content', get_query_var( 'is_handbook' ) ? 'handbook-archive' : 'reference-archive' ); ?>
     34                                <?php get_template_part( 'content', $GLOBALS['wp_query']->is_handbook ? 'handbook-archive' : 'reference-archive' ); ?>
    3535
    3636                        <?php endwhile; ?>

  • sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/searchform.php

    r4551 r7319  
    3535
    3636        <?php
    37                 $is_handbook = get_query_var( 'is_handbook' );
     37                $is_handbook = $GLOBALS['wp_query']->is_handbook;
    3838                $search_url  = get_query_var( 'current_handbook_home_url' );
    3939                $search_url  = $search_url ? $search_url : home_url( '/' );
    … …  
    6060                                        'wp-parser-method'   => __( 'Methods',   'wporg' ),
    6161                                );
    62                                
    63                                 $qv_post_type = array_filter( (array) get_query_var( 'post_type' ) );   
    64                                 $no_filters   = get_query_var( 'empty_post_type_search' );
     62
     63                                $qv_post_type = array_filter( (array) get_query_var( 'post_type' ) );
     64                                $no_filters   = $GLOBALS['wp_query']->is_empty_post_type_search;
    6565
    6666                                if ( ! is_search() || in_array( 'any', $qv_post_type ) || $no_filters ) {
    … …  
    6868                                        $qv_post_type = array();
    6969                                }
    70                                                
     70
    7171                                foreach ( $search_post_types as $post_type => $label ) {
    7272                                        $checked = checked( in_array( $post_type, $qv_post_type ), true, false );
Note: See TracChangeset for help on using the changeset viewer.