diff options
| author |   gerv%gerv.net <> | 2002-11-22 08:07:19 +0000 |
|---|---|---|
| committer | gerv%gerv.net <> | 2002-11-22 08:07:19 +0000 |
| commit | 0197f0738a4e649338a6ac647bc924d34da9ad11 (patch) | |
| tree | a5d40edde835736ed0822f7934c918339846afed | |
| parent | Bug 181286 - Invalid html in banner.html.tmpl (diff) | |
| download | bugzilla-0197f0738a4e649338a6ac647bc924d34da9ad11.tar.gz bugzilla-0197f0738a4e649338a6ac647bc924d34da9ad11.tar.bz2 bugzilla-0197f0738a4e649338a6ac647bc924d34da9ad11.zip | |
Bug 181182 - Reporting fix pack 2. Fixes bug 179198 (Don't print labels for pie chart wedges when smaller than a certain size), bug 180255 (Tabular report CSV downloads should suggest csv filename), and bug 180967 (csv reports swap rows/columns). Patch by gerv; r=joel, a=justdave.
| -rwxr-xr-x | report.cgi | 10 | ||||
| -rw-r--r-- | template/en/default/reports/report-pie.png.tmpl | 7 | ||||
| -rw-r--r-- | template/en/default/reports/report.html.tmpl | 2 |
3 files changed, 12 insertions, 7 deletions
diff --git a/report.cgi b/report.cgi index c532d523d..f727ee466 100755 --- a/report.cgi +++ b/report.cgi @@ -120,13 +120,13 @@ $columns{''} = "42217354"; # Validate the values in the axis fields or throw an error. !$row_field || ($columns{$row_field} && trick_taint($row_field)) - || ThrowCodeError("report_axis_invalid", { fld=>"x", val=>$row_field }); + || ThrowCodeError("report_axis_invalid", {fld => "x", val => $row_field}); !$col_field || ($columns{$col_field} && trick_taint($col_field)) - || ThrowCodeError("report_axis_invalid", { fld=>"y", val=>$col_field }); + || ThrowCodeError("report_axis_invalid", {fld => "y", val => $col_field}); !$tbl_field || ($columns{$tbl_field} && trick_taint($tbl_field)) - || ThrowCodeError("report_axis_invalid", { fld=>"z", val=>$tbl_field }); + || ThrowCodeError("report_axis_invalid", {fld => "z", val => $tbl_field}); my @axis_fields = ($row_field, $col_field, $tbl_field); @@ -260,6 +260,10 @@ my $format = GetFormat("reports/report", $formatparam, $cgi->param('ctype')); # set debug=1 to always get an HTML content-type, and view the error. $format->{'ctype'} = "text/html" if $::FORM{'debug'}; +my @time = localtime(time()); +my $date = sprintf "%04d-%02d-%02d", 1900+$time[5],$time[4]+1,$time[3]; +my $filename = "report-$date.$format->{extension}"; +print "Content-Disposition: inline; filename=$filename\n"; print "Content-Type: $format->{'ctype'}\n\n"; # Problems with this CGI are often due to malformed data. Setting debug=1 diff --git a/template/en/default/reports/report-pie.png.tmpl b/template/en/default/reports/report-pie.png.tmpl index f34397e12..76a717779 100644 --- a/template/en/default/reports/report-pie.png.tmpl +++ b/template/en/default/reports/report-pie.png.tmpl @@ -26,9 +26,10 @@ [% FILTER null; USE graph = GD.Graph.pie(width, height); - graph.set(title => col_field_disp, - pie_height => 20, - start_angle => 180); + graph.set(title => col_field_disp, + pie_height => 20, + suppress_angle => 2, + start_angle => 180); graph.plot(data.0).png | stdout(1); END; diff --git a/template/en/default/reports/report.html.tmpl b/template/en/default/reports/report.html.tmpl index 07d926a64..494dd796f 100644 --- a/template/en/default/reports/report.html.tmpl +++ b/template/en/default/reports/report.html.tmpl @@ -119,7 +119,7 @@ [% other_format.description %] [% "</a>" UNLESS other_format.name == format %] | [% END %] - <a href="[% formaturl %]&ctype=csv">CSV</a> + <a href="[% formaturl %]&ctype=csv&format=table">CSV</a> </td> [% IF format != "table" %] |
