[freeside-commits] branch master updated. fb33bc6850eb5ebf1f63d79afd503202b277b9a3

[Mark Wells]

The branch, master has been updated
       via  fb33bc6850eb5ebf1f63d79afd503202b277b9a3 (commit)
      from  576bec49a5433f1c141caa2f01d82ad239805bff (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 fb33bc6850eb5ebf1f63d79afd503202b277b9a3
Author: Mark Wells <mark at freeside.biz>
Date:   Fri Feb 8 16:24:45 2013 -0800

    keep package dates when changing packages and not crediting, from #16066

diff --git a/FS/FS/cust_pkg.pm b/FS/FS/cust_pkg.pm
index c2077c5..9c3d16a 100644
--- a/FS/FS/cust_pkg.pm
+++ b/FS/FS/cust_pkg.pm
@@ -1753,9 +1753,11 @@ sub change {
   # going to be credited for remaining time, don't keep setup, bill, 
   # or last_bill dates, and DO pass the flag to cancel() to credit 
   # the customer.
-  if ( $opt->{'pkgpart'} and $opt->{'pkgpart'} != $self->pkgpart ) {
+  if ( $opt->{'pkgpart'} 
+       and $opt->{'pkgpart'} != $self->pkgpart
+       and $self->part_pkg->option('unused_credit_change', 1) ) {
+    $unused_credit = 1;
     $keep_dates = 0;
-    $unused_credit = 1 if $self->part_pkg->option('unused_credit_change', 1);
     $hash{$_} = '' foreach qw(setup bill last_bill);
   }
 

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

Summary of changes:
 FS/FS/cust_pkg.pm |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)