WordPress.org
Get WordPress

Make WordPress Core

Changeset 31106


Ignore:
Timestamp:
01/08/2015 10:51:30 PM (12 years ago)
Author:
wonderboymusic
Message:

After [31105], don't ditch the isset() calls for BC. Declare $page_hook as null so it is initialized for all execution paths but will still fail isset() checks.

Fixes #30958.

Location:
trunk/src
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-admin/admin.php

    r31105 r31106  
    9696global $pagenow, $hook_suffix, $plugin_page, $typenow, $taxnow;
    9797
    98 $page_hook = '';
    99 $hook_suffix = '';
    100 $plugin_page = '';
    101 $typenow = '';
    102 $taxnow = '';
     98$page_hook = null;
    10399
    104100$editing = false;
    105101
    106 if ( isset( $_GET['page'] ) ) {
     102if ( isset($_GET['page']) ) {
    107103        $plugin_page = wp_unslash( $_GET['page'] );
    108         $plugin_page = plugin_basename( $plugin_page );
    109 }
    110 
    111 if ( isset( $_REQUEST['post_type'] ) && post_type_exists( $_REQUEST['post_type'] ) ) {
     104        $plugin_page = plugin_basename($plugin_page);
     105}
     106
     107if ( isset( $_REQUEST['post_type'] ) && post_type_exists( $_REQUEST['post_type'] ) )
    112108        $typenow = $_REQUEST['post_type'];
    113 }
    114 
    115 if ( isset( $_REQUEST['taxonomy'] ) && taxonomy_exists( $_REQUEST['taxonomy'] ) ) {
     109else
     110        $typenow = '';
     111
     112if ( isset( $_REQUEST['taxonomy'] ) && taxonomy_exists( $_REQUEST['taxonomy'] ) )
    116113        $taxnow = $_REQUEST['taxonomy'];
    117 }
     114else
     115        $taxnow = '';
    118116
    119117if ( WP_NETWORK_ADMIN )
    … …  
    154152do_action( 'admin_init' );
    155153
    156 if ( $plugin_page ) {
    157         if ( $typenow ) {
     154if ( isset($plugin_page) ) {
     155        if ( !empty($typenow) )
    158156                $the_parent = $pagenow . '?post_type=' . $typenow;
    159         } else {
     157        else
    160158                $the_parent = $pagenow;
    161         }
    162159        if ( ! $page_hook = get_plugin_page_hook($plugin_page, $the_parent) ) {
    163160                $page_hook = get_plugin_page_hook($plugin_page, $plugin_page);
    … …  
    177174}
    178175
    179 if ( $page_hook ) {
     176$hook_suffix = '';
     177if ( isset( $page_hook ) ) {
    180178        $hook_suffix = $page_hook;
    181 } elseif ( $plugin_page ) {
     179} elseif ( isset( $plugin_page ) ) {
    182180        $hook_suffix = $plugin_page;
    183 } elseif ( $pagenow ) {
     181} elseif ( isset( $pagenow ) ) {
    184182        $hook_suffix = $pagenow;
    185183}
    … …  
    188186
    189187// Handle plugin admin pages.
    190 if ( $plugin_page ) {
     188if ( isset($plugin_page) ) {
    191189        if ( $page_hook ) {
    192190                /**

  • trunk/src/wp-admin/includes/plugin.php

    r31105 r31106  
    15091509        }
    15101510
    1511         if ( $pagenow == 'admin.php' && $plugin_page ) {
     1511        if ( $pagenow == 'admin.php' && isset( $plugin_page ) ) {
    15121512                foreach ( (array)$menu as $parent_menu ) {
    15131513                        if ( $parent_menu[2] == $plugin_page ) {
    … …  
    15261526        }
    15271527
    1528         if ( $plugin_page && isset( $_wp_submenu_nopriv[$pagenow][$plugin_page] ) ) {
     1528        if ( isset( $plugin_page ) && isset( $_wp_submenu_nopriv[$pagenow][$plugin_page] ) ) {
    15291529                $parent_file = $pagenow;
    15301530                if ( isset( $_wp_real_parent_file[$parent_file] ) )
    … …  
    15371537                        if ( isset( $_wp_real_parent_file[$parent] ) )
    15381538                                $parent = $_wp_real_parent_file[$parent];
    1539                         if ( $typenow && ($submenu_array[2] == "$pagenow?post_type=$typenow") ) {
     1539                        if ( !empty($typenow) && ($submenu_array[2] == "$pagenow?post_type=$typenow") ) {
    15401540                                $parent_file = $parent;
    15411541                                return $parent;
    1542                         } elseif ( $submenu_array[2] == $pagenow && ! $typenow && ( empty($parent_file) || false === strpos($parent_file, '?') ) ) {
     1542                        } elseif ( $submenu_array[2] == $pagenow && empty($typenow) && ( empty($parent_file) || false === strpos($parent_file, '?') ) ) {
    15431543                                $parent_file = $parent;
    15441544                                return $parent;
    1545                         } elseif ( $plugin_page && ($plugin_page == $submenu_array[2] ) ) {
     1545                        } elseif ( isset( $plugin_page ) && ($plugin_page == $submenu_array[2] ) ) {
    15461546                                $parent_file = $parent;
    15471547                                return $parent;
    … …  
    15761576                                        $title = $menu_array[3];
    15771577                                        return $menu_array[3];
    1578                                 } elseif ( $plugin_page && ($plugin_page == $menu_array[2] ) && ($hook == $menu_array[3] ) ) {
     1578                                } elseif ( isset( $plugin_page ) && ($plugin_page == $menu_array[2] ) && ($hook == $menu_array[3] ) ) {
    15791579                                        $title = $menu_array[3];
    15801580                                        return $menu_array[3];
    … …  
    15881588                foreach ( array_keys( $submenu ) as $parent ) {
    15891589                        foreach ( $submenu[$parent] as $submenu_array ) {
    1590                                 if ( $plugin_page &&
     1590                                if ( isset( $plugin_page ) &&
    15911591                                        ( $plugin_page == $submenu_array[2] ) &&
    15921592                                        (
    … …  
    15951595                                                ( $plugin_page == $hook ) ||
    15961596                                                ( $pagenow == 'admin.php' && $parent1 != $submenu_array[2] ) ||
    1597                                                 ( $typenow && $parent == $pagenow . '?post_type=' . $typenow)
     1597                                                ( !empty($typenow) && $parent == $pagenow . '?post_type=' . $typenow)
    15981598                                        )
    15991599                                        ) {
    … …  
    16161616                if ( empty ( $title ) ) {
    16171617                        foreach ( $menu as $menu_array ) {
    1618                                 if ( $plugin_page &&
     1618                                if ( isset( $plugin_page ) &&
    16191619                                        ( $plugin_page == $menu_array[2] ) &&
    16201620                                        ( $pagenow == 'admin.php' ) &&
    … …  
    16711671        $parent = get_admin_page_parent();
    16721672
    1673         if ( ! $plugin_page && isset( $_wp_submenu_nopriv[$parent][$pagenow] ) )
     1673        if ( !isset( $plugin_page ) && isset( $_wp_submenu_nopriv[$parent][$pagenow] ) )
    16741674                return false;
    16751675
    1676         if ( $plugin_page ) {
     1676        if ( isset( $plugin_page ) ) {
    16771677                if ( isset( $_wp_submenu_nopriv[$parent][$plugin_page] ) )
    16781678                        return false;
    … …  
    16891689                if ( isset( $_wp_submenu_nopriv[$pagenow][$pagenow] ) )
    16901690                        return false;
    1691                 if ( $plugin_page && isset( $_wp_submenu_nopriv[$pagenow][$plugin_page] ) )
     1691                if ( isset( $plugin_page ) && isset( $_wp_submenu_nopriv[$pagenow][$plugin_page] ) )
    16921692                        return false;
    1693                 if ( $plugin_page && isset( $_wp_menu_nopriv[$plugin_page] ) )
     1693                if ( isset( $plugin_page ) && isset( $_wp_menu_nopriv[$plugin_page] ) )
    16941694                        return false;
    16951695                foreach (array_keys( $_wp_submenu_nopriv ) as $key ) {
    16961696                        if ( isset( $_wp_submenu_nopriv[$key][$pagenow] ) )
    16971697                                return false;
    1698                         if ( $plugin_page && isset( $_wp_submenu_nopriv[$key][$plugin_page] ) )
     1698                        if ( isset( $plugin_page ) && isset( $_wp_submenu_nopriv[$key][$plugin_page] ) )
    16991699                        return false;
    17001700                }
    … …  
    17021702        }
    17031703
    1704         if ( $plugin_page && ( $plugin_page == $parent ) && isset( $_wp_menu_nopriv[$plugin_page] ) )
     1704        if ( isset( $plugin_page ) && ( $plugin_page == $parent ) && isset( $_wp_menu_nopriv[$plugin_page] ) )
    17051705                return false;
    17061706
    17071707        if ( isset( $submenu[$parent] ) ) {
    17081708                foreach ( $submenu[$parent] as $submenu_array ) {
    1709                         if ( $plugin_page && ( $submenu_array[2] == $plugin_page ) ) {
     1709                        if ( isset( $plugin_page ) && ( $submenu_array[2] == $plugin_page ) ) {
    17101710                                if ( current_user_can( $submenu_array[1] ))
    17111711                                        return true;

  • trunk/src/wp-admin/menu-header.php

    r31105 r31106  
    6767                }
    6868
    69                 if ( ( $parent_file && $item[2] == $parent_file ) || ( ! $typenow && $self == $item[2] ) ) {
     69                if ( ( $parent_file && $item[2] == $parent_file ) || ( empty($typenow) && $self == $item[2] ) ) {
    7070                        $class[] = ! empty( $submenu_items ) ? 'wp-has-current-submenu wp-menu-open' : 'current';
    7171                } else {
    … …  
    168168
    169169                                // Handle current for post_type=post|page|foo pages, which won't match $self.
    170                                 $self_type = $typenow ? $self . '?post_type=' . $typenow : 'nothing';
     170                                $self_type = ! empty( $typenow ) ? $self . '?post_type=' . $typenow : 'nothing';
    171171
    172172                                if ( isset( $submenu_file ) ) {
    … …  
    176176                                // This allows plugin pages with the same hook to exist under different parents.
    177177                                } elseif (
    178                                         ( ! $plugin_page && $self == $sub_item[2] ) ||
    179                                         ( $plugin_page && $plugin_page == $sub_item[2] && ( $item[2] == $self_type || $item[2] == $self || file_exists($menu_file) === false ) )
     178                                        ( ! isset( $plugin_page ) && $self == $sub_item[2] ) ||
     179                                        ( isset( $plugin_page ) && $plugin_page == $sub_item[2] && ( $item[2] == $self_type || $item[2] == $self || file_exists($menu_file) === false ) )
    180180                                ) {
    181181                                        $class[] = 'current';

  • trunk/src/wp-includes/deprecated.php

    r31105 r31106  
    25772577        global $plugin_page;
    25782578
    2579         if ( $plugin_page )
     2579        if ( isset($plugin_page) )
    25802580                return true;
    25812581
Note: See TracChangeset for help on using the changeset viewer.