Closed Bug 346280 Opened 18 years ago Closed 14 years ago

Two javascript errors when entering "Change over time" "New Charts"

Categories

(Bugzilla :: Reporting/Charting, defect)

2.23
x86
Windows 2000
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 284650

People

(Reporter: pjdemarco, Unassigned)

References

()

Details

A Runtime Error has occurred.
Do you wish to Debug?

Line: 77
Error: Syntax error

--------------------------------
A Runtime Error has occurred.
Do you wish to Debug?

Line: 440
Error: Object expected


Happens every time in IE 6.0.2800
This looks familiar... Maybe I'm thinking of some other JavaScript bug, but I do recall somebody reporting one with Charts.
Whiteboard: DUPME?
Whiteboard: DUPME?
Target Milestone: --- → Bugzilla 2.22
https://bugzilla.mozilla.org/chart.cgi  WORKS W/O errors
Paul: you say it works on bugzilla.mozilla.org? So exactly which version of Bugzilla does it not work on?

Can you reproduce the JS errors in Firefox, where the JavaScript Console will give you much better information about what the problem is?

Gerv
Gerv,
  I set the URL so that there wouldn't be any confusion.  It's running close to HEAD.
I don't have access to IE at home but I ran firefox and got this:


Error: syntax error
Source File: http://landfill.bugzilla.org/paul/chart.cgi
Line: 116, Column: 27
Source Code:
            "__ChrisTest": , 


Error: catSelected is not defined
Source File: http://landfill.bugzilla.org/paul/chart.cgi
Line: 475


In that database, it seems that someone's managed to create a series (-All-/-All-/__ChrisTest) without it being assigned an ID number. This is leading to a key:value pair definition with no value, and so a JS error which is causing the rest of the script block not to be parsed (which gives the second error).

I don't seem to have permissions to look at the underlying MySQL database, so I can't tell what value is actually in the field.

Paul: did you create that chart.


Gerv
mysql> select series_id from series where name = '__ChrisTest';
+-----------+
| series_id |
+-----------+
|       155 |
+-----------+
1 row in set (0.00 sec)

Probably the problem is that __ChrisTest, if stored in a hash, will be considered as a secret key. This could explain you are having.
> This could explain you are having.

This could explain *the problem* you are having.
That might be it. We've been bitten by this sort of bug before. Is there any way we can work around it? Or do we have to forbid series names starting with _, and munge any incoming ones that do?

Gerv
Target Milestone: Bugzilla 2.22 → Bugzilla 3.0
Assignee: gerv → charting
The Bugzilla 3.0 branch is now locked to security bugs and dataloss fixes only. This bug doesn't fit into one of these two categories and is retargetted to 3.2 as part of a mass-change. To catch bugmails related to this mass-change, use lts081207 in your email client filter.
Target Milestone: Bugzilla 3.0 → Bugzilla 3.2
Bugzilla 3.2 is restricted to security bugs only. Moreover, this bug is either assigned to nobody or got no traction for several months now. Rather than retargetting it at each new release, I'm clearing the target milestone and the bug will be retargetted to some sensible release when someone starts fixing this bug for real (Bugzilla 3.8 more likely).
Target Milestone: Bugzilla 3.2 → ---
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.