From: Ivan Kohler <ivan@freeside.biz>
Date: Tue, 12 Jun 2012 08:29:35 +0000 (-0700)
Subject: fix discount-show_avail without invoice_sections from hiding previous balances, RT... 
X-Git-Url: http://git.freeside.biz/gitweb/?a=commitdiff_plain;h=bb8515fe4d9b5c6e67813bae051ce8943f6b3dcf;p=freeside.git

fix discount-show_avail without invoice_sections from hiding previous balances, RT#15159
---

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 {