Closed (fixed)
Project:
DHTML Menu
Version:
5.x-1.4
Component:
PHP Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
18 Feb 2008 at 13:12 UTC
Updated:
7 Jul 2008 at 09:12 UTC
Jump to comment: Most recent file
When installed, DHTML Menu offers a link to replace the static navigation block automatically. This link should ideally stop working as soon as it's used, because visiting it again will break the configuration by disabling the DHTML menu block entirely. Currently, it not only remains operational but is in fact visible in the menu as a sub-item of admin/settings/dhtml_menu, which is pretty silly.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | dhtml_menu-menu-callback-d5-223251-5.patch | 1.52 KB | cburschka |
| #1 | dhtml_menu-menu-callback-223251-1.patch | 628 bytes | cburschka |
Comments
Comment #1
cburschkahook_menu should set the type of this item to MENU_CALLBACK instead of the default.
Ideally, the item should also become inaccessible once called, but that is a different issue that requires more code.
Comment #2
cburschkaFixed in commit #102066.
Comment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #4
cburschkaThe 5.x version still has this problem.
Comment #5
cburschkaHere's the patch that adds MENU_CALLBACK type.
Comment #6
cburschkaFixed and in 5.x-1.4
Comment #7
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.