fix Excel download of A/R report, from #25994
authorMark Wells <mark@freeside.biz>
Thu, 16 Jul 2015 00:18:04 +0000 (17:18 -0700)
committerMark Wells <mark@freeside.biz>
Thu, 16 Jul 2015 05:17:33 +0000 (22:17 -0700)
httemplate/search/elements/cust_main_dayranges.html
httemplate/search/elements/search.html

index e5b1f47..0765653 100644 (file)
@@ -271,7 +271,7 @@ if (delete($opt{'email_checkboxes'})) {
     qq!<input type="checkbox" name="custnum" value="$custnum">!;
   };
   $act_align = 'l';
-  push @act_blank, '';
+  push @act_blank, {nodownload => 1};
 }
 
 </%init>
index f136832..beb0173 100644 (file)
@@ -222,6 +222,18 @@ if ( !$curuser->access_right('Download report data') ) {
   $type = 'html';
 }
 
+# split/map aligns here before doing anything else
+my %align = (
+  'l' => 'left',
+  'r' => 'right',
+  'c' => 'center',
+  ' ' => '',
+  '.' => '',
+);
+
+$opt{align} = [ map $align{$_}, split(//, $opt{align}) ],
+  unless !$opt{align} || ref($opt{align});
+
 if($type =~ /csv|xls/) {
   my $h = $opt{'header'};
   my @del;
@@ -245,18 +257,6 @@ $opt{disable_download} = 0
 $opt{disable_download} = 1
   if $opt{really_disable_download};
 
-# split/map aligns here, so that agent_virt can add a column
-# (search-html.html will split aligns also if they aren't already split)
-my %align = (
-  'l' => 'left',
-  'r' => 'right',
-  'c' => 'center',
-  ' ' => '',
-  '.' => '',
-);
-$opt{align} = [ map $align{$_}, split(//, $opt{align}) ],
-  unless !$opt{align} || ref($opt{align});
-
 # get our queries ready
 my $query = $opt{query} or die "query required";
 my $count_query = $opt{count_query} or die "count_query required";