[freeside-commits] branch master updated. f054f4db2a70c60b3adbaaa1672eefcddd62d96e

[Mark Wells]

The branch, master has been updated
       via  f054f4db2a70c60b3adbaaa1672eefcddd62d96e (commit)
      from  d8484ba64691b5bf02b289393706d73ab531c403 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit f054f4db2a70c60b3adbaaa1672eefcddd62d96e
Author: Mark Wells <mark at freeside.biz>
Date:   Sat Feb 28 13:20:39 2015 -0800

    change interaction between unsuspend_adjust_bill and unused_credit_suspend, #31651

diff --git a/FS/FS/cust_pkg.pm b/FS/FS/cust_pkg.pm
index ae86ca0..8eb4ce3 100644
--- a/FS/FS/cust_pkg.pm
+++ b/FS/FS/cust_pkg.pm
@@ -1684,15 +1684,20 @@ sub unsuspend {
            and ! $self->option('no_suspend_bill',1)
          )
       or $hash{'order_date'} == $hash{'susp'}
-      or $self->part_pkg->option('unused_credit_suspend')
-      or ( ref($reason) and $reason->unused_credit )
   ) {
     $adjust_bill = 0;
   }
 
-  # then add the length of time suspended to the bill date
   if ( $adjust_bill ) {
-    $hash{'bill'} = ( $hash{'bill'} || $hash{'setup'} ) + $inactive
+    if (    $self->part_pkg->option('unused_credit_suspend')
+         or ( ref($reason) and $reason->unused_credit ) ) {
+      # then the customer was credited for the unused time before suspending,
+      # so their next bill should be immediate 
+      $hash{'bill'} = time;
+    } else {
+      # add the length of time suspended to the bill date
+      $hash{'bill'} = ( $hash{'bill'} || $hash{'setup'} ) + $inactive;
+    }
   }
 
   $hash{'susp'} = '';

-----------------------------------------------------------------------

Summary of changes:
 FS/FS/cust_pkg.pm |   13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)