'Object doesn't support this property' error: Fix provided [#282284]

Build with this ID:
$Id: dhtml_menu.js,v 1.10.2.10 2008/04/15 00:05:34 brmassa Exp $
contains this line:
menu = document.getElementById('menu-' + cookieList[i]);
That causes a runtime error to display in IE: Object doesn't support this property or method.

For this release only (?), the fix is to modify dhtml_menu.js and change that one line to:
var menu = document.getElementById('menu-' + cookieList[i]);
Note the 'var' keyword is required to create this new object.

I didn't check intermediate releases but the following latest dev cut (6.x.3-3.0-alpha) doesn't contain that line anymore:
dhtml_menu.js,v 1.16 2008/07/08 13:44:25 arancaytar Exp $

It might be worth it to anyone running v1.0.5 or prior to check their code.

I hope that helps someone.

Comment	File	Size	Author
#5	dhtml_menu-6-3-var-282284-5.patch	817 bytes	cburschka
#3	dhtml_menu_6-1-var-282284-3.patch	1.83 KB	cburschka
#3	dhtml_menu_6-2-var-282284-3.patch	1.84 KB	cburschka

Comments

Comment #1

cburschka

they

commented 14 July 2008 at 08:07

Version:

6.x-1.05

» 6.x-2.x-dev

It might still be in 6.x-2.x-dev, and I'll fix it there.

Comment #2

cburschka

they

commented 14 July 2008 at 08:14

I'm assuming you also need to declare the other variables? There's submenu right above that. I don't claim to understand the mysterious ways of IE, but it would make sense to put a var before all the first usages of the variable.