183170 - Get a "1 exceeds the number of menu layers" javascript error

Status: VERIFIED DUPLICATE of bug 88596

(Core :: JavaScript Engine, defect)

Description

[Aris Stathakis]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2.1) Gecko/20021130
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2.1) Gecko/20021130

Going to http://www.pcworld.co.uk/ when putting the mouse cursor over the manu
menu on the left of the screen undre "browse pc world", the following error is
displayed in a JavaScript Application error window:

1 exceeds the number of menu layers

One must then close this window before you can do anything else on the page -
and thus it is impossible to choose any of the selections on that page.




Reproducible: Always

Steps to Reproduce:
1. Go to web page
2. put cursor over menu options (no need to click)
3. get error message on screen.

Actual Results:  
The Error message appears.

Expected Results:  
It should have worked :-)

Comment 1

[Hector Fabio Meza]

I think this is related to bad javascript coding in pcworld.co.uk

After loading the page, I found the error layersAll has no properties in the
javascript console, with the sourcefile layerobj.js

I checked that file and found the following code (which I think is the problem):

        if(UA.NS) layersAll = document.layers; // NS4 only collection
	if(UA.IE) layersAll = document.all; // IE4/5 only collection
	
	// resolve ref to layer for passed ID
	var myLayer = eval("layersAll."+resolveNest(args[0],args[1]));

From what I understand, this page is ie/ns4 only, they use 2 proprietary
features (document.all and document.layers) which are not supported by Mozilla,
when they should use the getElementById functions.

Comment 2

[Aris Stathakis]

Should Mozilla perhaps just ignore these unsupported commands instead of showing
the error?   I don't want to get into a jihad about IE compatibility here - but
in the real world this is what people are going to come across :-(

Comment 3

[Olivier Cahagne]

*** This bug has been marked as a duplicate of 88596 ***

Comment 4

[Aris Stathakis]

Sorry about the duplication.

Why didn't bugzilla come up with that duplicate when I searched for javascript
error text?

Comment 5

[Phil Schwartau]

Verified Duplicate.

Aris: thank you for this report. You have been cc'ed on bug 88596
so you can follow progress on this issue -