X-Git-Url: http://git.freeside.biz/gitweb/?a=blobdiff_plain;f=httemplate%2Fsearch%2Freport_tax.cgi;h=587665740d9808c27a5d960ae264182ad0ad6551;hb=5fc8c5edf574ab024d4646914b6432d458e2ffbd;hp=601763bba29e8f77b5e941c60d8dfaee0efcc28e;hpb=5b385021976c25fd312da16e9430df1fe4e89457;p=freeside.git

diff --git a/httemplate/search/report_tax.cgi b/httemplate/search/report_tax.cgi
index 601763bba..587665740 100755
--- a/httemplate/search/report_tax.cgi
+++ b/httemplate/search/report_tax.cgi
@@ -42,6 +42,7 @@ my $gotcust = "
             )
 ";
 
+my $monthly_exempt_warning = 0;
 my($total, $exempt, $taxable, $owed, $tax) = ( 0, 0, 0, 0, 0 );
 my $out = 'Out of taxable region(s)';
 my %regions;
@@ -76,23 +77,46 @@ foreach my $r (qsearch('cust_main_county', {}, '', $gotcust) ) {
     $regions{$label}->{'exempt'} += $x;
   }
 
+  my($t, $x) = (0, 0);
   foreach my $e ( grep { $r->get($_.'tax') !~ /^Y/i }
                        qw( cust_bill_pkg.setup cust_bill_pkg.recur ) ) {
-    my $t = scalar_sql($r, \@param, 
+    $t += scalar_sql($r, \@param, 
       "SELECT SUM($e) $fromwhere AND $nottax AND ( tax != 'Y' OR tax IS NULL )"
     );
-    $taxable += $t;
-    $regions{$label}->{'taxable'} += $t;
-    $owed += $t * ($r->tax/100);
-    $regions{$label}->{'owed'} += $t * ($r->tax/100);
 
-    my $x = scalar_sql($r, \@param, 
+    $x += scalar_sql($r, \@param, 
       "SELECT SUM($e) $fromwhere AND $nottax AND tax = 'Y'"
     );
-    $exempt += $x;
-    $regions{$label}->{'exempt'} += $x;
   }
 
+  my($sday,$smon,$syear) = (localtime($beginning) )[ 3, 4, 5 ];
+  $monthly_exempt_warning=1 if $sday != 1 && $beginning;
+  $smon++; $syear+=1900;
+
+  my $eending = ( $ending == 4294967295 ) ? time : $ending;
+  my($eday,$emon,$eyear) = (localtime($eending) )[ 3, 4, 5 ];
+  $emon++; $eyear+=1900;
+
+  my $monthly_exemption = scalar_sql($r, [ 'taxnum' ],
+    "SELECT SUM(amount) FROM cust_tax_exempt where taxnum = ? ".
+    "  AND ( year > $syear OR ( year = $syear and month >= $smon ) )".
+    "  AND ( year < $eyear OR ( year = $eyear and month <= $emon ) )"
+  );
+  #warn $r->taxnum(). ": $monthly_exemption\n";
+  if ( $monthly_exemption ) {
+    $t -= $monthly_exemption;
+    $x += $monthly_exemption;
+  }
+
+  $taxable += $t;
+  $regions{$label}->{'taxable'} += $t;
+
+  $exempt += $x;
+  $regions{$label}->{'exempt'} += $x;
+
+  $owed += $t * ($r->tax/100);
+  $regions{$label}->{'owed'} += $t * ($r->tax/100);
+
   if ( defined($regions{$label}->{'rate'})
        && $regions{$label}->{'rate'} != $r->tax.'%' ) {
     $regions{$label}->{'rate'} = 'variable';
@@ -125,7 +149,7 @@ foreach my $r (
   my $named_tax =
     $r->taxname
       ? 'AND itemdesc = '. dbh->quote($r->taxname)
-      : "AND itemdesc IS NULL OR itemdesc = '' OR itemdesc = 'Tax'";
+      : "AND ( itemdesc IS NULL OR itemdesc = '' OR itemdesc = 'Tax' )";
   my $x = scalar_sql($r, \@taxparam,
     "SELECT SUM(cust_bill_pkg.setup+cust_bill_pkg.recur) $taxwhere ".
     "AND pkgnum = 0 $named_tax",
@@ -146,6 +170,7 @@ push @regions, {
   'exempt'    => $exempt,
   'taxable'   => $taxable,
   'rate'      => '',
+  'owed'      => $owed,
   'tax'       => $tax,
 };
 
@@ -214,6 +239,14 @@ sub scalar_sql {
 
 </TABLE>
 
+<% if ( $monthly_exempt_warning ) { %>
+  <BR>
+  Partial-month tax reports (except for current month) may not be correct due
+  to month-granularity tax exemption (usually "texas tax").  For an accurate
+  report, start on the first of a month and end on the last day of a month (or
+  leave blank for to now).
+<% } %>
+
 </BODY>
 </HTML>