Bug 11944: Remove bad FIXME in guided_report
authorJonathan Druart <jonathan.druart@biblibre.com>
Thu, 8 May 2014 10:59:07 +0000 (12:59 +0200)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Tue, 13 Jan 2015 16:07:31 +0000 (13:07 -0300)
Bug 11679 introduced a FIXME for this but it is wrong. The strings have
to be encoded before generating the ods file.

However, it is useless to decode column names.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Dobrica Pavlinusic <dpavlin@rot13.org>

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>

reports/guided_reports.pl

index c357c70..9b09ac0 100755 (executable)
@@ -912,14 +912,7 @@ elsif ($phase eq 'Save Compound'){
 # pass $sth, get back an array of names for the column headers
 sub header_cell_values {
     my $sth = shift or return ();
-    my @cols;
-    foreach my $c (@{$sth->{NAME}}) {
-        # TODO in Bug 11944
-        #FIXME apparently DBI still needs a utf8 fix for this?
-        $c = Encode::decode('UTF-8', $c);
-        push @cols, $c;
-    }
-    return @cols;
+    return @{$sth->{NAME}};
 }
 
 # pass $sth, get back a TMPL_LOOP-able set of names for the column headers