Closed (fixed)
Project:
Imagemenu
Version:
6.x-1.1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Jul 2009 at 12:15 UTC
Updated:
22 Mar 2010 at 13:40 UTC
Jump to comment: Most recent file
I have created two imagemenu's each with 5 underlying items.
When i want to edit an existing item in the second menu i get the parent selection list of the first menu.
When i create a new item in the second menu i get the correct parent selection list.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | imagemenu.admin_.inc_.patch | 820 bytes | hovel |
Comments
Comment #1
hovel commentedI have found a solution to this problem. In function imagemenu_edit_item, when $type = 'edit', the menu id ($mid) is not used. Therefor the parents of the first menu are given. The attached patch solves this problem.
I am not fully aware of the requirements and internal logic of this module, so my correction may not be the best way to solve the problem. But it works.
Best regards,
hovel
Comment #2
techninja commentedYep.. Same problem here, awesome job with the patch hovel, I should get in the habit more often myself =)
Why exactly are there no other posts here? So.. Does no one edits their imagemenus? or.. is this module so unpopular as no one cares.. I care!! hovel cared enough to make a patch! He's awesome!
The maintainer should probably have a look to make sure this is kosher before it gets thrown in, but yea, silly bug, easy fix.
Comment #3
EastWan commentedSame problem here. Patch works just fine.
Thanks!
Comment #4
brankoc commentedThis report appears to be a duplicate of http://drupal.org/node/298562.
The patch in this thread (523354) seems to work, the one in the original thread (298562) doesn't do anything for me.
Comment #5
marcus_clements commentedI've tested this patch and it works, will be included in an imminent release.
Thanks :)
Comment #6
marcus_clements commentedFixed in latest release