From: Ivan Kohler <ivan@freeside.biz>
Date: Wed, 5 Jun 2013 20:26:16 +0000 (-0700)
Subject: fix intro periods getting re-added on package change when delay_setup is on, RT#23440
X-Git-Url: http://git.freeside.biz/gitweb/?a=commitdiff_plain;h=e2956e14f9ddb53302c594aa343db3a59bacdda3;p=freeside.git

fix intro periods getting re-added on package change when delay_setup is on, RT#23440
---

diff --git a/FS/FS/cust_pkg.pm b/FS/FS/cust_pkg.pm
index 4ce2e8e0a..65bfc80ae 100644
--- a/FS/FS/cust_pkg.pm
+++ b/FS/FS/cust_pkg.pm
@@ -255,7 +255,8 @@ The following options are available:
 
 =item change
 
-If set true, supresses any referral credit to a referring customer.
+If set true, supresses actions that should only be taken for new package
+orders.  (Currently this includes: intro periods when delay_setup is on.)
 
 =item options
 
@@ -303,8 +304,12 @@ sub insert {
     }
   }
 
-  my $free_days = $part_pkg->option('free_days',1);
-  if ( $free_days && $part_pkg->option('delay_setup',1) ) { #&& !$self->start_date
+  if (    ! $options{'change'}
+       && ( my $free_days = $part_pkg->option('free_days',1) )
+       && $part_pkg->option('delay_setup',1)
+       #&& ! $self->start_date
+     )
+  {
     my ($mday,$mon,$year) = (localtime(time) )[3,4,5];
     #my $start_date = ($self->start_date || timelocal(0,0,0,$mday,$mon,$year)) + 86400 * $free_days;
     my $start_date = timelocal(0,0,0,$mday,$mon,$year) + 86400 * $free_days;