Closed Bug 289565 Opened 20 years ago Closed 20 years ago

Clicking on a product when creating new dataset doesn't update Components list

Categories

(Bugzilla :: Reporting/Charting, defect)

defect
Not set
major

Tracking

()

RESOLVED FIXED
Bugzilla 2.20

People

(Reporter: mcsmurf, Assigned: gerv)

References

()

Details

Attachments

(1 file)

To reproduce: 1. Go to URL 2. Click on any Product 3. Note that Components list doesn't get updated This is a JS Problem, since JS Console spits out Error: queryform is not defined Source File: http://landfill.mozilla.org/bugzilla-tip/query.cgi?format=create-series Line: 114 The form is named chartform here.
Attached patch Patch v.1Splinter Review
Thanks for the bug report :-) I think Kiko broke this in bug 123030, when he moved the JS to a separate file and didn't update the chart definition template to include it. It may have been broken already by some JS rewriting too. :-(( Gerv
Attachment #180209 - Flags: review?(kiko)
Comment on attachment 180209 [details] [diff] [review] Patch v.1 Hmmm. I guess that javascript has more callsites than immediately apparent. So it's been broken for almost a year now? r=kiko on the change, at any rate.
Attachment #180209 - Flags: review?(kiko) → review+
Flags: approval?
So if it's been broken for a year it probably needs to be fixed in 2.18, too.
Flags: blocking2.18.2+
Flags: approval?
Flags: approval+
Target Milestone: --- → Bugzilla 2.18
No, it works in 2.18. Maybe we've got our "broken for" estimates wrong. Gerv
Flags: blocking2.18.2+
Fixed. Checking in template/en/default/search/search-create-series.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/search/search-create-series.html.tmpl,v <-- search-create-series.html.tmpl new revision: 1.11; previous revision: 1.10 done Gerv
Severity: normal → major
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Target Milestone: Bugzilla 2.18 → Bugzilla 2.20
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: