Hi,

Full text submenu (with or without links) is not displayed when Panels 3 module is activated.

This is because Panels 3 requires Delegator modules. This module handles 'page_callback' as 'delegator_node_view' and not as 'node_page_view'.

This makes the test at line 346 of submenutree.module fail (if ($v['link']['hidden'] == false && $v['link']['page_callback'] == 'node_page_view') ...).

Proposed fix :

      if ($v['link']['hidden'] == false && 
        ($v['link']['page_callback'] == 'node_page_view' || $v['link']['page_callback'] == 'delegator_node_view') )

Thanks

Laurent

Comments

bengtan’s picture

bengtan commented
Status: Active » Fixed

Hi,

I have no idea what you guys are doing in Panels 3, but I'll take your word for it :)

I have committed a fix, but differently from yours: 

      if ($v['link']['hidden'] == false && in_array($v['link']['page_callback'], array('node_page_view', 'delegator_node_view'))) {

Either your version or my version will work.

Now, do you need me to make an official release for this, or are you okay with patching your system and waiting indefinitely?

agence web coheractio’s picture

agence web coheractio commented

Thanks bengtan

I'm fine with patching my system.

Laurent

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.