Closed (fixed)
Project:
DHTML Menu
Version:
7.x-1.x-dev
Component:
Javascript code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Anonymous (not verified)
Created:
16 Apr 2011 at 09:07 UTC
Updated:
28 Oct 2019 at 17:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
yuriy.babenko commentedExperiencing the same issue without the media module. Latest 7.x version breaks all JS on the site with the same error as the OP.
Comment #2
yuriy.babenko commentedFixed the bug on my local copy. Attaching patch for review.
Comment #3
pianomansam commentedThis is a huge error that is corrected by the patch in #2. Please get this committed ASAP as it is breaking site' JS.
Comment #4
Rory commentedAlternatively you can apply the patch here which fixes this issue, and another.
Comment #5
vadym.kononenko commentedPatch from #2 solved my issue too (and patch from #4 is bad).
Comment #6
vadym.kononenko commentedI've modified a little patch from #2.
It use size only from '.menu li.dhtml-folder' now that is more correct.
Comment #7
RStrydom commentedI came across the same error and all JavaScript on my site did not want to work. This only happened once I selected the "Expand on Bullet" navigation option. The patch from #6 has worked for me. (I did not test #2 as I preferred the more specific css selector)
Comment #8
dspcore commentedhow to add these js patch for dhtml menu in drupal 7.x?
Comment #9
rooby commented@dspcore:
If you are on command line linux/mac/etc. you can copy the patch in comment #6 into the dhtml_menu module directory, cd to it, then run
patch -p < dhtml_menu-dhtml-folder.patchFor more thorough information on applying patches see https://www.drupal.org/patch/apply
Comment #10
benleivian commentedThanks for the patches (I used #6).
Comment #11
vuilRe-roll the patch of #6 to the latest 7.x-1.x-dev branch, and assign the issue to that branch.
Comment #13
vuilCommit the issue and close it as Fixed. Thank you!
Comment #15
vuil