[wp-trac] [WordPress Trac] #19647: Use add_node() instead of add_menu() in core

[WordPress Trac]

#19647: Use add_node() instead of add_menu() in core
-------------------------+-----------------------------------
 Reporter:  linuxologos  |      Owner:
     Type:  enhancement  |     Status:  new
 Priority:  normal       |  Milestone:  Awaiting Review
Component:  Toolbar      |    Version:  3.3
 Severity:  minor        |   Keywords:  has-patch 2nd-opinion
-------------------------+-----------------------------------
 It's clear through [http://core.trac.wordpress.org/browser/tags/3.3/wp-
 includes/class-wp-admin-bar.php class-wp-admin-bar.php] and Nacin
 [http://wpdevel.wordpress.com/2011/12/07/admin-bar-api-changes-in-3-3/ has
 stated this] too:

   The API previously emphasized {{{add_menu()}}}, but this can be
 confusing, so {{{add_node()}}} is now being promoted a bit more.

 Another step towards this would be that the core used {{{add_node()}}}
 instead of {{{add_menu()}}} when building menu items in
 [http://core.trac.wordpress.org/browser/tags/3.3/wp-includes/admin-bar.php
 admin-bar.php].

 Proposed 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()}}}.)

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/19647>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software