On page admin/config/user-interface/dynamic_persistent_menu, I can't edit or delete menus, only create new ones.

Comments

alberto56’s picture

Priority: Normal » Critical

The existance of several dynamic menus in the system can cause #1360030: dpms.menus[settings_id] is undefined when more than one dynamic menu defined, prevents submenus from hiding after timeout, so right now for this module to work properly only one dynamic menu can be defined. Hence, fixing this is critical.

alberto56’s picture

Same for editing dynamic menus directly on the block page.

alberto56’s picture

Version: 7.x-2.0-alpha1 » 7.x-2.0-alpha2
StatusFileSize
new1.43 KB

The enclosed patch, when applied to 7.x-2.x-dev (not 7.x-2.0-alpha2), creates a failing test demonstrating this problem. Fixing the problem should make this test pass.

alberto56’s picture

I created a new git branch for this issue called 1356324. It contains the patch at #3 (which fails).

alberto56’s picture

StatusFileSize
new6.61 KB

There are still some glitches on the 1356324 branch, especially related to deleting dynamic persistent menus when they are already related to a block. All this is in the test: you can either check out branch 1356324 from git or apply the enclosed patch to the 7.x branch.

I will merge this into 7.x when all tests will pass.

Cheers,

Albert.

alberto56’s picture

Status: Active » Fixed

OK, fixed. All tests pass on the 7.x-2.x branch. See also related #1499524: Impossible to modify a dynamic menu directly in the block config page

Status: Fixed » Closed (fixed)

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