From: wurblzap%gmail.com <>
Date: Fri, 31 Mar 2006 06:23:06 +0000 (+0000)
Subject: Bug 306695: Boolean charts forgets "0" values.
X-Git-Tag: bugzilla-2.22~14
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=90a0410767f7c4a10d05b5b6ba44b2ad638e6eaa;p=thirdparty%2Fbugzilla.git

Bug 306695: Boolean charts forgets "0" values.
Patch by Marc Schumann <wurblzap@gmail.com>,
r=LpSolit, bugreport, a=justdave
---

diff --git a/Bugzilla/CGI.pm b/Bugzilla/CGI.pm
index b722aee3d3..bc2d39b998 100644
--- a/Bugzilla/CGI.pm
+++ b/Bugzilla/CGI.pm
@@ -116,7 +116,7 @@ sub canonicalise_query {
         my $esc_key = url_quote($key);
 
         foreach my $value ($self->param($key)) {
-            if ($value) {
+            if (defined($value)) {
                 my $esc_value = url_quote($value);
 
                 push(@parameters, "$esc_key=$esc_value");
diff --git a/query.cgi b/query.cgi
index 9576966e1e..efd407ebcc 100755
--- a/query.cgi
+++ b/query.cgi
@@ -376,9 +376,13 @@ for (my $chart = 0; $cgi->param("field$chart-0-0"); $chart++) {
     for (my $row = 0; $cgi->param("field$chart-$row-0"); $row++) {
         my @cols;
         for (my $col = 0; $cgi->param("field$chart-$row-$col"); $col++) {
+            my $value = $cgi->param("value$chart-$row-$col");
+            if (!defined($value)) {
+                $value = '';
+            }
             push(@cols, { field => $cgi->param("field$chart-$row-$col"),
                           type => $cgi->param("type$chart-$row-$col") || 'noop',
-                          value => $cgi->param("value$chart-$row-$col") || '' });
+                          value => $value });
         }
         push(@rows, \@cols);
     }