[freeside-commits] branch FREESIDE_2_3_BRANCH updated. efb2486d14bb726c53a6f06e6b877352c648c2fc

[Ivan]

The branch, FREESIDE_2_3_BRANCH has been updated
       via  efb2486d14bb726c53a6f06e6b877352c648c2fc (commit)
      from  1929eefb29ccec72feabb3313ddb8a0640f42483 (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 efb2486d14bb726c53a6f06e6b877352c648c2fc
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Tue Sep 10 22:20:06 2013 -0700

    improve performance of package ordering, RT#24955, fallout from future start dates, RT#20686

diff --git a/FS/FS/part_pkg.pm b/FS/FS/part_pkg.pm
index 8db0e99..332bb62 100644
--- a/FS/FS/part_pkg.pm
+++ b/FS/FS/part_pkg.pm
@@ -875,28 +875,15 @@ sub can_discount { 0; }
 # whether the plan allows changing the start date
 sub can_start_date { 1; }
 
-# the default start date; takes an FS::cust_main as an argument
-sub default_start_date {
+# the delay start date if present
+sub delay_start_date {
   my $self = shift;
-  my $cust_main = shift;
-  my $conf = FS::Conf->new;
 
-  if ( $self->delay_start ) {
-    my $delay = $self->delay_start;
+  my $delay = $self->delay_start or return '';
     
-    my ($mday,$mon,$year) = (localtime(time))[3,4,5];
-    my $start_date = timelocal(0,0,0,$mday,$mon,$year) + 86400 * $delay;
-    return $start_date;
+  my ($mday,$mon,$year) = (localtime(time))[3,4,5];
+  timelocal(0,0,0,$mday,$mon,$year) + 86400 * $delay;
 
-  } elsif ( $conf->exists('order_pkg-no_start_date') ) {
-
-    return ''
-
-  } else {
-    
-    return $cust_main->next_bill_date;
-  
-  }
 }
 
 sub freqs_href {
diff --git a/httemplate/misc/cust-part_pkg.cgi b/httemplate/misc/cust-part_pkg.cgi
index 4662be1..178d12e 100644
--- a/httemplate/misc/cust-part_pkg.cgi
+++ b/httemplate/misc/cust-part_pkg.cgi
@@ -23,10 +23,17 @@ my @part_pkg = qsearch({
   'order_by'  => 'ORDER BY pkg',
 });
 
-my $date_format = FS::Conf->new->config('date_format') || '%m/%d/%Y';
+my $conf = new FS::Conf;
+
+my $date_format = $conf->config('date_format') || '%m/%d/%Y';
+
+my $default_start_date = $conf->exists('order_pkg-no_start-date')
+                           ? ''
+                           : $cust_main->next_bill_date;
 
 my @return = map  {
-                    my $start_date = $_->default_start_date($cust_main);
+                    my $start_date = $_->delay_start_date
+                                   || $default_start_date;
                     $start_date = time2str($date_format, $start_date)
                       if $start_date;
                     ( $_->pkgpart,

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

Summary of changes:
 FS/FS/part_pkg.pm                 |   23 +++++------------------
 httemplate/misc/cust-part_pkg.cgi |   11 +++++++++--
 2 files changed, 14 insertions(+), 20 deletions(-)