Seems to break DHTML Menu
fred0 - March 31, 2009 - 05:56
| Project: | me aliases |
| Version: | 6.x-2.0 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
Just tried installing Me. Right off the bat, it broke DHTML Menu. I'm running Drupal 6.10 and DHTML Menu 3.4.
#1
Yup, it will do that. HAHA. Damn :(
The problem appears to be that both modules attempt to override
theme_menu_item_link. I'll need to ponder on this one.#2
I've just put together a patch that I hope will fix this. Can you apply the patch and let me know if it is still broken? I'll try and install DHTML Menu in the mean time. Be sure to clear all your caches before passing final verdict. :)
#3
Wow! Speedy response!
Yeah... that seems to fix it. Also seems to fix another issue I was seeing that broke the display of tabs in my custom genesis based theme.
However, now that it works, it looks like the use I had intended isn't supported... I had hoped to make custom menu items but trying to add a menu item with the path /user/me or /user/me/whatever fails. Is this not a valid use of this module?
#4
hmmm... you should be able to add a custom menu item with path user/me... I just did a test myself with both "user/me" and "user/me/edit".
I'm assuming that user/[uid]/whatever are valid paths that drupal knows about? If not, then no, the me module just allows the 'insertion' of 'me' in a place where a uid would normally be required, and dynamically swaps it out with the current users uid when the path is hit.
Did you try creating the paths without the leading slash? I don't think the leading slash is supported there. Drupal takes care of making sure the slash is put in when needed.
#5
Duh. The leading slash was the problem. Thanks!
#6
Commited
#7
Automatically closed -- issue fixed for 2 weeks with no activity.