[freeside-commits] freeside/httemplate/search report_newtax.cgi, 1.3.4.4, 1.3.4.5

[Ivan,,,]

Update of /home/cvs/cvsroot/freeside/httemplate/search
In directory wavetail.420.am:/tmp/cvs-serv17940/httemplate/search

Modified Files:
      Tag: FREESIDE_2_1_BRANCH
	report_newtax.cgi 
Log Message:
fix new tax report w/FS::tax_rate_location->location_sql, RT#12332

Index: report_newtax.cgi
===================================================================
RCS file: /home/cvs/cvsroot/freeside/httemplate/search/report_newtax.cgi,v
retrieving revision 1.3.4.4
retrieving revision 1.3.4.5
diff -u -w -d -r1.3.4.4 -r1.3.4.5
--- report_newtax.cgi	15 Apr 2011 22:33:13 -0000	1.3.4.4
+++ report_newtax.cgi	15 Apr 2011 22:52:30 -0000	1.3.4.5
@@ -16,7 +16,7 @@
   <TR>
     <TH CLASS="grid" BGCOLOR="#cccccc"></TH>
     <TH CLASS="grid" BGCOLOR="#cccccc"></TH>
-    <TH CLASS="grid" BGCOLOR="#cccccc">Tax collected</TH>
+    <TH CLASS="grid" BGCOLOR="#cccccc">Tax invoiced</TH>
     <TH CLASS="grid" BGCOLOR="#cccccc">&nbsp;&nbsp;&nbsp;&nbsp;</TH>
     <TH CLASS="grid" BGCOLOR="#cccccc"></TH>
     <TH CLASS="grid" BGCOLOR="#cccccc">Tax credited</TH>
@@ -91,12 +91,9 @@
   $where .= ' AND cust_main.agentnum = '. $agent->agentnum;
 }
 
-# my ( $location_sql, @location_param ) = FS::cust_pkg->location_sql;
-# $where .= " AND $location_sql";
-#my @taxparam = ( 'itemdesc', @location_param );
-# now something along the lines of geocode matching ?
-#$where .= FS::cust_pkg->_location_sql_where('cust_tax_location');;
-my @taxparam = ( 'itemdesc', 'tax_rate_location.state', 'tax_rate_location.county', 'tax_rate_location.city', 'cust_bill_pkg_tax_rate_location.locationtaxid' );
+#my @taxparam = ( 'itemdesc', 'tax_rate_location.state', 'tax_rate_location.county', 'tax_rate_location.city', 'cust_bill_pkg_tax_rate_location.locationtaxid' );
+my @taxparams = qw( city county state locationtaxid );
+my @params = ('itemdesc', @taxparams);
 
 my $select = 'DISTINCT itemdesc,locationtaxid,tax_rate_location.state,tax_rate_location.county,tax_rate_location.city';
 
@@ -112,7 +109,7 @@
                       })
               )
 {
-  my @params = map { my $f = $_; $f =~ s/.*\.//; $f } @taxparam;
+  #my @params = map { my $f = $_; $f =~ s/.*\.//; $f } @taxparam;
   my $label = join('~', map { $t->$_ } @params);
   $label = 'Tax'. $label if $label =~ /^~/;
   unless ( exists( $taxes{$label} ) ) {
@@ -122,24 +119,31 @@
     $taxes{$label}->{'url_param'} =
       join(';', map { "$_=". uri_escape($t->$_) } @params);
 
-    my $taxwhere = "FROM cust_bill_pkg $addl_from $where AND payby != 'COMP' ".
-      "AND ". join( ' AND ', map { "( $_ = ? OR ? = '' AND $_ IS NULL)" } @taxparam );
+    my $payby_itemdesc_loc = 
+      "    payby != 'COMP' ".
+      "AND itemdesc = ? OR ? = '' AND itemdesc IS NULL ".
+      "AND ". FS::tax_rate_location->location_sql( map { $_ => $t->$_ }
+                                                       @taxparams
+                                                 );
+
+    my $taxwhere =
+      "FROM cust_bill_pkg $addl_from $where AND $payby_itemdesc_loc";
 
     my $sql = "SELECT SUM(amount) $taxwhere AND cust_bill_pkg.pkgnum = 0";
 
-    my $x = scalar_sql($t, [ map { $_, $_ } @params ], $sql );
+    my $x = scalar_sql($t, [ $itemdesc, $itemdesc ], $sql );
     $tax += $x;
     $taxes{$label}->{'tax'} += $x;
 
-    my $creditfrom = " JOIN cust_credit_bill_pkg USING (billpkgnum,billpkgtaxratelocationnum) ";
-    my $creditwhere = "FROM cust_bill_pkg $addl_from $creditfrom $where ".
-      "AND payby != 'COMP' ".
-      "AND ". join( ' AND ', map { "( $_ = ? OR ? = '' AND $_ IS NULL)" } @taxparam );
+    my $creditfrom =
+      "JOIN cust_credit_bill_pkg USING (billpkgnum,billpkgtaxratelocationnum)";
+    my $creditwhere =
+      "FROM cust_bill_pkg $addl_from $creditfrom $where $payby_itemdesc_loc";
 
     $sql = "SELECT SUM(cust_credit_bill_pkg.amount) ".
            " $creditwhere AND cust_bill_pkg.pkgnum = 0";
 
-    my $y = scalar_sql($t, [ map { $_, $_ } @params ], $sql );
+    my $y = scalar_sql($t, [], $sql );
     $credit += $y;
     $taxes{$label}->{'credit'} += $y;