[wp-trac] [WordPress Trac] #23119: UX Improvements to nav-menus.php

[WordPress Trac]

#23119: UX Improvements to nav-menus.php
-----------------------------------+------------------
 Reporter:  lessbloat              |       Owner:
     Type:  enhancement            |      Status:  new
 Priority:  normal                 |   Milestone:  3.6
Component:  Menus                  |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  3.6-menus needs-patch  |
-----------------------------------+------------------

Comment (by DrewAPicture):

 RE: the notice from comment:213 / comment:220:
 {{{
 Notice: Undefined offset: 0 in wp-admin/nav-menus.php on line 475
 }}}

 After a conversation [https://irclogs.wordpress.org/chanlog.php?channel
 =wordpress-dev&day=2013-02-16&sort=asc#m557309 in IRC], @markoheijnen
 pasted a var_dump of `$menu_locations`:
 {{{
  <DrewAPicture> markoheijnen: Can you share a var_dump on $menu_locations
 from before the foreach?
  <markoheijnen> array(2) {
  <markoheijnen> [0]=>
  <markoheijnen> int(2)
  <markoheijnen> ["primary"]=>
  <markoheijnen> int(2)
  <markoheijnen> }
 }}}

 My best guess is there's a problem with `set_theme_mod()` setting an
 invalid, absint-ed menu location with a value of 0, which therefore
 translates into an undefined offset into the second foreach for the select
 box.

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