Excel currency format, #1313

1 files changed, 36 insertions, 10 deletions

diff --git a/httemplate/search/elements/search-xls.html b/httemplate/search/elements/search-xls.html
index 8323f55de..0b5636c0e 100644
--- a/httemplate/search/elements/search-xls.html
+++ b/httemplate/search/elements/search-xls.html
@@ -37,8 +37,30 @@ my $header_format = $workbook->add_format(
   bg_color => 55, #22,
   bottom   => 3,
 );
+my $default_format = $workbook->add_format(locked => 0);
+
+my %money_format;
+my $money_char = FS::Conf->new->config('money_char') || '$';
+
+my $writer = sub {
+  # Wrapper for $worksheet->write.
+  # Do any massaging of the value/format here.
+  my ($r, $c, $value, $format) = @_;
+  if ( $value =~ /^\Q$money_char\E(\d+\.?\d*)$/ ) {
+    # Currency: strip the symbol, clone the requested format,
+    # and format it for currency
+    $value = $1;
+    if ( !exists($money_format{$format}) ) {
+      $money_format{$format} = $workbook->add_format();
+      $money_format{$format}->copy($format);
+      $money_format{$format}->set_num_format($money_char.'#0.00#');
+    }
+    $format = $money_format{$format};
+  }
+  $worksheet->write($r, $c, $value, $format);
+};
 
-$worksheet->write($r, $c++, $_, $header_format ) foreach @$header;
+$writer->( $r, $c++, $_, $header_format ) foreach @$header;
 
 foreach my $row ( @$rows ) {
   $r++;
@@ -53,27 +75,31 @@ foreach my $row ( @$rows ) {
 
     foreach my $field ( @{$opt{'fields'}} ) {
 
-      my $format = shift @$formats;
-      $format = &{$format}($row) if ref($format) eq 'CODE';
-      $format ||= {};
-      my $xls_format = $workbook->add_format(locked=>0, %$format);
+      my $xls_format = $default_format;
+
+      if ( my $format = shift @$formats ) {
+        $format = &{$format}($row) if ref($format) eq 'CODE';
+        $format ||= {};
+        $xls_format = $workbook->add_format(locked=>0, %$format);
+      }
 
       if ( ref($field) eq 'CODE' ) {
         foreach my $value ( &{$field}($row) ) {
           if ( ref($value) eq 'ARRAY' ) { 
-            $worksheet->write($r, $c++, '(N/A)' ); #unimplemented
+            $writer->($r, $c++, '(N/A)' ); #unimplemented
           } else {
-            $worksheet->write($r, $c++, $value, $xls_format );
+            $writer->($r, $c++, $value, $xls_format );
           }
         }
       } else {
-        $worksheet->write($r, $c++, $row->$field(), $xls_format );
+        $writer->( $r, $c++, $row->$field(), $xls_format );
       }
     }
 
   } else {
-    my $xls_format = $workbook->add_format(locked=>0);
-    $worksheet->write($r, $c++, $_, $xls_format ) foreach @$row;
+    # no need for each row to need a new format
+    #my $xls_format = $workbook->add_format(locked=>0);
+    $writer->( $r, $c++, $_, $default_format ) foreach @$row;
   }
 
 }