Closed (fixed)
Project:
Active menus
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
7 May 2008 at 03:09 UTC
Updated:
5 Jul 2008 at 02:13 UTC
I have book menus on my site inside the block provided by the book menu. Active menus works fine on the main navigation menu, but fails on the book menus. Active Menus basically hangs looking for the links.
Comments
Comment #1
nedjoDavid, good to hear from albeit through the issue queue.
Looks like I forgot to test with book (which was extensively rewritten for 6.x).
Quick fix: delete modules/book.inc.
I'll have a look at a fix when I get a chance.
Comment #2
berenErchamion commentedI'm having the same problem. Great module - adding the book support would be super cool.
Comment #3
korvus commentedFor the record, I am pretty new to Drupal modules and very new to Drupal 6 modules.
With that out of the way, I believe modifying modules/book.inc to replace the existing activemenu_js_book() function with this one makes activemenu work for books (I only have one book on my site, so I've only tested it under the most basic of circumstances).
If someone really wants I can make a proper patch, but I don't have the Drupal CVS set up on the machine where I made this change. And it's a change to a very small file.
Comment #4
nedjoThanks for submitting the fix!
Anyone available to test and mark RTBC?
Comment #5
dgeilhufe@yahoo.com commentedTested and did not work for me.
Menus did not expand and got this warning:
user warning: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'ORD\ ER BY mlid ASC LIMIT 0, 1' at line 1 query: SELECT mlid FROM community_menu_links WHERE link_path = 'node/0' AND module = 'book' ORD\ ER BY mlid ASC LIMIT 0, 1 in /..../sites/all/modules/activemenu/modules/book.inc on line 46.
I have four books and a multilevel hierarchy.
Comment #6
nedjoI committed a patch that seems to work. Pls reopen if not.
Comment #7
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.