diff options
| author | ivan <ivan> | 2011-02-10 01:03:58 +0000 |
|---|---|---|
| committer | ivan <ivan> | 2011-02-10 01:03:58 +0000 |
| commit | 64ccb0c8d26c387b323fb34e671b34c62d2c0c81 (patch) | |
| tree | 030a1a30461ac8c5e941c12fb7615795fcb8147b /httemplate | |
| parent | 83b363898f0f95a4e85d44b88cba2f0751c27361 (diff) | |
fix cch tax report for more than one cust_bill_pkg_tax_rate_location per line item, RT#11483
Diffstat (limited to 'httemplate')
| -rw-r--r-- | httemplate/search/cust_bill_pkg.cgi | 10 | ||||
| -rwxr-xr-x | httemplate/search/report_newtax.cgi | 2 |
2 files changed, 9 insertions, 3 deletions
diff --git a/httemplate/search/cust_bill_pkg.cgi b/httemplate/search/cust_bill_pkg.cgi index 96de75b..b974304 100644 --- a/httemplate/search/cust_bill_pkg.cgi +++ b/httemplate/search/cust_bill_pkg.cgi @@ -463,7 +463,8 @@ if ( $cgi->param('pkg_tax') ) { } else { - $count_query = "SELECT COUNT(*), "; + #$count_query = "SELECT COUNT(*), "; + $count_query = "SELECT COUNT(DISTINCT billpkgnum), "; if ( $use_usage eq 'recurring' ) { $count_query .= "SUM(setup + recur - usage)"; @@ -472,7 +473,12 @@ if ( $cgi->param('pkg_tax') ) { } elsif ( $unearned ) { $count_query .= "SUM(cust_bill_pkg.recur)"; } else { - $count_query .= "SUM(cust_bill_pkg.setup + cust_bill_pkg.recur)"; + if ( scalar( grep( /locationtaxid/, $cgi->param ) ) || + $cgi->param('iscredit') eq 'rate') { + $count_query .= "SUM( COALESCE(amount, cust_bill_pkg.setup + cust_bill_pkg.recur))"; + } else { + $count_query .= "SUM(cust_bill_pkg.setup + cust_bill_pkg.recur)"; + } } if ( $unearned ) { diff --git a/httemplate/search/report_newtax.cgi b/httemplate/search/report_newtax.cgi index 6a2cbb0..7851462 100755 --- a/httemplate/search/report_newtax.cgi +++ b/httemplate/search/report_newtax.cgi @@ -123,7 +123,7 @@ foreach my $t (qsearch({ table => 'cust_bill_pkg', my $taxwhere = "FROM cust_bill_pkg $addl_from $where AND payby != 'COMP' ". "AND ". join( ' AND ', map { "( $_ = ? OR ? = '' AND $_ IS NULL)" } @taxparam ); - my $sql = "SELECT SUM(cust_bill_pkg.setup+cust_bill_pkg.recur) ". + my $sql = "SELECT SUM(COALESCE(amount, cust_bill_pkg.setup+cust_bill_pkg.recur)) ". " $taxwhere AND cust_bill_pkg.pkgnum = 0"; my $x = scalar_sql($t, [ map { $_, $_ } @params ], $sql ); |