WordPress.org

Make WordPress Core

Changeset 46642


Ignore:
Timestamp:
11/03/2019 10:12:32 PM (2 weeks 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.