In dependent.js, loop variables are not declared causing them to be stored in the global namespace which can cause name collisions if some other module introduces the same bug.

CommentFileSizeAuthor
#1 views-1003760.patch2.21 KBrecrit

Comments

recrit’s picture

recrit commented
StatusFileSize
new views-1003760.patch2.21 KB

attached patch fixes this issue.

dawehner’s picture

dawehner
Primary language German
commented

This might need a fix in ctools, too.

iamjon’s picture

iamjon commented
Status: Active » Needs review

changing status

merlinofchaos’s picture

merlinofchaos commented
Status: Needs review » Fixed

Yes. Something similar needed for CTools in both D6 and D7. Any takers?

Status: Fixed » Closed (fixed)

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