Javascript error with certain characters in array.

RESOLVED FIXED in Bugzilla 2.16



Query/Bug List
19 years ago
6 years ago


(Reporter: Michael Maher, Assigned: Christian Reis)


Bugzilla 2.16





19 years ago
If there are components or programs in the query list that have 
a ' in their name, the query list will not work.

here is the Javascript error for a component name of  " rpm's "

JavaScript Error:,
line 53:

missing ] in index expression. 

cpts['rpm's'] = new Array();

JavaScript Error:,
line 6:

selectProduct is not defined.

Comment 1

18 years ago is the new owner of Bugzilla and Bonsai.  (For details,
see my posting in netscape.public.mozilla.webtools,
news:// .)
Assignee: terry → tara
This bug has not been touched for more than nine months. In most cases, that 
means it has "slipped through the net". Please could the owner take a moment to 
add a comment to the bug with current status, and/or close it.

Thank you :-)

QA Contact: matty
Whiteboard: 2.14


18 years ago
Whiteboard: 2.14 → 2.16
moving to real milestones...
Target Milestone: --- → Bugzilla 2.16

Comment 4

17 years ago
*** Bug 63919 has been marked as a duplicate of this bug. ***
*** Bug 81195 has been marked as a duplicate of this bug. ***
Severity: minor → normal
Priority: P3 → P2
Whiteboard: 2.16

Comment 6

17 years ago
moving, this needs to be escaped.
Assignee: tara → endico
Component: Bugzilla → Query/Bug List
Product: Webtools → Bugzilla
Version: other → 2.10

Comment 7

17 years ago
I'm fixing this together with 96534. It seems nobody had the guts to fix that JS
code :)
Assignee: endico → kiko
Depends on: 96534

Comment 8

17 years ago
Reporter, can you please upgrade to 2.14, or apply the patch included in
96534, and reopen if you see the problem isn't solved? Testing on (Product
MyOwnBadSelf) has shown that single quotes are now properly escaped.

Marking FIXED due to checkin for 96534.
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 9

17 years ago
Sorry for the extra spam; this didn't make it into 2.14 but is included
in the latest CVS version (and remember patch v13 too).
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.