[freeside-commits] branch master updated. 79110e4c9f798aa38c4bccec6d5be2a0100a5714

[Mark Wells]

The branch, master has been updated
       via  79110e4c9f798aa38c4bccec6d5be2a0100a5714 (commit)
      from  76bf98af668225a893150347998aea8cc0856c7d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 79110e4c9f798aa38c4bccec6d5be2a0100a5714
Author: Mark Wells <mark at freeside.biz>
Date:   Wed Jul 15 17:18:04 2015 -0700

    fix Excel download of A/R report, from #25994

diff --git a/httemplate/search/elements/cust_main_dayranges.html b/httemplate/search/elements/cust_main_dayranges.html
index e5b1f47..0765653 100644
--- a/httemplate/search/elements/cust_main_dayranges.html
+++ b/httemplate/search/elements/cust_main_dayranges.html
@@ -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>
diff --git a/httemplate/search/elements/search.html b/httemplate/search/elements/search.html
index f136832..beb0173 100644
--- a/httemplate/search/elements/search.html
+++ b/httemplate/search/elements/search.html
@@ -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";

-----------------------------------------------------------------------

Summary of changes:
 .../search/elements/cust_main_dayranges.html       |    2 +-
 httemplate/search/elements/search.html             |   24 ++++++++++----------
 2 files changed, 13 insertions(+), 13 deletions(-)