Bug 179264 csv output for the buglist (see url inside) is invalid (too many "s)

r,a=justdave

Bug 179264 csv output for the buglist (see url inside) is invalid (too many "s)
81094348 · bugreport%peshkin.net · 9a0ca71b · 81094348
Commit 81094348 authored Nov 10, 2002 by bugreport%peshkin.net
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 6 deletions

globals.pl globals.pl +3 -6

No files found.
--- a/globals.pl
+++ b/globals.pl
@@ -1586,15 +1586,12 @@ $::template ||= Template->new(
        # characters that need encoding.
        url_quote => \&Bugzilla::Util::url_quote,
        
-        # In CSV, quotes are doubled, and any value containing a quote or a
-        # comma is enclosed in quotes.
+        # In CSV, quotes are doubled, and we enclose the whole value in quotes
        csv => sub
        {
            my ($var) = @_;
-            $var =~ s/"/""/;
-            if ($var =~ /",/) {
-                $var = "\"$var\"";
-            }
+            $var =~ s/"/""/g;
+            $var = "\"$var\"";
            return $var;
        } ,
      } ,