Closed (fixed)
Project:
The Better Mega Menu
Version:
8.x-1.x-dev
Component:
User interface
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
20 Jan 2022 at 11:19 UTC
Updated:
15 Mar 2022 at 22:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
knaffles commentedCan you try applying the attached patch and see if it fixes the issue for you? Thanks!
Comment #3
crasx commentedThanks so much for the fix! Working well on my end. To validate I did the following:
- Moved a child menu item using the menu UI to place tbmm in a "broken" state.
- Confirmed the error was present in the tbmm UI (ie: Saw my links had been duplicated)
- Applied the above patch
- Refreshed the tbmm page and confirmed the error was gone (ie: did not see duplicate items).
I updated the patch to remove an unused line, also added some spacing
Comment #4
imclean commentedI can confirm the patch in #2 removes the duplicate links for us.
Comment #5
imclean commentedI didn't mean to hide #3, I hadn't refreshed the browser before posting. #3 looks good but adds 2 blank lines. These should be removed from the patch as they aren't related to the fix.
Comment #6
imclean commentedComment #7
crasx commentedFair!
Comment #10
knaffles commentedPatch #7 has been merged into the 8.x-1.x branch and will be included in the next release. Thanks everyone!