113431 - hotelplease.com - Javascript problem

Status: RESOLVED WORKSFORME

Description

[Josep Quereda]

There's a problem browsing this URL:

http://www.hotelplease.com

I think there's a problem running javascript.

Comment 1

[Olivier Cahagne]

Must be a layer issue by seeing the included JavaScript files and 95% sure it
should be moved to Tech Evangelism.

Comment 2

[Phil Schwartau]

Right, it's an Evangelism issue. In the JavaScript Console, we see:

Error: document.all has no properties
Source File: http://www.hotelplease.com/include_js/standard.js
Line: 51


This occurs as follows. From http://www.hotelplease.com/main.html,
here is the browser detection for Netscape: 

<script language="Javascript">

  var n = document.layers ? 1:0;


Then, in http://www.hotelplease.com/include_js/standard.js, we have
many functions like this:


function VOLCAR(div,text)
{
 if (n)
 {
  var lyr = document.layers[div].document
  lyr.open()
  lyr.write(text)
  lyr.close()
 }
 else document.all[div].innerHTML = text  <<<<<<<<<<<<<<<<<<<<<< Line 51
}


Since |n| evaluates to false in Mozilla, (layers are no longer supported),
we fall into the IE codepath, which errors since document.all is IE-only.


Reassigning to Tech Evangelism -

Comment 3

[Tristan Nitot]

Confirming. Setting P4

Comment 4

[Josep Quereda]

This website has been redesigned, and with its actual design it works properly 
with Mozilla 0.9.7
Setting Resolution to WONTFIX.

Comment 5

[z-aillon.c-obsolete2]

Should be marked fixed for tracking purposes.

Comment 6

[z-aillon.c-obsolete2]

Fixed

Comment 7

[z-aillon.c-obsolete2]

And verified fixed with 2002021821 builds

Comment 8

[Pascal Chevrel:pascalc]

New Component. Page only displays blue background, reopening.

Comment 9

[Bob Clary [:bc:]]

en other default owner

Comment 10

[Tom Graham]

This seems to have been fixed again / the site has changed its code making the
bug invalid. Marking WFM : Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US;
rv:1.8a) Gecko/20040416 Firefox/0.8.0+