[wp-trac] [WordPress Trac] #14527: ajax call for add-menu-item doesnt supprot wp_edit_nav_menu_walker filter

[WordPress Trac]

#14527: ajax call for add-menu-item doesnt supprot  wp_edit_nav_menu_walker filter
------------------------------+--------------------
 Reporter:  wonderslug        |       Owner:
     Type:  defect (bug)      |      Status:  new
 Priority:  normal            |   Milestone:  3.4
Component:  Menus             |     Version:  3.0.1
 Severity:  normal            |  Resolution:
 Keywords:  has-patch commit  |
------------------------------+--------------------

Comment (by nacin):

 -1 implies a permissions issue, 0 is generally used for failures. (Does
 the code that triggers this ajax know to look for -1/0?)

 Normally, we'd probably avoid the class_exists() (and show devs their
 error), since this is already in a released version, it makes sense to
 avoid the errors.

 A few plugins use the wp_edit_nav_menu_walker filter. How many of them
 don't load the class they want via Ajax?

 {{{
 ./plugins/gantry/widgets/menu.php:51:
 add_filter('wp_edit_nav_menu_walker', array("GantryWidgetMenu",
 "setupWalker"), 1000, 2);
 ./plugins/gecka-submenu/models/NavMenuHacks.php:10:            add_filter(
 'wp_edit_nav_menu_walker', array($this, 'custom_walker') );
 ./plugins/menu-items-visibility-control/menu-items-visibility.php:26:
 add_filter( 'wp_edit_nav_menu_walker', 'boom_nav_menu_edit_custom_walker'
 );
 }}}

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/14527#comment:3>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software