[freeside-commits] branch master updated. 6244b8c0e65cfdfe4c7d3c856e6fb377e2118d5e

[Christopher Burger]

The branch, master has been updated
       via  6244b8c0e65cfdfe4c7d3c856e6fb377e2118d5e (commit)
      from  9fa1bf13b76c0177a3aa12d70aabe5358a9fa3e5 (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 6244b8c0e65cfdfe4c7d3c856e6fb377e2118d5e
Author: Christopher Burger <burgerc at freeside.biz>
Date:   Wed Jun 28 08:37:52 2017 -0400

    RT# 76312 - improved code by using an array datatype rather than a string that is then later manipulated.

diff --git a/httemplate/search/elements/cust_pay_or_refund.html b/httemplate/search/elements/cust_pay_or_refund.html
index 6bd3db6..187fc83 100755
--- a/httemplate/search/elements/cust_pay_or_refund.html
+++ b/httemplate/search/elements/cust_pay_or_refund.html
@@ -463,17 +463,15 @@ if ( $cgi->param('magic') ) {
   }
 
   #check for customer tags
-  my $tagcount;
-  my $tags;
+  my @tags;
   foreach my $p (keys $cgi->Vars) {
     if ($p =~ /^tagnum(\d+)/ && $1) {
-      $tagcount++;
-      $addl_from .= " LEFT JOIN cust_tag ON (cust_tag.custnum = cust_pay.custnum)" unless $tagcount > 1;
-      $tags .= "$1,";
+      $addl_from .= " LEFT JOIN cust_tag ON (cust_tag.custnum = cust_pay.custnum)" unless @tags;
+      push @tags, $1;
     }
   }
-  if ($tags) {
-    $tags =~ s/,$//;
+  if (@tags) {
+    my $tags = join(',', at tags);
     push @search, "cust_tag.tagnum in ($tags)";
   }
 

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

Summary of changes:
 httemplate/search/elements/cust_pay_or_refund.html |   12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)