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.

Comments

cburschka’s picture

Priority: Normal » Critical
Status: Active » Reviewed & tested by the community
StatusFileSize
new628 bytes

hook_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.

cburschka’s picture

Status: Reviewed & tested by the community » Fixed

Fixed in commit #102066.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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

cburschka’s picture

Version: 6.x-1.0 » 5.x-1.3
Status: Closed (fixed) » Patch (to be ported)

The 5.x version still has this problem.

cburschka’s picture

Status: Patch (to be ported) » Needs review
StatusFileSize
new1.52 KB

Here's the patch that adds MENU_CALLBACK type.

cburschka’s picture

Version: 5.x-1.3 » 5.x-1.4
Status: Needs review » Fixed

Fixed and in 5.x-1.4

Anonymous’s picture

Status: Fixed » Closed (fixed)

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