Closed Bug 180105 Opened 22 years ago Closed 22 years ago

CSV reports occasionally break

Categories

(Bugzilla :: Reporting/Charting, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 2.18

People

(Reporter: dswegen, Assigned: gerv)

References

()

Details

User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2b) Gecko/20021016 Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2b) Gecko/20021016 When attempting to view a buglist produced by a report as a csv, bugzilla will return the Internal Error page. Reproducible: Always Steps to Reproduce: 1.Go to http://bugzilla.mozilla.org/query.cgi?format=report-table 2.Select Assignee on vertical axis, Products on horizontal axis, and Bugzilla as Product. Generate the report. 3.From the report click on the link for Total for bbaetz@student.usyd.edu.au 4.From the buglist click on CSV at the bottom page. 5.Sit back and enjoy the error page (which BTW isn't rendered in mozilla) Actual Results: Error page Expected Results: It should have returned a CSV list of the buglist on the previous page
The error reported in the middle of that text/plain document full of HTML is: Template::Exception at /opt/gnu/files/perl-5.6.0/lib/5.6.0/CGI/Carp.pm line 301
Status: UNCONFIRMED → NEW
Ever confirmed: true
I filed a bug on bmo yesterday for the mimetypes for cvs/png being wrong; myk needs to merge in the update to localconfig. Theres anotehr bug on errors for on text/html documents being silly So, re this bug. Template::Exception is beinging printed because of a perl bug we end up triggering. Locally, running with the patch to work arround this in TT, I get: file error - list/list.htmlcsv.tmpl: not found 'oops'. Some interaction with ctype/format/etc params?
OK, the problem is that the CSV link just reloads the same page with "&ctype=csv"; it doesn't remove any previous ctypes, like the html one you get when you come from report.cgi. So the ctype ends up as htmlcsv, which is bad. Possible solutions: - Get the ctype checker to take the last one from the query string. This is a bit of a hack, but would make doing this class of thing easier. - Hack buglist.cgi to provide a copy of its URL in a template variable but without the ctype parameter. This is more correct, but the same fix would have to be implemented in all places which had this problem. Gerv
Depends on: 180205
Fixed by bug 180205. Gerv
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Target Milestone: --- → Bugzilla 2.18
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.