Nav Walker, why do you hate me? (I'm missing something here)

Hi,

They’re a very approachable subject although some people can get lost in recursive functions. They’re hard to debug.

This isn’t my first nav walker extend. I made it conform to Bootstrap previously and had no such issues.

I am practically doing the same exact thing as the “advanced” walker. If ( 0 != $depth ) return; is the inverse of the tutorials if ($depth == 0) return; because we want inverse things. I want only main nav items, they want only nav items under the current nav.

The bit unorthodox 0 != $depth instead of $depth > 0 is just a JavaScript habit for other reasons but the statement doesn’t work any way I lay it out anyhow.

I even put an extra debug line in start/end_lvl() where the offending <ul class=”sub-menu”> and are being invalidly created. I put it right under the $depth check, and it prints out! This is one of these “but that’s just not possible” times. e.g.:

function start_lvl(...){
  if ($depth > 0) { return; }
  print "<!-- debug, huh how am I here? -->";
  print "<ul class=\"sub-nav222222\" ...";
}

If the $depth is above 0 there is no freaking way that should print. start_el and end_el do not print, they respect the return. However, both the comment is printed (ONLY around the main nav, not this offending sub-nav), and I verified the class was sub-nav222222 just to make sure this
< ul > wasn’t being printed from something I’m inheriting. Nope, the class was sub-nav222222 so I know that function is printing it.

This makes no sense! I’m sure there’s a reason but if anyone knows how this possible, please save my sanity!

Otherwise the only thing I see from here is needing to make the walker class specifically to get secondary nav items from the current selection and set the depth => 1 in arguments on the main menu, essentially making me walk nav twice. I should be able to get the main nav and the subnav of the selected item in a single walk. That or I’m going to need to set .sub-nav{display:none;} which is equally impure.

Thanks very much for pointing that out. I’m not even sure if they intend this or if messing with any of this code is going to desync anything internally (I thought perhaps classes), but it doesn’t appear to.

I’m now overriding display_element grabbed from class-wp-walker.php and just added $depth+1 on line 156, which was:

$cb_args = array_merge( array(&$output, $depth), $args);

But is now:

$cb_args = array_merge( array( &$output, $depth + 1 ), $args);

I’d love to know their reasoning on why the depth should start at 0 and I’ve run through my many-tier menus and they all work just fine with this change. The code clearly states “$newlevel = true;”, so why wouldn’t you consider adding a depth? Most importantly it doesn’t feel hacky, no classes changed and the way I wanted to do it now works perfectly fine with:

if ( $depth > 0 ) { return; }

Thanks a bunch for pointing me to those lines bcworkz!

Again I’m not sure if there is a context in which doing this is going to break the menu system output, but so far it’s working perfectly fine. If anyone thinks this is a bad practice please let me know as well as a better way to do it!

I totally agree with starting at 0. However where the next depth starts should be extremely clean and clear to me. In that case, it’s the next container of nav.

For example in a typical nested unordered list, this makes sense to me:

<ul depth=0>
  <li>
    <ul depth=1>
      <li>
        <ul depth=2> .. </ul>
      </li>
     </ul>
  </li>
</ul>

But how it is right now, this happens:

<ul depth=0>
  <li>
    <ul>
      <li depth=1>
        <ul>
          <li depth=2></li>
        </ul>
      </li>
     </ul>
  </li>
</ul>

To the navigation elements themselves which are in the < li > list items the depth will work fine but from any other angle I look at this it’s just invalid markup.

The depth doesn’t need to be reduced for the rest of the original < ul >s content on the main menu because the $depth+1 is being passed as an argument to the function, never actually affecting the functions current $depth.

I can create navs all day and this prints reliably, such as:

Main 1
  Sub 1
    Sub 2
Main 2
Main 3
  Sub 1
Main 4
  Sub 1
    Sub 2
      Sub 3
        Sub 4
      Sub 3

And that is exactly how it prints out when I print $indent+$depth:

0
  1
    2
0
0
  1
    2
      3
        4
      3

I can’t see how this is more desirable which is what I get if I don’t use $depth+1:

0
  0
    1
0
0
  0
    1
      2
        3
      2

Perhaps there’s just still something internal I don’t understand about that being desirable but nonetheless this is working perfectly for me. Thank you again!