tax report showing all taxes, #41656
authorMark Wells <mark@freeside.biz>
Sat, 16 Apr 2016 01:21:17 +0000 (18:21 -0700)
committerMark Wells <mark@freeside.biz>
Sat, 16 Apr 2016 01:28:20 +0000 (18:28 -0700)
FS/FS/Mason.pm
FS/FS/Report/Tax.pm [deleted file]
FS/FS/Report/Tax/All.pm [new file with mode: 0644]
FS/FS/Report/Tax/ByName.pm [new file with mode: 0644]
FS/MANIFEST
httemplate/search/report_tax-xls.cgi
httemplate/search/report_tax.cgi
httemplate/search/report_tax.html
httemplate/search/tax_sales.cgi

index 9174ec3..2648ef9 100644 (file)
@@ -150,7 +150,8 @@ if ( -e $addl_handler_use_file ) {
   use FS::Report::Table;
   use FS::Report::Table::Monthly;
   use FS::Report::Table::Daily;
-  use FS::Report::Tax;
+  use FS::Report::Tax::ByName;
+  use FS::Report::Tax::All;
   use FS::TicketSystem;
   use FS::NetworkMonitoringSystem;
   use FS::Tron qw( tron_lint );
diff --git a/FS/FS/Report/Tax.pm b/FS/FS/Report/Tax.pm
deleted file mode 100644 (file)
index f1f6be3..0000000
+++ /dev/null
@@ -1,675 +0,0 @@
-package FS::Report::Tax;
-
-use strict;
-use vars qw($DEBUG);
-use FS::Record qw(dbh qsearch qsearchs group_concat_sql);
-use Date::Format qw( time2str );
-
-use Data::Dumper;
-
-$DEBUG = 0;
-
-=item report_internal OPTIONS
-
-Constructor.  Generates a tax report using the internal tax rate system 
-(L<FS::cust_main_county>).
-
-Required parameters:
-
-- beginning, ending: the date range as Unix timestamps.
-- taxname: the name of the tax (corresponds to C<cust_bill_pkg.itemdesc>).
-- country: the country code.
-
-Optional parameters:
-- agentnum: limit to this agentnum.num.
-- breakdown: hashref of the fields to group by.  Keys can be 'city', 'district',
-  'pkgclass', or 'taxclass'; values should be true.
-- debug: sets the debug level.  1 will warn the data collected for the report;
-  2 will also warn all of the SQL statements.
-
-=cut
-
-sub report_internal {
-  my $class = shift;
-  my %opt = @_;
-
-  $DEBUG ||= $opt{debug};
-
-  my $conf = new FS::Conf;
-
-  my($beginning, $ending) = @opt{'beginning', 'ending'};
-
-  my ($taxname, $country, %breakdown);
-
-  # taxname can contain arbitrary punctuation; escape it properly and 
-  # include $taxname unquoted elsewhere
-  $taxname = dbh->quote($opt{'taxname'});
-
-  if ( $opt{country} =~ /^(\w\w)$/ ) {
-    $country = $1;
-  } else {
-    die "country required";
-  }
-
-  # %breakdown: short name => field identifier
-  # null classnum should remain null, not be converted to zero
-  %breakdown = (
-    'taxclass'  => 'cust_main_county.taxclass',
-    'pkgclass'  => 'COALESCE(part_fee.classnum,part_pkg.classnum)',
-    'city'      => 'cust_main_county.city',
-    'district'  => 'cust_main_county.district',
-    'state'     => 'cust_main_county.state',
-    'county'    => 'cust_main_county.county',
-  );
-  foreach (qw(taxclass pkgclass city district)) {
-    delete $breakdown{$_} unless $opt{breakdown}->{$_};
-  }
-
-  my $join_cust =     '      JOIN cust_bill     USING ( invnum  )
-                        LEFT JOIN cust_main     USING ( custnum ) ';
-
-  my $join_cust_pkg = $join_cust.
-                      ' LEFT JOIN cust_pkg      USING ( pkgnum  )
-                        LEFT JOIN part_pkg      USING ( pkgpart )
-                        LEFT JOIN part_fee      USING ( feepart ) ';
-
-  my $from_join_cust_pkg = " FROM cust_bill_pkg $join_cust_pkg "; 
-
-  # all queries MUST be linked to both cust_bill and cust_main_county
-
-  # Either or both of these can be used to link cust_bill_pkg to 
-  # cust_main_county. This one links a taxed line item (billpkgnum) to a tax rate
-  # (taxnum), and gives the amount of tax charged on that line item under that
-  # rate (as tax_amount).
-  my $pkg_tax = "SELECT SUM(amount) as tax_amount, taxnum, ".
-    "taxable_billpkgnum AS billpkgnum ".
-    "FROM cust_bill_pkg_tax_location JOIN cust_bill_pkg USING (billpkgnum) ".
-    "GROUP BY taxable_billpkgnum, taxnum";
-
-  # This one links a tax-exempted line item (billpkgnum) to a tax rate
-  # (taxnum), and gives the amount of the tax exemption.  EXEMPT_WHERE must 
-  # be replaced with an expression to further limit the tax exemptions
-  # that will be included, or "TRUE" to not limit them.
-  #
-  # Note that tax exemptions with non-null creditbillpkgnum are always
-  # excluded. Those are "negative exemptions" created by crediting a sale 
-  # that had received an exemption.
-  my $pkg_tax_exempt = "SELECT SUM(amount) AS exempt_charged, billpkgnum, taxnum ".
-    "FROM cust_tax_exempt_pkg WHERE
-      ( EXEMPT_WHERE )
-      AND cust_tax_exempt_pkg.creditbillpkgnum IS NULL
-     GROUP BY billpkgnum, taxnum";
-
-  my $where = "WHERE cust_bill._date >= $beginning AND cust_bill._date <= $ending ".
-              "AND COALESCE(cust_main_county.taxname,'Tax') = $taxname ".
-              "AND cust_main_county.country = '$country'";
-  # SELECT/GROUP clauses for first-level queries
-  my $select = "SELECT ";
-  my $group = "GROUP BY ";
-  foreach (qw(pkgclass taxclass state county city district)) {
-    if ( $breakdown{$_} ) {
-      $select .= "$breakdown{$_} AS $_, ";
-      $group  .= "$breakdown{$_}, ";
-    } else {
-      $select .= "NULL AS $_, ";
-    }
-  }
-  $select .= group_concat_sql('DISTINCT(cust_main_county.taxnum)', ',') .
-             ' AS taxnums, ';
-  $group =~ s/, $//;
-
-  # SELECT/GROUP clauses for second-level (totals) queries
-  # breakdown by package class only, if anything
-  my $select_all = "SELECT NULL AS pkgclass, ";
-  my $group_all = "";
-  if ( $breakdown{pkgclass} ) {
-    $select_all = "SELECT $breakdown{pkgclass} AS pkgclass, ";
-    $group_all = "GROUP BY $breakdown{pkgclass}";
-  }
-  $select_all .= group_concat_sql('DISTINCT(cust_main_county.taxnum)', ',') .
-                 ' AS taxnums, ';
-
-  my $agentnum;
-  if ( $opt{agentnum} and $opt{agentnum} =~ /^(\d+)$/ ) {
-    $agentnum = $1;
-    my $agent = qsearchs('agent', { 'agentnum' => $agentnum } );
-    die "agent not found" unless $agent;
-    $where .= " AND cust_main.agentnum = $agentnum";
-  }
-
-  my $nottax = 
-    '(cust_bill_pkg.pkgnum != 0 OR cust_bill_pkg.feepart IS NOT NULL)';
-
-  # one query for each column of the report
-  # plus separate queries for the totals row
-  my (%sql, %all_sql);
-
-  # SALES QUERIES (taxable sales, all types of exempt sales)
-  # -------------
-
-  # general form
-  my $exempt = "$select SUM(exempt_charged)
-    FROM cust_main_county
-    JOIN ($pkg_tax_exempt) AS pkg_tax_exempt
-    USING (taxnum)
-    JOIN cust_bill_pkg USING (billpkgnum)
-    $join_cust_pkg $where AND $nottax
-    $group";
-
-  my $all_exempt = "$select_all SUM(exempt_charged)
-    FROM cust_main_county
-    JOIN ($pkg_tax_exempt) AS pkg_tax_exempt
-    USING (taxnum)
-    JOIN cust_bill_pkg USING (billpkgnum)
-    $join_cust_pkg $where AND $nottax
-    $group_all";
-
-  # sales to tax-exempt customers
-  $sql{exempt_cust} = $exempt;
-  $sql{exempt_cust} =~ s/EXEMPT_WHERE/exempt_cust = 'Y' OR exempt_cust_taxname = 'Y'/;
-  $all_sql{exempt_cust} = $all_exempt;
-  $all_sql{exempt_cust} =~ s/EXEMPT_WHERE/exempt_cust = 'Y' OR exempt_cust_taxname = 'Y'/;
-
-  # sales of tax-exempt packages
-  $sql{exempt_pkg} = $exempt;
-  $sql{exempt_pkg} =~ s/EXEMPT_WHERE/exempt_setup = 'Y' OR exempt_recur = 'Y'/;
-  $all_sql{exempt_pkg} = $all_exempt;
-  $all_sql{exempt_pkg} =~ s/EXEMPT_WHERE/exempt_setup = 'Y' OR exempt_recur = 'Y'/;
-
-  # monthly per-customer exemptions
-  $sql{exempt_monthly} = $exempt;
-  $sql{exempt_monthly} =~ s/EXEMPT_WHERE/exempt_monthly = 'Y'/;
-  $all_sql{exempt_monthly} = $all_exempt;
-  $all_sql{exempt_monthly} =~ s/EXEMPT_WHERE/exempt_monthly = 'Y'/;
-
-  # credits applied to taxable sales
-  # Note that negative exemptions (from exempt sales being credited) are NOT
-  # counted when calculating the exempt amount. (See above.) Therefore we need
-  # to NOT include any credits against exempt sales in this amount, either.
-  # These two subqueries implement that. They have joins to cust_credit_bill
-  # and cust_bill so that credits can be filtered by application date if
-  # requested.
-
-  # Each row here is the sum of credits applied to a line item.
-  my $sales_credit =
-    "SELECT billpkgnum, SUM(cust_credit_bill_pkg.amount) AS credited
-    FROM cust_credit_bill_pkg
-    JOIN cust_credit_bill USING (creditbillnum)
-    JOIN cust_bill USING (invnum)
-    WHERE cust_bill._date >= $beginning AND cust_bill._date <= $ending
-    GROUP BY billpkgnum
-    ";
-
-  # Each row here is the sum of negative exemptions applied to a combination
-  # of line item and tax definition.
-  my $exempt_credit =
-    "SELECT cust_credit_bill_pkg.billpkgnum, taxnum,
-      0 - SUM(cust_tax_exempt_pkg.amount) AS exempt_credited
-    FROM cust_credit_bill_pkg
-    LEFT JOIN cust_tax_exempt_pkg USING (creditbillpkgnum)
-    JOIN cust_credit_bill USING (creditbillnum)
-    JOIN cust_bill USING (invnum)
-    WHERE cust_bill._date >= $beginning AND cust_bill._date <= $ending
-    GROUP BY cust_credit_bill_pkg.billpkgnum, taxnum
-    ";
-  
-  if ( $opt{credit_date} eq 'cust_credit_bill' ) {
-    $sales_credit =~ s/cust_bill._date/cust_credit_bill._date/g;
-    $exempt_credit =~ s/cust_bill._date/cust_credit_bill._date/g;
-  }
-
-  $sql{sales_credited} = "$select
-    SUM(COALESCE(credited, 0) - COALESCE(exempt_credited, 0))
-    FROM cust_main_county
-    JOIN ($pkg_tax) AS pkg_tax USING (taxnum)
-    JOIN cust_bill_pkg USING (billpkgnum)
-    LEFT JOIN ($sales_credit) AS sales_credit USING (billpkgnum)
-    LEFT JOIN ($exempt_credit) AS exempt_credit USING (billpkgnum, taxnum)
-    $join_cust_pkg $where AND $nottax
-    $group
-    ";
-
-  $all_sql{sales_credited} = "$select_all
-    SUM(COALESCE(credited, 0) - COALESCE(exempt_credited, 0))
-    FROM cust_main_county
-    JOIN ($pkg_tax) AS pkg_tax USING (taxnum)
-    JOIN cust_bill_pkg USING (billpkgnum)
-    LEFT JOIN ($sales_credit) AS sales_credit USING (billpkgnum)
-    LEFT JOIN ($exempt_credit) AS exempt_credit USING (billpkgnum, taxnum)
-    $join_cust_pkg $where AND $nottax
-    $group_all
-    ";
-
-  # also include the exempt-sales credit amount, for the credit report
-  $sql{exempt_credited} = "$select
-    SUM(COALESCE(exempt_credited, 0))
-    FROM cust_main_county
-    LEFT JOIN ($exempt_credit) AS exempt_credit USING (taxnum)
-    JOIN cust_bill_pkg USING (billpkgnum)
-    $join_cust_pkg $where AND $nottax
-    $group
-    ";
-
-  $all_sql{exempt_credited} = "$select_all
-    SUM(COALESCE(exempt_credited, 0))
-    FROM cust_main_county
-    LEFT JOIN ($exempt_credit) AS exempt_credit USING (taxnum)
-    JOIN cust_bill_pkg USING (billpkgnum)
-    $join_cust_pkg $where AND $nottax
-    $group_all
-    ";
-
-  # taxable sales
-  $sql{taxable} = "$select
-    SUM(cust_bill_pkg.setup + cust_bill_pkg.recur
-      - COALESCE(exempt_charged, 0)
-      - COALESCE(credited, 0)
-      + COALESCE(exempt_credited, 0)
-    )
-    FROM cust_main_county
-    JOIN ($pkg_tax) AS pkg_tax USING (taxnum)
-    JOIN cust_bill_pkg USING (billpkgnum)
-    LEFT JOIN ($pkg_tax_exempt) AS pkg_tax_exempt USING (billpkgnum, taxnum)
-    LEFT JOIN ($sales_credit) AS sales_credit USING (billpkgnum)
-    LEFT JOIN ($exempt_credit) AS exempt_credit USING (billpkgnum, taxnum)
-    $join_cust_pkg $where AND $nottax 
-    $group";
-
-  $all_sql{taxable} = "$select_all
-    SUM(cust_bill_pkg.setup + cust_bill_pkg.recur
-      - COALESCE(exempt_charged, 0)
-      - COALESCE(credited, 0)
-      + COALESCE(exempt_credited, 0)
-    )
-    FROM cust_main_county
-    JOIN ($pkg_tax) AS pkg_tax USING (taxnum)
-    JOIN cust_bill_pkg USING (billpkgnum)
-    LEFT JOIN ($pkg_tax_exempt) AS pkg_tax_exempt USING (billpkgnum, taxnum)
-    LEFT JOIN ($sales_credit) AS sales_credit USING (billpkgnum)
-    LEFT JOIN ($exempt_credit) AS exempt_credit USING (billpkgnum, taxnum)
-    $join_cust_pkg $where AND $nottax 
-    $group_all";
-
-  $sql{taxable} =~ s/EXEMPT_WHERE/TRUE/; # unrestricted
-  $all_sql{taxable} =~ s/EXEMPT_WHERE/TRUE/;
-
-  # estimated tax (taxable * rate)
-  $sql{estimated} = "$select
-    SUM(cust_main_county.tax / 100 * 
-      (cust_bill_pkg.setup + cust_bill_pkg.recur
-      - COALESCE(exempt_charged, 0)
-      - COALESCE(credited, 0)
-      + COALESCE(exempt_credited, 0)
-      )
-    )
-    FROM cust_main_county
-    JOIN ($pkg_tax) AS pkg_tax USING (taxnum)
-    JOIN cust_bill_pkg USING (billpkgnum)
-    LEFT JOIN ($pkg_tax_exempt) AS pkg_tax_exempt USING (billpkgnum, taxnum)
-    LEFT JOIN ($sales_credit) AS sales_credit USING (billpkgnum)
-    LEFT JOIN ($exempt_credit) AS exempt_credit USING (billpkgnum, taxnum)
-    $join_cust_pkg $where AND $nottax 
-    $group";
-
-  $all_sql{estimated} = "$select_all
-    SUM(cust_main_county.tax / 100 * 
-      (cust_bill_pkg.setup + cust_bill_pkg.recur
-      - COALESCE(exempt_charged, 0)
-      - COALESCE(credited, 0)
-      + COALESCE(exempt_credited, 0)
-      )
-    )
-    FROM cust_main_county
-    JOIN ($pkg_tax) AS pkg_tax USING (taxnum)
-    JOIN cust_bill_pkg USING (billpkgnum)
-    LEFT JOIN ($pkg_tax_exempt) AS pkg_tax_exempt USING (billpkgnum, taxnum)
-    LEFT JOIN ($sales_credit) AS sales_credit USING (billpkgnum)
-    LEFT JOIN ($exempt_credit) AS exempt_credit USING (billpkgnum, taxnum)
-    $join_cust_pkg $where AND $nottax 
-    $group_all";
-
-  $sql{estimated} =~ s/EXEMPT_WHERE/TRUE/; # unrestricted
-  $all_sql{estimated} =~ s/EXEMPT_WHERE/TRUE/;
-
-  # there isn't one for 'sales', because we calculate sales by adding up 
-  # the taxable and exempt columns.
-  
-  # TAX QUERIES (billed tax, credited tax, collected tax)
-  # -----------
-
-  # sum of billed tax:
-  # join cust_bill_pkg to cust_main_county via cust_bill_pkg_tax_location
-  my $taxfrom = " FROM cust_bill_pkg 
-                  $join_cust 
-                  LEFT JOIN cust_bill_pkg_tax_location USING ( billpkgnum )
-                  LEFT JOIN cust_main_county USING ( taxnum )";
-
-  if ( $breakdown{pkgclass} ) {
-    # If we're not grouping by package class, this is unnecessary, and
-    # probably really expensive.
-    # Remember that fees also have package classes.
-    $taxfrom .= "
-                  LEFT JOIN cust_bill_pkg AS taxable
-                    ON (cust_bill_pkg_tax_location.taxable_billpkgnum = taxable.billpkgnum)
-                  LEFT JOIN cust_pkg ON (taxable.pkgnum = cust_pkg.pkgnum)
-                  LEFT JOIN part_pkg USING (pkgpart)
-                  LEFT JOIN part_fee ON (taxable.feepart = part_fee.feepart) ";
-  }
-
-  my $istax = "cust_bill_pkg.pkgnum = 0 and cust_bill_pkg.feepart is null";
-
-  $sql{tax} = "$select COALESCE(SUM(cust_bill_pkg_tax_location.amount),0)
-               $taxfrom
-               $where AND $istax
-               $group";
-
-  $all_sql{tax} = "$select_all COALESCE(SUM(cust_bill_pkg_tax_location.amount),0)
-               $taxfrom
-               $where AND $istax
-               $group_all";
-
-  # sum of credits applied against billed tax
-  # ($creditfrom includes join of taxable item to part_pkg/part_fee if 
-  # with_pkgclass is on)
-  my $creditfrom = $taxfrom .
-    ' JOIN cust_credit_bill_pkg USING (billpkgtaxlocationnum)' .
-    ' JOIN cust_credit_bill     USING (creditbillnum)';
-  my $creditwhere = $where . 
-    ' AND billpkgtaxratelocationnum IS NULL';
-
-  # if the credit_date option is set to application date, change
-  # $creditwhere accordingly
-  if ( $opt{credit_date} eq 'cust_credit_bill' ) {
-    $creditwhere     =~ s/cust_bill._date/cust_credit_bill._date/g;
-  }
-
-  $sql{tax_credited} = "$select COALESCE(SUM(cust_credit_bill_pkg.amount),0)
-                  $creditfrom
-                  $creditwhere AND $istax
-                  $group";
-
-  $all_sql{tax_credited} = "$select_all COALESCE(SUM(cust_credit_bill_pkg.amount),0)
-                  $creditfrom
-                  $creditwhere AND $istax
-                  $group_all";
-
-  # sum of tax paid
-  # this suffers from the same ambiguity as anything else that applies 
-  # received payments to specific packages, but in reality the discrepancy
-  # should be minimal since people either pay their bill or don't.
-  # the join is on billpkgtaxlocationnum to avoid cross-producting.
-  my $paidfrom = $taxfrom .
-    ' JOIN cust_bill_pay_pkg'.
-    ' ON (cust_bill_pay_pkg.billpkgtaxlocationnum ='.
-    ' cust_bill_pkg_tax_location.billpkgtaxlocationnum)';
-
-  $sql{tax_paid} = "$select COALESCE(SUM(cust_bill_pay_pkg.amount),0)
-                    $paidfrom
-                    $where AND $istax
-                    $group";
-
-  $all_sql{tax_paid} = "$select_all COALESCE(SUM(cust_bill_pay_pkg.amount),0)
-                    $paidfrom
-                    $where AND $istax
-                    $group_all";
-
-  my %data;
-  my %total;
-  # note that we use keys(%sql) here and keys(%all_sql) later. nothing
-  # obligates us to use the same set of variables for the total query 
-  # as for the individual category queries
-  foreach my $k (keys(%sql)) {
-    my $stmt = $sql{$k};
-    warn "\n".uc($k).":\n".$stmt."\n" if $DEBUG > 1;
-    my $sth = dbh->prepare($stmt);
-    # eight columns: pkgclass, taxclass, state, county, city, district
-    # taxnums (comma separated), value
-    $sth->execute 
-      or die "failed to execute $k query: ".$sth->errstr;
-    while ( my $row = $sth->fetchrow_arrayref ) {
-      my $bin = $data
-                {$row->[0]} # pkgclass
-                {$row->[1]  # taxclass
-                  || ($breakdown{taxclass} ? 'Unclassified' : '')}
-                {$row->[2]} # state
-                {$row->[3] ? $row->[3] . ' County' : ''} # county
-                {$row->[4]} # city
-                {$row->[5]} # district
-              ||= [];
-      push @$bin, [ $k, $row->[6], $row->[7] ];
-    }
-  }
-  warn "DATA:\n".Dumper(\%data) if $DEBUG;
-
-  foreach my $k (keys %all_sql) {
-    warn "\nTOTAL ".uc($k).":\n".$all_sql{$k}."\n" if $DEBUG;
-    my $sth = dbh->prepare($all_sql{$k});
-    # three columns: pkgclass, taxnums (comma separated), value
-    $sth->execute 
-      or die "failed to execute $k totals query: ".$sth->errstr;
-    while ( my $row = $sth->fetchrow_arrayref ) {
-      my $bin = $total{$row->[0]} ||= [];
-      push @$bin, [ $k, $row->[1], $row->[2] ];
-    }
-  }
-  warn "TOTALS:\n".Dumper(\%total) if $DEBUG > 1;
-
-  # $data{$pkgclass}{$taxclass}{$state}{$county}{$city}{$district} = [
-  #   [ 'taxable',     taxnums, amount ],
-  #   [ 'exempt_cust', taxnums, amount ],
-  #   ...
-  # ]
-  # non-requested grouping levels simply collapse into key = ''
-
-  # the much-maligned "out of taxable region"...
-  # find sales that are not linked to any tax with this name
-  # but are still inside the date range/agent criteria.
-  #
-  # This doesn't use $select_all/$group_all because we want a single number,
-  # not a breakdown by pkgclass. Unless someone needs that eventually, 
-  # in which case we'll turn it into an %all_sql query.
-  
-  my $outside_where =
-    "WHERE cust_bill._date >= $beginning AND cust_bill._date <= $ending";
-  if ( $agentnum ) {
-    $outside_where .= " AND cust_main.agentnum = $agentnum";
-  }
-  $outside_where .= "
-    AND NOT EXISTS(
-      SELECT 1 FROM cust_tax_exempt_pkg
-        JOIN cust_main_county USING (taxnum)
-        WHERE cust_tax_exempt_pkg.billpkgnum = cust_bill_pkg.billpkgnum
-          AND COALESCE(cust_main_county.taxname,'Tax') = $taxname
-          AND cust_tax_exempt_pkg.creditbillpkgnum IS NULL
-    )
-    AND NOT EXISTS(
-      SELECT 1 FROM cust_bill_pkg_tax_location
-        JOIN cust_main_county USING (taxnum)
-        WHERE cust_bill_pkg_tax_location.taxable_billpkgnum = cust_bill_pkg.billpkgnum
-          AND COALESCE(cust_main_county.taxname,'Tax') = $taxname
-    )";
-  my $sql_outside = "SELECT SUM(cust_bill_pkg.setup + cust_bill_pkg.recur)
-    FROM cust_bill_pkg
-    $join_cust_pkg
-    $outside_where
-    AND $nottax
-  ";
-  warn "\nOUT_SALES:\n$sql_outside\n" if $DEBUG;
-  my $out_sales = FS::Record->scalar_sql($sql_outside);
-
-  # and out-of-region credit applications, also (excluding those applied
-  # to out-of-region sales _or taxes_)
-  if ( $opt{credit_date} eq 'cust_credit_bill' ) {
-    $outside_where     =~ s/cust_bill._date/cust_credit_bill._date/g;
-  }
-
-  $sql_outside = "SELECT SUM(cust_credit_bill_pkg.amount)
-    FROM cust_credit_bill_pkg
-    JOIN cust_bill_pkg USING (billpkgnum)
-    $join_cust_pkg
-    JOIN cust_credit_bill USING (creditbillnum)
-    $outside_where
-    AND NOT EXISTS(
-      SELECT 1 FROM cust_bill_pkg_tax_location
-        JOIN cust_main_county USING (taxnum)
-        WHERE cust_bill_pkg_tax_location.billpkgnum = cust_bill_pkg.billpkgnum
-          AND COALESCE(cust_main_county.taxname,'Tax') = $taxname
-    )
-  ";
-  warn "\nOUT_CREDIT:\n$sql_outside\n" if $DEBUG;
-  my $out_credit = FS::Record->scalar_sql($sql_outside);
-
-  my %taxrates;
-  foreach my $tax (
-    qsearch('cust_main_county', {
-              country => $country,
-              tax => { op => '>', value => 0 }
-            }) )
-    {
-    $taxrates{$tax->taxnum} = $tax->tax;
-  }
-
-  # return the data
-  bless {
-    'opt'         => \%opt,
-    'data'        => \%data,
-    'total'       => \%total,
-    'taxrates'    => \%taxrates,
-    'out_sales'   => $out_sales,
-    'out_credit'  => $out_credit,
-  }, $class;
-}
-
-sub opt {
-  my $self = shift;
-  $self->{opt};
-}
-
-sub data {
-  my $self = shift;
-  $self->{data};
-}
-
-# sub fetchall_array...
-
-sub table {
-  my $self = shift;
-  my @columns = (qw(pkgclass taxclass state county city district));
-  # taxnums, field headings, and amounts
-  my @rows;
-  my %row_template;
-
-  # de-treeify this thing
-  my $descend;
-  $descend = sub {
-    my ($tree, $level) = @_;
-    if ( ref($tree) eq 'HASH' ) {
-      foreach my $k ( sort {
-           -1*($b eq '')    # sort '' to the end
-          or  ($a eq '')    # sort '' to the end
-          or  ($a <=> $b)   # sort numbers as numbers
-          or  ($a cmp $b)   # sort alphabetics as alphabetics
-        } keys %$tree )
-      {
-        $row_template{ $columns[$level] } = $k;
-        &{ $descend }($tree->{$k}, $level + 1);
-        if ( $level == 0 ) {
-          # then insert the total row for the pkgclass
-          $row_template{'total'} = 1; # flag it as a total
-          &{ $descend }($self->{total}->{$k}, 1);
-          $row_template{'total'} = 0;
-        }
-      }
-    } elsif ( ref($tree) eq 'ARRAY' ) {
-      # then we've reached the bottom; elements of this array are arrayrefs
-      # of [ field, taxnums, amount ].
-      # start with the inherited location-element fields
-      my %this_row = %row_template;
-      my %taxnums;
-      foreach my $x (@$tree) {
-        # accumulate taxnums
-        foreach (split(',', $x->[1])) {
-          $taxnums{$_} = 1;
-        }
-        # and money values
-        $this_row{ $x->[0] } = $x->[2];
-      }
-      # store combined taxnums
-      $this_row{taxnums} = join(',', sort { $a cmp $b } keys %taxnums);
-      # and calculate row totals
-      $this_row{sales} = sprintf('%.2f',
-                          $this_row{taxable} +
-                          $this_row{sales_credited} +
-                          $this_row{exempt_cust} +
-                          $this_row{exempt_pkg} + 
-                          $this_row{exempt_monthly}
-                        );
-      $this_row{credits} = sprintf('%.2f',
-                          $this_row{sales_credited} +
-                          $this_row{exempt_credited} +
-                          $this_row{tax_credited}
-                        );
-      # and give it a label
-      if ( $this_row{total} ) {
-        $this_row{label} = 'Total';
-      } else {
-        $this_row{label} = join(', ', grep $_,
-                            $this_row{taxclass},
-                            $this_row{state},
-                            $this_row{county}, # already has ' County' suffix
-                            $this_row{city},
-                            $this_row{district}
-                           );
-      }
-      # and indicate the tax rate, if any
-      my $rate;
-      foreach (keys %taxnums) {
-        $rate ||= $self->{taxrates}->{$_};
-        if ( $rate != $self->{taxrates}->{$_} ) {
-          $rate = 'variable';
-          last;
-        }
-      }
-      if ( $rate eq 'variable' ) {
-        $this_row{rate} = 'variable';
-      } elsif ( $rate > 0 ) {
-        $this_row{rate} = sprintf('%.2f', $rate);
-      }
-      push @rows, \%this_row;
-    }
-  };
-
-  &{ $descend }($self->{data}, 0);
-
-  warn "TABLE:\n".Dumper(\@rows) if $self->{opt}->{debug};
-  return @rows;
-}
-
-sub taxrates {
-  my $self = shift;
-  $self->{taxrates}
-}
-
-sub title {
-  my $self = shift;
-  my $string = '';
-  if ( $self->{opt}->{agentnum} ) {
-    my $agent = qsearchs('agent', { agentnum => $self->{opt}->{agentnum} });
-    $string .= $agent->agent . ' ';
-  }
-  $string .= 'Tax Report: '; # XXX localization
-  if ( $self->{opt}->{beginning} ) {
-    $string .= time2str('%h %o %Y ', $self->{opt}->{beginning});
-  }
-  $string .= 'through ';
-  if ( $self->{opt}->{ending} and $self->{opt}->{ending} < 4294967295 ) {
-    $string .= time2str('%h %o %Y', $self->{opt}->{ending});
-  } else {
-    $string .= 'now';
-  }
-  $string .= ' - ' . $self->{opt}->{taxname};
-  return $string;
-}
-
-1;
diff --git a/FS/FS/Report/Tax/All.pm b/FS/FS/Report/Tax/All.pm
new file mode 100644 (file)
index 0000000..26dbf5f
--- /dev/null
@@ -0,0 +1,110 @@
+package FS::Report::Tax::All;
+
+use strict;
+use vars qw($DEBUG);
+use FS::Record qw(dbh qsearch qsearchs group_concat_sql);
+use FS::Report::Tax::ByName;
+use Date::Format qw( time2str );
+
+use Data::Dumper;
+
+$DEBUG = 0;
+
+=item report OPTIONS
+
+Constructor. Generates a tax report using the internal tax rate system,
+showing all taxes, broken down by tax name and country.
+
+Required parameters:
+- beginning, ending: the date range as Unix timestamps.
+
+Optional parameters:
+- debug: sets the debug level.  1 will warn the data collected for the report;
+2 will also warn all of the SQL statements.
+
+=cut
+
+# because there's not yet a "DBIx::DBSchema::View"...
+
+sub report {
+  my $class = shift;
+  my %opt = @_;
+
+  $DEBUG ||= $opt{debug};
+
+  my($beginning, $ending) = @opt{'beginning', 'ending'};
+
+  # figure out which reports we need to run
+  my @taxname_and_country = qsearch({
+      table     => 'cust_main_county',
+      select    => 'country, taxname',
+      hashref   => {
+        tax => { op => '>', value => '0' }
+      },
+      order_by  => 'GROUP BY country, taxname ORDER BY country, taxname',
+  });
+  my @table;
+  foreach (@taxname_and_country) {
+    my $taxname = $_->taxname || 'Tax';
+    my $country = $_->country;
+    my $report = FS::Report::Tax::ByName->report(
+      %opt,
+      taxname     => $taxname,
+      country     => $country,
+      total_only  => 1,
+    );
+    # will have only one total row (should be only one row at all)
+    my ($total_row) = grep { $_->{total} } $report->table;
+    $total_row->{total} = 0; # but in this context it's a detail row
+    $total_row->{taxname} = $taxname;
+    $total_row->{country} = $country;
+    $total_row->{label} = "$country - $taxname";
+    push @table, $total_row;
+  }
+  my $self = bless {
+    'opt'   => \%opt,
+    'table' => \@table,
+  }, $class;
+
+  $self;
+}
+
+sub opt {
+  my $self = shift;
+  $self->{opt};
+}
+
+sub data {
+  my $self = shift;
+  $self->{data};
+}
+
+# sub fetchall_array...
+
+sub table {
+  my $self = shift;
+  @{ $self->{table} };
+}
+
+sub title {
+  my $self = shift;
+  my $string = '';
+  if ( $self->{opt}->{agentnum} ) {
+    my $agent = qsearchs('agent', { agentnum => $self->{opt}->{agentnum} });
+    $string .= $agent->agent . ' ';
+  }
+  $string .= 'Tax Report: '; # XXX localization
+  if ( $self->{opt}->{beginning} ) {
+    $string .= time2str('%h %o %Y ', $self->{opt}->{beginning});
+  }
+  $string .= 'through ';
+  if ( $self->{opt}->{ending} and $self->{opt}->{ending} < 4294967295 ) {
+    $string .= time2str('%h %o %Y', $self->{opt}->{ending});
+  } else {
+    $string .= 'now';
+  }
+  $string .= ' - all taxes';
+  return $string;
+}
+
+1;
diff --git a/FS/FS/Report/Tax/ByName.pm b/FS/FS/Report/Tax/ByName.pm
new file mode 100644 (file)
index 0000000..88695b9
--- /dev/null
@@ -0,0 +1,678 @@
+package FS::Report::Tax::ByName;
+
+use strict;
+use vars qw($DEBUG);
+use FS::Record qw(dbh qsearch qsearchs group_concat_sql);
+use Date::Format qw( time2str );
+
+use Data::Dumper;
+
+$DEBUG = 0;
+
+=item report OPTIONS
+
+Constructor.  Generates a tax report using the internal tax rate system 
+(L<FS::cust_main_county>), showing all taxes with a specified tax name,
+broken down by state/county. Optionally, the taxes can be broken down further
+by city/district, tax class, or package class.
+
+Required parameters:
+
+- beginning, ending: the date range as Unix timestamps.
+- taxname: the name of the tax (corresponds to C<cust_bill_pkg.itemdesc>).
+- country: the country code.
+
+Optional parameters:
+- agentnum: limit to this agentnum.num.
+- breakdown: hashref of the fields to group by.  Keys can be 'city',
+'district', 'pkgclass', or 'taxclass'; values should be true.
+- total_only: don't run the tax group queries, only the totals queries.
+Returns one row, except in the unlikely event you're using breakdown by
+package class.
+- debug: sets the debug level.  1 will warn the data collected for the report;
+2 will also warn all of the SQL statements.
+
+=cut
+
+sub report {
+  my $class = shift;
+  my %opt = @_;
+
+  $DEBUG ||= $opt{debug};
+
+  my($beginning, $ending) = @opt{'beginning', 'ending'};
+
+  my ($taxname, $country, %breakdown);
+
+  # taxname can contain arbitrary punctuation; escape it properly and 
+  # include $taxname unquoted elsewhere
+  $taxname = dbh->quote($opt{'taxname'});
+
+  if ( $opt{country} =~ /^(\w\w)$/ ) {
+    $country = $1;
+  } else {
+    die "country required";
+  }
+
+  # %breakdown: short name => field identifier
+  # null classnum should remain null, not be converted to zero
+  %breakdown = (
+    'taxclass'  => 'cust_main_county.taxclass',
+    'pkgclass'  => 'COALESCE(part_fee.classnum,part_pkg.classnum)',
+    'city'      => 'cust_main_county.city',
+    'district'  => 'cust_main_county.district',
+    'state'     => 'cust_main_county.state',
+    'county'    => 'cust_main_county.county',
+  );
+  foreach (qw(taxclass pkgclass city district)) {
+    delete $breakdown{$_} unless $opt{breakdown}->{$_};
+  }
+
+  my $join_cust =     '      JOIN cust_bill     USING ( invnum  )
+                        LEFT JOIN cust_main     USING ( custnum ) ';
+
+  my $join_cust_pkg = $join_cust.
+                      ' LEFT JOIN cust_pkg      USING ( pkgnum  )
+                        LEFT JOIN part_pkg      USING ( pkgpart )
+                        LEFT JOIN part_fee      USING ( feepart ) ';
+
+  my $from_join_cust_pkg = " FROM cust_bill_pkg $join_cust_pkg "; 
+
+  # all queries MUST be linked to both cust_bill and cust_main_county
+
+  # Either or both of these can be used to link cust_bill_pkg to 
+  # cust_main_county. This one links a taxed line item (billpkgnum) to a tax rate
+  # (taxnum), and gives the amount of tax charged on that line item under that
+  # rate (as tax_amount).
+  my $pkg_tax = "SELECT SUM(amount) as tax_amount, taxnum, ".
+    "taxable_billpkgnum AS billpkgnum ".
+    "FROM cust_bill_pkg_tax_location JOIN cust_bill_pkg USING (billpkgnum) ".
+    "GROUP BY taxable_billpkgnum, taxnum";
+
+  # This one links a tax-exempted line item (billpkgnum) to a tax rate
+  # (taxnum), and gives the amount of the tax exemption.  EXEMPT_WHERE must 
+  # be replaced with an expression to further limit the tax exemptions
+  # that will be included, or "TRUE" to not limit them.
+  #
+  # Note that tax exemptions with non-null creditbillpkgnum are always
+  # excluded. Those are "negative exemptions" created by crediting a sale 
+  # that had received an exemption.
+  my $pkg_tax_exempt = "SELECT SUM(amount) AS exempt_charged, billpkgnum, taxnum ".
+    "FROM cust_tax_exempt_pkg WHERE
+      ( EXEMPT_WHERE )
+      AND cust_tax_exempt_pkg.creditbillpkgnum IS NULL
+     GROUP BY billpkgnum, taxnum";
+
+  my $where = "WHERE cust_bill._date >= $beginning AND cust_bill._date <= $ending ".
+              "AND COALESCE(cust_main_county.taxname,'Tax') = $taxname ".
+              "AND cust_main_county.country = '$country'";
+  # SELECT/GROUP clauses for first-level queries
+  my $select = "SELECT ";
+  my $group = "GROUP BY ";
+  foreach (qw(pkgclass taxclass state county city district)) {
+    if ( $breakdown{$_} ) {
+      $select .= "$breakdown{$_} AS $_, ";
+      $group  .= "$breakdown{$_}, ";
+    } else {
+      $select .= "NULL AS $_, ";
+    }
+  }
+  $select .= group_concat_sql('DISTINCT(cust_main_county.taxnum)', ',') .
+             ' AS taxnums, ';
+  $group =~ s/, $//;
+
+  # SELECT/GROUP clauses for second-level (totals) queries
+  # breakdown by package class only, if anything
+  my $select_all = "SELECT NULL AS pkgclass, ";
+  my $group_all = "";
+  if ( $breakdown{pkgclass} ) {
+    $select_all = "SELECT $breakdown{pkgclass} AS pkgclass, ";
+    $group_all = "GROUP BY $breakdown{pkgclass}";
+  }
+  $select_all .= group_concat_sql('DISTINCT(cust_main_county.taxnum)', ',') .
+                 ' AS taxnums, ';
+
+  my $agentnum;
+  if ( $opt{agentnum} and $opt{agentnum} =~ /^(\d+)$/ ) {
+    $agentnum = $1;
+    my $agent = qsearchs('agent', { 'agentnum' => $agentnum } );
+    die "agent not found" unless $agent;
+    $where .= " AND cust_main.agentnum = $agentnum";
+  }
+
+  my $nottax = 
+    '(cust_bill_pkg.pkgnum != 0 OR cust_bill_pkg.feepart IS NOT NULL)';
+
+  # one query for each column of the report
+  # plus separate queries for the totals row
+  my (%sql, %all_sql);
+
+  # SALES QUERIES (taxable sales, all types of exempt sales)
+  # -------------
+
+  # general form
+  my $exempt = "$select SUM(exempt_charged)
+    FROM cust_main_county
+    JOIN ($pkg_tax_exempt) AS pkg_tax_exempt
+    USING (taxnum)
+    JOIN cust_bill_pkg USING (billpkgnum)
+    $join_cust_pkg $where AND $nottax
+    $group";
+
+  my $all_exempt = "$select_all SUM(exempt_charged)
+    FROM cust_main_county
+    JOIN ($pkg_tax_exempt) AS pkg_tax_exempt
+    USING (taxnum)
+    JOIN cust_bill_pkg USING (billpkgnum)
+    $join_cust_pkg $where AND $nottax
+    $group_all";
+
+  # sales to tax-exempt customers
+  $sql{exempt_cust} = $exempt;
+  $sql{exempt_cust} =~ s/EXEMPT_WHERE/exempt_cust = 'Y' OR exempt_cust_taxname = 'Y'/;
+  $all_sql{exempt_cust} = $all_exempt;
+  $all_sql{exempt_cust} =~ s/EXEMPT_WHERE/exempt_cust = 'Y' OR exempt_cust_taxname = 'Y'/;
+
+  # sales of tax-exempt packages
+  $sql{exempt_pkg} = $exempt;
+  $sql{exempt_pkg} =~ s/EXEMPT_WHERE/exempt_setup = 'Y' OR exempt_recur = 'Y'/;
+  $all_sql{exempt_pkg} = $all_exempt;
+  $all_sql{exempt_pkg} =~ s/EXEMPT_WHERE/exempt_setup = 'Y' OR exempt_recur = 'Y'/;
+
+  # monthly per-customer exemptions
+  $sql{exempt_monthly} = $exempt;
+  $sql{exempt_monthly} =~ s/EXEMPT_WHERE/exempt_monthly = 'Y'/;
+  $all_sql{exempt_monthly} = $all_exempt;
+  $all_sql{exempt_monthly} =~ s/EXEMPT_WHERE/exempt_monthly = 'Y'/;
+
+  # credits applied to taxable sales
+  # Note that negative exemptions (from exempt sales being credited) are NOT
+  # counted when calculating the exempt amount. (See above.) Therefore we need
+  # to NOT include any credits against exempt sales in this amount, either.
+  # These two subqueries implement that. They have joins to cust_credit_bill
+  # and cust_bill so that credits can be filtered by application date if
+  # requested.
+
+  # Each row here is the sum of credits applied to a line item.
+  my $sales_credit =
+    "SELECT billpkgnum, SUM(cust_credit_bill_pkg.amount) AS credited
+    FROM cust_credit_bill_pkg
+    JOIN cust_credit_bill USING (creditbillnum)
+    JOIN cust_bill USING (invnum)
+    WHERE cust_bill._date >= $beginning AND cust_bill._date <= $ending
+    GROUP BY billpkgnum
+    ";
+
+  # Each row here is the sum of negative exemptions applied to a combination
+  # of line item and tax definition.
+  my $exempt_credit =
+    "SELECT cust_credit_bill_pkg.billpkgnum, taxnum,
+      0 - SUM(cust_tax_exempt_pkg.amount) AS exempt_credited
+    FROM cust_credit_bill_pkg
+    LEFT JOIN cust_tax_exempt_pkg USING (creditbillpkgnum)
+    JOIN cust_credit_bill USING (creditbillnum)
+    JOIN cust_bill USING (invnum)
+    WHERE cust_bill._date >= $beginning AND cust_bill._date <= $ending
+    GROUP BY cust_credit_bill_pkg.billpkgnum, taxnum
+    ";
+  
+  if ( $opt{credit_date} eq 'cust_credit_bill' ) {
+    $sales_credit =~ s/cust_bill._date/cust_credit_bill._date/g;
+    $exempt_credit =~ s/cust_bill._date/cust_credit_bill._date/g;
+  }
+
+  $sql{sales_credited} = "$select
+    SUM(COALESCE(credited, 0) - COALESCE(exempt_credited, 0))
+    FROM cust_main_county
+    JOIN ($pkg_tax) AS pkg_tax USING (taxnum)
+    JOIN cust_bill_pkg USING (billpkgnum)
+    LEFT JOIN ($sales_credit) AS sales_credit USING (billpkgnum)
+    LEFT JOIN ($exempt_credit) AS exempt_credit USING (billpkgnum, taxnum)
+    $join_cust_pkg $where AND $nottax
+    $group
+    ";
+
+  $all_sql{sales_credited} = "$select_all
+    SUM(COALESCE(credited, 0) - COALESCE(exempt_credited, 0))
+    FROM cust_main_county
+    JOIN ($pkg_tax) AS pkg_tax USING (taxnum)
+    JOIN cust_bill_pkg USING (billpkgnum)
+    LEFT JOIN ($sales_credit) AS sales_credit USING (billpkgnum)
+    LEFT JOIN ($exempt_credit) AS exempt_credit USING (billpkgnum, taxnum)
+    $join_cust_pkg $where AND $nottax
+    $group_all
+    ";
+
+  # also include the exempt-sales credit amount, for the credit report
+  $sql{exempt_credited} = "$select
+    SUM(COALESCE(exempt_credited, 0))
+    FROM cust_main_county
+    LEFT JOIN ($exempt_credit) AS exempt_credit USING (taxnum)
+    JOIN cust_bill_pkg USING (billpkgnum)
+    $join_cust_pkg $where AND $nottax
+    $group
+    ";
+
+  $all_sql{exempt_credited} = "$select_all
+    SUM(COALESCE(exempt_credited, 0))
+    FROM cust_main_county
+    LEFT JOIN ($exempt_credit) AS exempt_credit USING (taxnum)
+    JOIN cust_bill_pkg USING (billpkgnum)
+    $join_cust_pkg $where AND $nottax
+    $group_all
+    ";
+
+  # taxable sales
+  $sql{taxable} = "$select
+    SUM(cust_bill_pkg.setup + cust_bill_pkg.recur
+      - COALESCE(exempt_charged, 0)
+      - COALESCE(credited, 0)
+      + COALESCE(exempt_credited, 0)
+    )
+    FROM cust_main_county
+    JOIN ($pkg_tax) AS pkg_tax USING (taxnum)
+    JOIN cust_bill_pkg USING (billpkgnum)
+    LEFT JOIN ($pkg_tax_exempt) AS pkg_tax_exempt USING (billpkgnum, taxnum)
+    LEFT JOIN ($sales_credit) AS sales_credit USING (billpkgnum)
+    LEFT JOIN ($exempt_credit) AS exempt_credit USING (billpkgnum, taxnum)
+    $join_cust_pkg $where AND $nottax 
+    $group";
+
+  $all_sql{taxable} = "$select_all
+    SUM(cust_bill_pkg.setup + cust_bill_pkg.recur
+      - COALESCE(exempt_charged, 0)
+      - COALESCE(credited, 0)
+      + COALESCE(exempt_credited, 0)
+    )
+    FROM cust_main_county
+    JOIN ($pkg_tax) AS pkg_tax USING (taxnum)
+    JOIN cust_bill_pkg USING (billpkgnum)
+    LEFT JOIN ($pkg_tax_exempt) AS pkg_tax_exempt USING (billpkgnum, taxnum)
+    LEFT JOIN ($sales_credit) AS sales_credit USING (billpkgnum)
+    LEFT JOIN ($exempt_credit) AS exempt_credit USING (billpkgnum, taxnum)
+    $join_cust_pkg $where AND $nottax 
+    $group_all";
+
+  $sql{taxable} =~ s/EXEMPT_WHERE/TRUE/; # unrestricted
+  $all_sql{taxable} =~ s/EXEMPT_WHERE/TRUE/;
+
+  # estimated tax (taxable * rate)
+  $sql{estimated} = "$select
+    SUM(cust_main_county.tax / 100 * 
+      (cust_bill_pkg.setup + cust_bill_pkg.recur
+      - COALESCE(exempt_charged, 0)
+      - COALESCE(credited, 0)
+      + COALESCE(exempt_credited, 0)
+      )
+    )
+    FROM cust_main_county
+    JOIN ($pkg_tax) AS pkg_tax USING (taxnum)
+    JOIN cust_bill_pkg USING (billpkgnum)
+    LEFT JOIN ($pkg_tax_exempt) AS pkg_tax_exempt USING (billpkgnum, taxnum)
+    LEFT JOIN ($sales_credit) AS sales_credit USING (billpkgnum)
+    LEFT JOIN ($exempt_credit) AS exempt_credit USING (billpkgnum, taxnum)
+    $join_cust_pkg $where AND $nottax 
+    $group";
+
+  $all_sql{estimated} = "$select_all
+    SUM(cust_main_county.tax / 100 * 
+      (cust_bill_pkg.setup + cust_bill_pkg.recur
+      - COALESCE(exempt_charged, 0)
+      - COALESCE(credited, 0)
+      + COALESCE(exempt_credited, 0)
+      )
+    )
+    FROM cust_main_county
+    JOIN ($pkg_tax) AS pkg_tax USING (taxnum)
+    JOIN cust_bill_pkg USING (billpkgnum)
+    LEFT JOIN ($pkg_tax_exempt) AS pkg_tax_exempt USING (billpkgnum, taxnum)
+    LEFT JOIN ($sales_credit) AS sales_credit USING (billpkgnum)
+    LEFT JOIN ($exempt_credit) AS exempt_credit USING (billpkgnum, taxnum)
+    $join_cust_pkg $where AND $nottax 
+    $group_all";
+
+  $sql{estimated} =~ s/EXEMPT_WHERE/TRUE/; # unrestricted
+  $all_sql{estimated} =~ s/EXEMPT_WHERE/TRUE/;
+
+  # there isn't one for 'sales', because we calculate sales by adding up 
+  # the taxable and exempt columns.
+  
+  # TAX QUERIES (billed tax, credited tax, collected tax)
+  # -----------
+
+  # sum of billed tax:
+  # join cust_bill_pkg to cust_main_county via cust_bill_pkg_tax_location
+  my $taxfrom = " FROM cust_bill_pkg 
+                  $join_cust 
+                  LEFT JOIN cust_bill_pkg_tax_location USING ( billpkgnum )
+                  LEFT JOIN cust_main_county USING ( taxnum )";
+
+  if ( $breakdown{pkgclass} ) {
+    # If we're not grouping by package class, this is unnecessary, and
+    # probably really expensive.
+    # Remember that fees also have package classes.
+    $taxfrom .= "
+                  LEFT JOIN cust_bill_pkg AS taxable
+                    ON (cust_bill_pkg_tax_location.taxable_billpkgnum = taxable.billpkgnum)
+                  LEFT JOIN cust_pkg ON (taxable.pkgnum = cust_pkg.pkgnum)
+                  LEFT JOIN part_pkg USING (pkgpart)
+                  LEFT JOIN part_fee ON (taxable.feepart = part_fee.feepart) ";
+  }
+
+  my $istax = "cust_bill_pkg.pkgnum = 0 and cust_bill_pkg.feepart is null";
+
+  $sql{tax} = "$select COALESCE(SUM(cust_bill_pkg_tax_location.amount),0)
+               $taxfrom
+               $where AND $istax
+               $group";
+
+  $all_sql{tax} = "$select_all COALESCE(SUM(cust_bill_pkg_tax_location.amount),0)
+               $taxfrom
+               $where AND $istax
+               $group_all";
+
+  # sum of credits applied against billed tax
+  # ($creditfrom includes join of taxable item to part_pkg/part_fee if 
+  # with_pkgclass is on)
+  my $creditfrom = $taxfrom .
+    ' JOIN cust_credit_bill_pkg USING (billpkgtaxlocationnum)' .
+    ' JOIN cust_credit_bill     USING (creditbillnum)';
+  my $creditwhere = $where . 
+    ' AND billpkgtaxratelocationnum IS NULL';
+
+  # if the credit_date option is set to application date, change
+  # $creditwhere accordingly
+  if ( $opt{credit_date} eq 'cust_credit_bill' ) {
+    $creditwhere     =~ s/cust_bill._date/cust_credit_bill._date/g;
+  }
+
+  $sql{tax_credited} = "$select COALESCE(SUM(cust_credit_bill_pkg.amount),0)
+                  $creditfrom
+                  $creditwhere AND $istax
+                  $group";
+
+  $all_sql{tax_credited} = "$select_all COALESCE(SUM(cust_credit_bill_pkg.amount),0)
+                  $creditfrom
+                  $creditwhere AND $istax
+                  $group_all";
+
+  # sum of tax paid
+  # this suffers from the same ambiguity as anything else that applies 
+  # received payments to specific packages, but in reality the discrepancy
+  # should be minimal since people either pay their bill or don't.
+  # the join is on billpkgtaxlocationnum to avoid cross-producting.
+  my $paidfrom = $taxfrom .
+    ' JOIN cust_bill_pay_pkg'.
+    ' ON (cust_bill_pay_pkg.billpkgtaxlocationnum ='.
+    ' cust_bill_pkg_tax_location.billpkgtaxlocationnum)';
+
+  $sql{tax_paid} = "$select COALESCE(SUM(cust_bill_pay_pkg.amount),0)
+                    $paidfrom
+                    $where AND $istax
+                    $group";
+
+  $all_sql{tax_paid} = "$select_all COALESCE(SUM(cust_bill_pay_pkg.amount),0)
+                    $paidfrom
+                    $where AND $istax
+                    $group_all";
+
+  my %data;
+  my %total;
+  # note that we use keys(%sql) here and keys(%all_sql) later. nothing
+  # obligates us to use the same set of variables for the total query 
+  # as for the individual category queries
+  foreach my $k (keys(%sql)) {
+    my $stmt = $sql{$k};
+    warn "\n".uc($k).":\n".$stmt."\n" if $DEBUG > 1;
+    my $sth = dbh->prepare($stmt);
+    # eight columns: pkgclass, taxclass, state, county, city, district
+    # taxnums (comma separated), value
+    $sth->execute 
+      or die "failed to execute $k query: ".$sth->errstr;
+    while ( my $row = $sth->fetchrow_arrayref ) {
+      my $bin = $data
+                {$row->[0]} # pkgclass
+                {$row->[1]  # taxclass
+                  || ($breakdown{taxclass} ? 'Unclassified' : '')}
+                {$row->[2]} # state
+                {$row->[3] ? $row->[3] . ' County' : ''} # county
+                {$row->[4]} # city
+                {$row->[5]} # district
+              ||= [];
+      push @$bin, [ $k, $row->[6], $row->[7] ];
+    }
+  }
+  warn "DATA:\n".Dumper(\%data) if $DEBUG;
+
+  foreach my $k (keys %all_sql) {
+    warn "\nTOTAL ".uc($k).":\n".$all_sql{$k}."\n" if $DEBUG;
+    my $sth = dbh->prepare($all_sql{$k});
+    # three columns: pkgclass, taxnums (comma separated), value
+    $sth->execute 
+      or die "failed to execute $k totals query: ".$sth->errstr;
+    while ( my $row = $sth->fetchrow_arrayref ) {
+      my $bin = $total{$row->[0]} ||= [];
+      push @$bin, [ $k, $row->[1], $row->[2] ];
+    }
+  }
+  warn "TOTALS:\n".Dumper(\%total) if $DEBUG > 1;
+
+  # $data{$pkgclass}{$taxclass}{$state}{$county}{$city}{$district} = [
+  #   [ 'taxable',     taxnums, amount ],
+  #   [ 'exempt_cust', taxnums, amount ],
+  #   ...
+  # ]
+  # non-requested grouping levels simply collapse into key = ''
+
+  # the much-maligned "out of taxable region"...
+  # find sales that are not linked to any tax with this name
+  # but are still inside the date range/agent criteria.
+  #
+  # This doesn't use $select_all/$group_all because we want a single number,
+  # not a breakdown by pkgclass. Unless someone needs that eventually, 
+  # in which case we'll turn it into an %all_sql query.
+  
+  my $outside_where =
+    "WHERE cust_bill._date >= $beginning AND cust_bill._date <= $ending";
+  if ( $agentnum ) {
+    $outside_where .= " AND cust_main.agentnum = $agentnum";
+  }
+  $outside_where .= "
+    AND NOT EXISTS(
+      SELECT 1 FROM cust_tax_exempt_pkg
+        JOIN cust_main_county USING (taxnum)
+        WHERE cust_tax_exempt_pkg.billpkgnum = cust_bill_pkg.billpkgnum
+          AND COALESCE(cust_main_county.taxname,'Tax') = $taxname
+          AND cust_tax_exempt_pkg.creditbillpkgnum IS NULL
+    )
+    AND NOT EXISTS(
+      SELECT 1 FROM cust_bill_pkg_tax_location
+        JOIN cust_main_county USING (taxnum)
+        WHERE cust_bill_pkg_tax_location.taxable_billpkgnum = cust_bill_pkg.billpkgnum
+          AND COALESCE(cust_main_county.taxname,'Tax') = $taxname
+    )";
+  my $sql_outside = "SELECT SUM(cust_bill_pkg.setup + cust_bill_pkg.recur)
+    FROM cust_bill_pkg
+    $join_cust_pkg
+    $outside_where
+    AND $nottax
+  ";
+  warn "\nOUT_SALES:\n$sql_outside\n" if $DEBUG;
+  my $out_sales = FS::Record->scalar_sql($sql_outside);
+
+  # and out-of-region credit applications, also (excluding those applied
+  # to out-of-region sales _or taxes_)
+  if ( $opt{credit_date} eq 'cust_credit_bill' ) {
+    $outside_where     =~ s/cust_bill._date/cust_credit_bill._date/g;
+  }
+
+  $sql_outside = "SELECT SUM(cust_credit_bill_pkg.amount)
+    FROM cust_credit_bill_pkg
+    JOIN cust_bill_pkg USING (billpkgnum)
+    $join_cust_pkg
+    JOIN cust_credit_bill USING (creditbillnum)
+    $outside_where
+    AND NOT EXISTS(
+      SELECT 1 FROM cust_bill_pkg_tax_location
+        JOIN cust_main_county USING (taxnum)
+        WHERE cust_bill_pkg_tax_location.billpkgnum = cust_bill_pkg.billpkgnum
+          AND COALESCE(cust_main_county.taxname,'Tax') = $taxname
+    )
+  ";
+  warn "\nOUT_CREDIT:\n$sql_outside\n" if $DEBUG;
+  my $out_credit = FS::Record->scalar_sql($sql_outside);
+
+  my %taxrates;
+  foreach my $tax (
+    qsearch('cust_main_county', {
+              country => $country,
+              tax => { op => '>', value => 0 }
+            }) )
+    {
+    $taxrates{$tax->taxnum} = $tax->tax;
+  }
+
+  # return the data
+  bless {
+    'opt'         => \%opt,
+    'data'        => \%data,
+    'total'       => \%total,
+    'taxrates'    => \%taxrates,
+    'out_sales'   => $out_sales,
+    'out_credit'  => $out_credit,
+  }, $class;
+}
+
+sub opt {
+  my $self = shift;
+  $self->{opt};
+}
+
+sub data {
+  my $self = shift;
+  $self->{data};
+}
+
+# sub fetchall_array...
+
+sub table {
+  my $self = shift;
+  my @columns = (qw(pkgclass taxclass state county city district));
+  # taxnums, field headings, and amounts
+  my @rows;
+  my %row_template;
+
+  # de-treeify this thing
+  my $descend;
+  $descend = sub {
+    my ($tree, $level) = @_;
+    if ( ref($tree) eq 'HASH' ) {
+      foreach my $k ( sort {
+           -1*($b eq '')    # sort '' to the end
+          or  ($a eq '')    # sort '' to the end
+          or  ($a <=> $b)   # sort numbers as numbers
+          or  ($a cmp $b)   # sort alphabetics as alphabetics
+        } keys %$tree )
+      {
+        $row_template{ $columns[$level] } = $k;
+        &{ $descend }($tree->{$k}, $level + 1);
+        if ( $level == 0 ) {
+          # then insert the total row for the pkgclass
+          $row_template{'total'} = 1; # flag it as a total
+          &{ $descend }($self->{total}->{$k}, 1);
+          $row_template{'total'} = 0;
+        }
+      }
+    } elsif ( ref($tree) eq 'ARRAY' ) {
+      # then we've reached the bottom; elements of this array are arrayrefs
+      # of [ field, taxnums, amount ].
+      # start with the inherited location-element fields
+      my %this_row = %row_template;
+      my %taxnums;
+      foreach my $x (@$tree) {
+        # accumulate taxnums
+        foreach (split(',', $x->[1])) {
+          $taxnums{$_} = 1;
+        }
+        # and money values
+        $this_row{ $x->[0] } = $x->[2];
+      }
+      # store combined taxnums
+      $this_row{taxnums} = join(',', sort { $a cmp $b } keys %taxnums);
+      # and calculate row totals
+      $this_row{sales} = sprintf('%.2f',
+                          $this_row{taxable} +
+                          $this_row{sales_credited} +
+                          $this_row{exempt_cust} +
+                          $this_row{exempt_pkg} + 
+                          $this_row{exempt_monthly}
+                        );
+      $this_row{credits} = sprintf('%.2f',
+                          $this_row{sales_credited} +
+                          $this_row{exempt_credited} +
+                          $this_row{tax_credited}
+                        );
+      # and give it a label
+      if ( $this_row{total} ) {
+        $this_row{label} = 'Total';
+      } else {
+        $this_row{label} = join(', ', grep $_,
+                            $this_row{taxclass},
+                            $this_row{state},
+                            $this_row{county}, # already has ' County' suffix
+                            $this_row{city},
+                            $this_row{district}
+                           );
+      }
+      # and indicate the tax rate, if any
+      my $rate;
+      foreach (keys %taxnums) {
+        $rate ||= $self->{taxrates}->{$_};
+        if ( $rate != $self->{taxrates}->{$_} ) {
+          $rate = 'variable';
+          last;
+        }
+      }
+      if ( $rate eq 'variable' ) {
+        $this_row{rate} = 'variable';
+      } elsif ( $rate > 0 ) {
+        $this_row{rate} = sprintf('%.2f', $rate);
+      }
+      push @rows, \%this_row;
+    }
+  };
+
+  &{ $descend }($self->{data}, 0);
+
+  warn "TABLE:\n".Dumper(\@rows) if $self->{opt}->{debug};
+  return @rows;
+}
+
+sub taxrates {
+  my $self = shift;
+  $self->{taxrates}
+}
+
+sub title {
+  my $self = shift;
+  my $string = '';
+  if ( $self->{opt}->{agentnum} ) {
+    my $agent = qsearchs('agent', { agentnum => $self->{opt}->{agentnum} });
+    $string .= $agent->agent . ' ';
+  }
+  $string .= 'Tax Report: '; # XXX localization
+  if ( $self->{opt}->{beginning} ) {
+    $string .= time2str('%h %o %Y ', $self->{opt}->{beginning});
+  }
+  $string .= 'through ';
+  if ( $self->{opt}->{ending} and $self->{opt}->{ending} < 4294967295 ) {
+    $string .= time2str('%h %o %Y', $self->{opt}->{ending});
+  } else {
+    $string .= 'now';
+  }
+  $string .= ' - ' . $self->{opt}->{taxname};
+  return $string;
+}
+
+1;
index eafb8e0..c060c14 100644 (file)
@@ -54,6 +54,8 @@ FS/Report.pm
 FS/Report/FCC_477.pm
 FS/Report/Table.pm
 FS/Report/Table/Monthly.pm
+FS/Report/Tax/All.pm
+FS/Report/Tax/ByName.pm
 FS/SearchCache.pm
 FS/UI/Web.pm
 FS/UID.pm
index c914d5a..30b32e8 100755 (executable)
@@ -13,9 +13,7 @@ my %params = (
   beginning => $beginning,
   ending    => $ending,
 );
-$params{country} = $cgi->param('country');
 $params{debug}   = $DEBUG;
-$params{breakdown} = { map { $_ => 1 } $cgi->param('breakdown') };
 
 my $agentname;
 if ( $cgi->param('agentnum') =~ /^(\d+)$/ ) {
@@ -24,15 +22,38 @@ if ( $cgi->param('agentnum') =~ /^(\d+)$/ ) {
   $agentname = $agent->agentname;
 }
 
-# allow anything in here; FS::Report::Tax will treat it as unsafe
-if ( length($cgi->param('taxname')) ) {
-  $params{taxname} = $cgi->param('taxname');
+# credit date behavior: limit by the date of the credit application, or
+# the invoice?
+if ( $cgi->param('credit_date') eq 'cust_credit_bill' ) {
+  $params{credit_date} = 'cust_credit_bill';
 } else {
-  die "taxname required";
+  $params{credit_date} = 'cust_bill';
+}
+
+my $all = $cgi->param('all');
+my $report_class;
+
+if ( $all ) {
+  $report_class = 'FS::Report::Tax::All';
+} else {
+  $report_class = 'FS::Report::Tax::ByName';
+  $params{country} = $cgi->param('country');
+  $params{breakdown} = { map { $_ => 1 } $cgi->param('breakdown') };
+
+  # allow anything in here; FS::Report::Tax will treat it as unsafe
+  if ( length($cgi->param('taxname')) ) {
+    $params{taxname} = $cgi->param('taxname');
+  } else {
+    die "taxname required";
+  }
+}
+
+if ($DEBUG) {
+  warn "REPORT: $report_class\nPARAMS:\n".Dumper(\%params)."\n\n";
 }
 
 # generate the report
-my $report = FS::Report::Tax->report_internal(%params);
+my $report = $report_class->report(%params);
 my @rows = $report->table; # array of hashrefs
 
 my %pkgclass_name = map { $_->classnum, $_->classname } qsearch('pkg_class');
index bbb3bc1..410fe46 100644 (file)
@@ -78,14 +78,6 @@ TD.rowhead { font-weight: bold; text-align: left; padding: 0px 3px }
 %   # cust_bill_pkg.cgi wants a list of specific taxnums (and package class)
 %   # cust_credit_bill_pkg.html wants a geographic scope (and package class)
 %   my $rowlink = ';taxnum=' . $row->{taxnums};
-% # DON'T EVER USE THIS
-% #  my $rowregion = ';country=' . $cgi->param('country');
-% #  foreach my $loc (qw(state county city district)) {
-% #    if ( $row->{$loc} ) {
-% #      $rowregion .= ";$loc=" . uri_escape($row->{$loc});
-% #    }
-% #  }
-%   # and also the package class, if we're limiting package class
 %   if ( $params{breakdown}->{pkgclass} ) {
 %     $rowlink .= ';classnum=' . ($row->{pkgclass} || 0);
 % #    $rowregion .= ';classnum=' . ($row->{pkgclass} || 0);
@@ -96,7 +88,26 @@ TD.rowhead { font-weight: bold; text-align: left; padding: 0px 3px }
 %   }
   <TR CLASS="row<% $rownum % 2 %>">
 %   # Row label
-    <TD CLASS="rowhead"><% $row->{label} |h %></TD>
+%   # Special: If this report is showing all taxes, link the row label to
+%   # the detailed tax report for that taxname/country.
+    <TD CLASS="rowhead">
+%   if ( $all ) {
+%     my $newcgi = CGI->new($cgi);
+%     $newcgi->delete('all');
+%     $newcgi->param('country', $row->{country});
+%     $newcgi->param('taxname', $row->{taxname});
+%     $newcgi->param('breakdown', qw(city district));
+
+      <A HREF="<% encode_entities( $newcgi->self_url ) %>">
+        <% $row->{label} |h %>
+      </A>
+
+%   } else { # on the per-taxname report, just show the label with no link
+
+        <% $row->{label} |h %>
+
+%   }
+    </TD>
     <TD>
 %   # Total sales
       <A HREF="<% $saleslink . $rowlink %>">
@@ -167,7 +178,8 @@ TD.rowhead { font-weight: bold; text-align: left; padding: 0px 3px }
 % } # foreach my $row
 % # at the end of everything
   </TBODY>
-% if ( $report->{out_sales} > 0 ) {
+% # the all-taxes report doesn't have "out of region"
+% if ( !$all and $report->{out_sales} > 0 ) {
   <TBODY CLASS="total" STYLE="background-color: #cccccc; line-height: 3">
     <TR>
       <TD CLASS="rowhead">
@@ -175,7 +187,7 @@ TD.rowhead { font-weight: bold; text-align: left; padding: 0px 3px }
       </TD>
       <TD STYLE="text-align: right">
         <A HREF="<% $saleslink %>;out=1;taxname=<% encode_entities($params{'taxname'}) %>">
-          <% $money_sprintf->( $report->{out_sales } ) %>
+          <% $money_sprintf->( $report->{out_sales} ) %>
         </A>
       </TD>
       <TD COLSPAN=0></TD>
@@ -254,7 +266,7 @@ TD.rowhead { font-weight: bold; text-align: left; padding: 0px 3px }
 %   $prev_row = $row;
 % } # foreach my $row
 % # "out of taxable region" for credits (there is a need for it)
-% if ( $report->{out_credit} > 0 ) {
+% if ( !$all and $report->{out_credit} > 0 ) {
 %   my $creditlink = "cust_credit_bill_pkg.html?out=1;$dateagentlink";
 %   if ( $params{'credit_date'} eq 'cust_credit_bill' ) {
 %     $creditlink =~ s/begin/credit_begin/;
@@ -268,7 +280,7 @@ TD.rowhead { font-weight: bold; text-align: left; padding: 0px 3px }
       </TD>
       <TD STYLE="text-align: right">
         <A HREF="<% $creditlink %>">
-          <% $money_sprintf->( $report->{out_credit } ) %>
+          <% $money_sprintf->( $report->{out_credit} ) %>
         </A>
       </TD>
       <TD COLSPAN=0></TD>
@@ -295,33 +307,48 @@ my %params = (
   beginning => $beginning,
   ending    => $ending,
 );
-$params{country} = $cgi->param('country');
 $params{debug}   = $DEBUG;
-$params{breakdown} = { map { $_ => 1 } $cgi->param('breakdown') };
-
 my $agentname;
+
+# filter by agentnum
 if ( $cgi->param('agentnum') =~ /^(\d+)$/ ) {
   my $agent = FS::agent->by_key($1) or die "unknown agentnum $1";
   $params{agentnum} = $1;
   $agentname = $agent->agentname;
 }
 
-# allow anything in here; FS::Report::Tax will treat it as unsafe
-if ( length($cgi->param('taxname')) ) {
-  $params{taxname} = $cgi->param('taxname');
-} else {
-  die "taxname required";
-}
-
+# credit date behavior: limit by the date of the credit application, or
+# the invoice?
 if ( $cgi->param('credit_date') eq 'cust_credit_bill' ) {
   $params{credit_date} = 'cust_credit_bill';
 } else {
   $params{credit_date} = 'cust_bill';
 }
 
-warn "PARAMS:\n".Dumper(\%params)."\n\n" if $DEBUG;
+my $all = $cgi->param('all');
+my $report_class;
+
+if ( $all ) {
+  # then show the master report, no country, no taxname, no breakdown
+  $report_class = 'FS::Report::Tax::All';
+} else {
+  $report_class = 'FS::Report::Tax::ByName';
+  $params{country} = $cgi->param('country');
+  $params{breakdown} = { map { $_ => 1 } $cgi->param('breakdown') };
+
+  # allow anything in here; FS::Report::Tax will treat it as unsafe
+  if ( length($cgi->param('taxname')) ) {
+    $params{taxname} = $cgi->param('taxname');
+  } else {
+    die "taxname required";
+  }
+}
+
+if ($DEBUG) {
+  warn "REPORT: $report_class\nPARAMS:\n".Dumper(\%params)."\n\n";
+}
 
-my $report = FS::Report::Tax->report_internal(%params);
+my $report = $report_class->report(%params);
 my @rows = $report->table; # array of hashrefs
 
 my $money_char = $conf->config('money_char') || '$';
index 8d8d108..f920adb 100755 (executable)
@@ -8,6 +8,20 @@
 
   <& /elements/tr-input-beginning_ending.html &>
 
+  <tr>
+    <td></td>
+    <td colspan=2 style="font-weight: bold">
+      <& /elements/radio.html,
+        'field' => 'all',
+        'value' => 1,
+        'curr_value' => 1,
+      &> All taxes
+      <& /elements/radio.html,
+        'field' => 'all',
+        'value' => 0,
+      &> A specific tax
+    </td>
+  </tr>
   <& /elements/tr-select.html,
     'label'         => 'Country',
     'field'         => 'country',
 
 </FORM>
 
+<script>
+$(document).ready(function() {
+  $('[name=all]').on('change', function(ev) {
+    // disable country/taxname/breakdown if 'all' = 1
+    if (this.checked) {
+      var disabled = (this.value == 1);
+      $('[name=country').prop('disabled', disabled);
+      $('[name=taxname').prop('disabled', disabled);
+      $('[name=breakdown').prop('disabled', disabled);
+    }
+  });
+  $('[name=all]').change();
+});
+</script>
+
 <% include('/elements/footer.html') %>
 <%init>
 
index 4b28c93..91abd1b 100644 (file)
@@ -113,7 +113,7 @@ while ($countdate < $enddate) {
   # run a report for each tax name
   foreach my $taxname (@taxnames) {
     $params{'taxname'} = $taxname;
-    my $report = FS::Report::Tax->report_internal(%params);
+    my $report = FS::Report::Tax::ByName->report(%params);
 
     # extract totals from report, kinda awkward
     my $pkgclass = ''; # this will get more complicated if we breakdown by pkgclass