WordPress.org
Get WordPress

Make WordPress Core

Changeset 46642


Ignore:
Timestamp:
11/03/2019 10:12:32 PM (7 years ago)
Author:
whyisjake
Message:

Toolbar: Use add_node() instead of add_menu() in core.

This patch replaces all references to the add_menu() method with the add_node() one. (Also some code structure modifications for wp_admin_bar_appearance_menu().)

Fixes: #19647
Props: linuxologos, paulschreiber, morganestes, akibjorklund, nacin, whyisjake.

Location:
trunk
Files:
4 edited

Legend:

Unmodified
Added
Removed

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

    r46164 r46642  
    132132        }
    133133
    134         $wp_admin_bar->add_menu( $wp_logo_menu_args );
     134        $wp_admin_bar->add_node( $wp_logo_menu_args );
    135135
    136136        if ( $about_url ) {
    137137                // Add "About WordPress" link
    138                 $wp_admin_bar->add_menu(
     138                $wp_admin_bar->add_node(
    139139                        array(
    140140                                'parent' => 'wp-logo',
    … …  
    147147
    148148        // Add WordPress.org link
    149         $wp_admin_bar->add_menu(
     149        $wp_admin_bar->add_node(
    150150                array(
    151151                        'parent' => 'wp-logo-external',
    … …  
    157157
    158158        // Add codex link
    159         $wp_admin_bar->add_menu(
     159        $wp_admin_bar->add_node(
    160160                array(
    161161                        'parent' => 'wp-logo-external',
    … …  
    167167
    168168        // Add forums link
    169         $wp_admin_bar->add_menu(
     169        $wp_admin_bar->add_node(
    170170                array(
    171171                        'parent' => 'wp-logo-external',
    … …  
    177177
    178178        // Add feedback link
    179         $wp_admin_bar->add_menu(
     179        $wp_admin_bar->add_node(
    180180                array(
    181181                        'parent' => 'wp-logo-external',
    … …  
    196196function wp_admin_bar_sidebar_toggle( $wp_admin_bar ) {
    197197        if ( is_admin() ) {
    198                 $wp_admin_bar->add_menu(
     198                $wp_admin_bar->add_node(
    199199                        array(
    200200                                'id'    => 'menu-toggle',
    … …  
    234234        $class = empty( $avatar ) ? '' : 'with-avatar';
    235235
    236         $wp_admin_bar->add_menu(
     236        $wp_admin_bar->add_node(
    237237                array(
    238238                        'id'     => 'my-account',
    … …  
    284284        }
    285285
    286         $wp_admin_bar->add_menu(
     286        $wp_admin_bar->add_node(
    287287                array(
    288288                        'parent' => 'user-actions',
    … …  
    297297
    298298        if ( false !== $profile_url ) {
    299                 $wp_admin_bar->add_menu(
     299                $wp_admin_bar->add_node(
    300300                        array(
    301301                                'parent' => 'user-actions',
    … …  
    307307        }
    308308
    309         $wp_admin_bar->add_menu(
     309        $wp_admin_bar->add_node(
    310310                array(
    311311                        'parent' => 'user-actions',
    … …  
    351351        $title = wp_html_excerpt( $blogname, 40, '…' );
    352352
    353         $wp_admin_bar->add_menu(
     353        $wp_admin_bar->add_node(
    354354                array(
    355355                        'id'    => 'site-name',
    … …  
    363363        if ( is_admin() ) {
    364364                // Add an option to visit the site.
    365                 $wp_admin_bar->add_menu(
     365                $wp_admin_bar->add_node(
    366366                        array(
    367367                                'parent' => 'site-name',
    … …  
    373373
    374374                if ( is_blog_admin() && is_multisite() && current_user_can( 'manage_sites' ) ) {
    375                         $wp_admin_bar->add_menu(
     375                        $wp_admin_bar->add_node(
    376376                                array(
    377377                                        'parent' => 'site-name',
    … …  
    384384        } elseif ( current_user_can( 'read' ) ) {
    385385                // We're on the front end, link to the Dashboard.
    386                 $wp_admin_bar->add_menu(
     386                $wp_admin_bar->add_node(
    387387                        array(
    388388                                'parent' => 'site-name',
    … …  
    429429        }
    430430
    431         $wp_admin_bar->add_menu(
     431        $wp_admin_bar->add_node(
    432432                array(
    433433                        'id'    => 'customize',
    … …  
    466466        }
    467467
    468         $wp_admin_bar->add_menu(
     468        $wp_admin_bar->add_node(
    469469                array(
    470470                        'id'    => 'my-sites',
    … …  
    482482                );
    483483
    484                 $wp_admin_bar->add_menu(
     484                $wp_admin_bar->add_node(
    485485                        array(
    486486                                'parent' => 'my-sites-super-admin',
    … …  
    491491                );
    492492
    493                 $wp_admin_bar->add_menu(
     493                $wp_admin_bar->add_node(
    494494                        array(
    495495                                'parent' => 'network-admin',
    … …  
    501501
    502502                if ( current_user_can( 'manage_sites' ) ) {
    503                         $wp_admin_bar->add_menu(
     503                        $wp_admin_bar->add_node(
    504504                                array(
    505505                                        'parent' => 'network-admin',
    … …  
    512512
    513513                if ( current_user_can( 'manage_network_users' ) ) {
    514                         $wp_admin_bar->add_menu(
     514                        $wp_admin_bar->add_node(
    515515                                array(
    516516                                        'parent' => 'network-admin',
    … …  
    523523
    524524                if ( current_user_can( 'manage_network_themes' ) ) {
    525                         $wp_admin_bar->add_menu(
     525                        $wp_admin_bar->add_node(
    526526                                array(
    527527                                        'parent' => 'network-admin',
    … …  
    534534
    535535                if ( current_user_can( 'manage_network_plugins' ) ) {
    536                         $wp_admin_bar->add_menu(
     536                        $wp_admin_bar->add_node(
    537537                                array(
    538538                                        'parent' => 'network-admin',
    … …  
    545545
    546546                if ( current_user_can( 'manage_network_options' ) ) {
    547                         $wp_admin_bar->add_menu(
     547                        $wp_admin_bar->add_node(
    548548                                array(
    549549                                        'parent' => 'network-admin',
    … …  
    581581
    582582                if ( current_user_can( 'read' ) ) {
    583                         $wp_admin_bar->add_menu(
     583                        $wp_admin_bar->add_node(
    584584                                array(
    585585                                        'parent' => 'my-sites-list',
    … …  
    590590                        );
    591591
    592                         $wp_admin_bar->add_menu(
     592                        $wp_admin_bar->add_node(
    593593                                array(
    594594                                        'parent' => $menu_id,
    … …  
    599599                        );
    600600                } else {
    601                         $wp_admin_bar->add_menu(
     601                        $wp_admin_bar->add_node(
    602602                                array(
    603603                                        'parent' => 'my-sites-list',
    … …  
    610610
    611611                if ( current_user_can( get_post_type_object( 'post' )->cap->create_posts ) ) {
    612                         $wp_admin_bar->add_menu(
     612                        $wp_admin_bar->add_node(
    613613                                array(
    614614                                        'parent' => $menu_id,
    … …  
    621621
    622622                if ( current_user_can( 'edit_posts' ) ) {
    623                         $wp_admin_bar->add_menu(
     623                        $wp_admin_bar->add_node(
    624624                                array(
    625625                                        'parent' => $menu_id,
    … …  
    631631                }
    632632
    633                 $wp_admin_bar->add_menu(
     633                $wp_admin_bar->add_node(
    634634                        array(
    635635                                'parent' => $menu_id,
    … …  
    661661        $html = '<input class="shortlink-input" type="text" readonly="readonly" value="' . esc_attr( $short ) . '" />';
    662662
    663         $wp_admin_bar->add_menu(
     663        $wp_admin_bar->add_node(
    664664                array(
    665665                        'id'    => $id,
    … …  
    703703                        if ( 'draft' == $post->post_status ) {
    704704                                $preview_link = get_preview_post_link( $post );
    705                                 $wp_admin_bar->add_menu(
     705                                $wp_admin_bar->add_node(
    706706                                        array(
    707707                                                'id'    => 'preview',
    … …  
    712712                                );
    713713                        } else {
    714                                 $wp_admin_bar->add_menu(
     714                                $wp_admin_bar->add_node(
    715715                                        array(
    716716                                                'id'    => 'view',
    … …  
    736736                        $tax = get_taxonomy( $tag->taxonomy );
    737737                        if ( is_taxonomy_viewable( $tax ) ) {
    738                                 $wp_admin_bar->add_menu(
     738                                $wp_admin_bar->add_node(
    739739                                        array(
    740740                                                'id'    => 'view',
    … …  
    748748                        $view_link   = get_author_posts_url( $user_object->ID );
    749749                        if ( $user_object->exists() && $view_link ) {
    750                                 $wp_admin_bar->add_menu(
     750                                $wp_admin_bar->add_node(
    751751                                        array(
    752752                                                'id'    => 'view',
    … …  
    771771                                && current_user_can( 'edit_post', $current_object->ID )
    772772                                && $post_type_object->show_in_admin_bar ) {
    773                                 $wp_admin_bar->add_menu(
     773                                $wp_admin_bar->add_node(
    774774                                        array(
    775775                                                'id'    => 'edit',
    … …  
    783783                        $edit_term_link = get_edit_term_link( $current_object->term_id, $current_object->taxonomy );
    784784                        if ( $tax && $edit_term_link && current_user_can( 'edit_term', $current_object->term_id ) ) {
    785                                 $wp_admin_bar->add_menu(
     785                                $wp_admin_bar->add_node(
    786786                                        array(
    787787                                                'id'    => 'edit',
    … …  
    794794                        $edit_user_link = get_edit_user_link( $current_object->ID );
    795795                        if ( $edit_user_link ) {
    796                                 $wp_admin_bar->add_menu(
     796                                $wp_admin_bar->add_node(
    797797                                        array(
    798798                                                'id'    => 'edit',
    … …  
    860860        $title = '<span class="ab-icon"></span><span class="ab-label">' . _x( 'New', 'admin bar menu group label' ) . '</span>';
    861861
    862         $wp_admin_bar->add_menu(
     862        $wp_admin_bar->add_node(
    863863                array(
    864864                        'id'    => 'new-content',
    … …  
    871871                list( $title, $id ) = $action;
    872872
    873                 $wp_admin_bar->add_menu(
     873                $wp_admin_bar->add_node(
    874874                        array(
    875875                                'parent' => 'new-content',
    … …  
    906906        $title .= '<span class="screen-reader-text comments-in-moderation-text">' . $awaiting_text . '</span>';
    907907
    908         $wp_admin_bar->add_menu(
     908        $wp_admin_bar->add_node(
    909909                array(
    910910                        'id'    => 'comments',
    … …  
    931931
    932932        if ( current_user_can( 'switch_themes' ) ) {
    933                 $wp_admin_bar->add_menu(
     933                $wp_admin_bar->add_node(
    934934                        array(
    935935                                'parent' => 'appearance',
    … …  
    946946
    947947        if ( current_theme_supports( 'widgets' ) ) {
    948                 $wp_admin_bar->add_menu(
     948                $wp_admin_bar->add_node(
    949949                        array(
    950950                                'parent' => 'appearance',
    … …  
    957957
    958958        if ( current_theme_supports( 'menus' ) || current_theme_supports( 'widgets' ) ) {
    959                 $wp_admin_bar->add_menu(
     959                $wp_admin_bar->add_node(
    960960                        array(
    961961                                'parent' => 'appearance',
    … …  
    968968
    969969        if ( current_theme_supports( 'custom-background' ) ) {
    970                 $wp_admin_bar->add_menu(
     970                $wp_admin_bar->add_node(
    971971                        array(
    972972                                'parent' => 'appearance',
    … …  
    982982
    983983        if ( current_theme_supports( 'custom-header' ) ) {
    984                 $wp_admin_bar->add_menu(
     984                $wp_admin_bar->add_node(
    985985                        array(
    986986                                'parent' => 'appearance',
    … …  
    10151015        $title .= '<span class="screen-reader-text">' . $update_data['title'] . '</span>';
    10161016
    1017         $wp_admin_bar->add_menu(
     1017        $wp_admin_bar->add_node(
    10181018                array(
    10191019                        'id'    => 'updates',
    … …  
    10451045        $form .= '</form>';
    10461046
    1047         $wp_admin_bar->add_menu(
     1047        $wp_admin_bar->add_node(
    10481048                array(
    10491049                        'parent' => 'top-secondary',
    … …  
    10741074        $url = wp_nonce_url( $url, WP_Recovery_Mode::EXIT_ACTION );
    10751075
    1076         $wp_admin_bar->add_menu(
     1076        $wp_admin_bar->add_node(
    10771077                array(
    10781078                        'parent' => 'top-secondary',

  • trunk/src/wp-includes/blocks/latest-comments.php

    r46607 r46642  
    4242 */
    4343function render_block_core_latest_comments( $attributes = array() ) {
     44        // This filter is documented in wp-includes/widgets/class-wp-widget-recent-comments.php.
    4445        $comments = get_comments(
    45                 /** This filter is documented in wp-includes/widgets/class-wp-widget-recent-comments.php */
    4646                apply_filters(
    4747                        'widget_comments_args',

  • trunk/src/wp-includes/class-wp-admin-bar.php

    r46140 r46642  
    8383
    8484        /**
    85          * @param array $node
     85         * Add a node (menu item) to the Admin Bar menu.
     86         *
     87         * @since 3.3.0
     88         * @since 5.4.0 Deprecated in favor of {@see WP_Admin_Bar::add_node()}.
     89         *
     90         * @param array $node The attributes that define the node.
    8691         */
    8792        public function add_menu( $node ) {
     93                _deprecated_function( __METHOD__, '5.4.0', __CLASS__ . '::add_node()' );
    8894                $this->add_node( $node );
    8995        }
    9096
    9197        /**
    92          * @param string $id
     98         * Remove a node from the admin bar.
     99         *
     100         * @since 3.1.0
     101         * @since 5.4.0 Deprecated in favor of {@see WP_Admin_Bar::remove_node()}.
     102         *
     103         * @param string $id The menu slug to remove.
    93104         */
    94105        public function remove_menu( $id ) {
     106                _deprecated_function( __METHOD__, '5.4.0', __CLASS__ . '::remove_node()' );
    95107                $this->remove_node( $id );
    96108        }

  • trunk/tests/phpunit/tests/adminbar.php

    r46586 r46642  
    347347
    348348        /**
     349         * @ticket 19647
     350         */
     351        public function test_deprecated_menu_methods() {
     352                $admin_bar = new WP_Admin_Bar();
     353                $node_data = array( 'id' => 'test-node-19647' );
     354
     355                // Test add_menu() deprecation.
     356                $this->setExpectedDeprecated( 'WP_Admin_Bar::add_menu' );
     357                $admin_bar->add_menu( $node_data );
     358                $node_test = $admin_bar->get_node( 'test-node-19647' );
     359                $this->assertNotNull( $node_test );
     360
     361                // Test remove_menu() deprecation.
     362                $this->setExpectedDeprecated( 'WP_Admin_Bar::remove_menu' );
     363                $admin_bar->remove_menu( 'test-node-19647' );
     364                $node_test = $admin_bar->get_node( 'test-node-19647' );
     365                $this->assertNull( $node_test );
     366        }
     367
     368        /**
    349369         * @ticket 22247
    350370         */
Note: See TracChangeset for help on using the changeset viewer.