diff options
| author | Ivan Kohler <ivan@freeside.biz> | 2012-06-12 01:29:35 -0700 |
|---|---|---|
| committer | Ivan Kohler <ivan@freeside.biz> | 2012-06-12 01:29:35 -0700 |
| commit | bb8515fe4d9b5c6e67813bae051ce8943f6b3dcf (patch) | |
| tree | 049a640062cf1ae5dea9b748c17c8c583f59cdab | |
| parent | a33279c7c63c2bae19204d0a583794fada01ae13 (diff) | |
fix discount-show_avail without invoice_sections from hiding previous balances, RT#15159
| -rw-r--r-- | FS/FS/cust_bill.pm | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/FS/FS/cust_bill.pm b/FS/FS/cust_bill.pm index 8cd508cf1..46172f3e8 100644 --- a/FS/FS/cust_bill.pm +++ b/FS/FS/cust_bill.pm @@ -2981,6 +2981,12 @@ sub print_generic { my $late_sections = []; my $extra_sections = []; my $extra_lines = (); + + my $default_section = { 'description' => '', + 'subtotal' => '', + 'no_subtotal' => 1, + }; + if ( $multisection ) { ($extra_sections, $extra_lines) = $self->_items_extra_usage_sections($escape_function_nonbsp, $format) @@ -3012,8 +3018,7 @@ sub print_generic { } } else {# not multisection # make a default section - push @sections, { 'description' => '', 'subtotal' => '', - 'no_subtotal' => 1 }; + push @sections, $default_section; # and calculate the finance charge total, since it won't get done otherwise. # XXX possibly other totals? # XXX possibly finance_pkgclass should not be used in this manner? @@ -3046,7 +3051,8 @@ sub print_generic { }; $detail->{'ref'} = $line_item->{'pkgnum'}; $detail->{'quantity'} = 1; - $detail->{'section'} = $previous_section; + $detail->{'section'} = $multisection ? $previous_section + : $default_section; $detail->{'description'} = &$escape_function($line_item->{'description'}); if ( exists $line_item->{'ext_description'} ) { @{$detail->{'ext_description'}} = map { |