180116 - Javascript links do not work in this web page

Status: VERIFIED WONTFIX

(Tech Evangelism Graveyard :: Portuguese, defect)

Description

[Pedro Madeira]

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

The orange links that contain javascript in this site do not work in mozilla

Reproducible: Always

Steps to Reproduce:

Comment 1

[Olivier Cahagne]

From JavaScript console:

Error: missing variable name
Source File: http://www.exit.pt/libraries/lib_utils.js
Line: 27, Column: 6
Source Code:
		var final = '$00';

Error: resv is not defined
Source File: http://www.exit.pt/?lnk=ifv,0&infotype=pacotes&infoid=22859
Line: 974

Not sure this is Tech Evangelism, it should work.

Comment 2

[Olivier Cahagne]

Attachment: Reduced testcase

Comment 3

[Marcus Campbell]

From testcase, Mozilla is acting properly. |resv| is not correctly declared
within the JavaScript function. It has no way of knowing what |resv| is,
according to scope. It needs to be referenced by |document.getElementById|,
|document.forms|, or similar:

var resv = document.getElementById("resv");
var resv = document.forms["resv"];

I'm not certain what the "$00" problem is, but it could be related.

-> Tech Evangelism

Comment 4

[Phil Schwartau]

> I'm not certain what the "$00" problem is, but it could be related.

The problem is the use of "final" as the identifier name:

                       var final = '$00';

See the ECMA-262 Ed.3 spec for the language, Section 7.5.3: "final"
is a reserved word and may not be used as an identifier name.

The spec is visible at http://www.mozilla.org/js/language

Comment 5

[Mathieu Pillard [:mat]]

yup, they shouldnt use final.

Comment 6

[Pedro Madeira]

I changing this to invalid since the problem is from the website's variables
name used.

Comment 7

[Mathieu Pillard [:mat]]

so this is a perfectly valid technology evangelism bug. we just need to let them
know about the problem and the _easy_ fix.
reopening. dont close tech evang bug like this please. is the problem still here
? do we have a valid contact to give them this information ?

Comment 8

[Olivier Cahagne]

[e-mail]
source: http://www.exit.pt/?lnk=aju,2

Comment 9

[Mathieu Pillard [:mat]]

can anyone contact them about the [easy] fix ?

Comment 10

[Pedro Madeira]

client support mail account:

[e-mail]

Comment 11

[Pedro Madeira]

TE so changing to WontFix

Comment 12

[Brant Gurganus]

v

Comment 13

[Kai Lahmann (is there, where MNG is)]

move...