I have a menu with a menu item Management Consulting. The path for this item is Management-Consulting. The item has children, but does not expand when clicked.

If I change the path to ManagementConsulting (no hypen), it does expand when clicked.

I put a print in the code, and see that in one case $_GET['q'] contains the path, and in the other it contains the node (eg node/24). I don't know why.

Comments

j. ayen green’s picture

j. ayen green commented

Additional info: this problem only seems to happen in conjunction with Paths being used.

j. ayen green’s picture

j. ayen green commented
Title: Parent items do not expand when clicked if parent path contains a hypen » Child items not selected if path in imagemenu item is an alias

What I had thought was the problem originally was not. I was getting retrieval on the children of some items but not others. It turns out, for example, that I had a page with an alias of 'Consulting' with two children. The children would not appear. I did some displaying inside the module to find the problem.

The module retrieves the children based on the url query string = the path in the parent entry. The problem was that the query string was coming back "node/111" even though the node has an alias, so the db select was coming back empty.

I've fixed it in my copy temporarily by doing the drupal call to retrieve the path's alias. If there is an alias, I have the select statement retrieve based on path = '%s' OR path = '%s' ($path,$alias) and it works fine. This needs to be incorporated into the code.

j. ayen green’s picture

j. ayen green commented
Title: Child items not selected if path in imagemenu item is an alias » Child items not selected if path is alias
marcus_clements’s picture

marcus_clements commented
Status: Active » Postponed

I'll look into this for the next release

buzzman’s picture

buzzman commented

any updates on this? bumping

Drupality’s picture

Drupality commented

Im also interested in this. Since this module isnt working with url aliases, its kinda useless :I
Also it would be quite nice, if the JS preload change will be included as well. I think both are important.

pobster’s picture

pobster commented
Issue summary: View changes
Status: Postponed » Closed (outdated)

Closing as D6.x is now unsupported.