[freeside-commits] branch master updated. fda4176a713fac94af01af665f6ac94bb14ea5c9

Ivan ivan at 420.am
Mon Jan 6 21:30:50 PST 2014


The branch, master has been updated
       via  fda4176a713fac94af01af665f6ac94bb14ea5c9 (commit)
      from  d7006fb84965da14819c4ad3581fcb5e907a14d3 (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 fda4176a713fac94af01af665f6ac94bb14ea5c9
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Mon Jan 6 21:30:49 2014 -0800

    fix perf edge case with multiple large packages not just one, RT#26097

diff --git a/FS/FS/cust_main/Packages.pm b/FS/FS/cust_main/Packages.pm
index f819970..847ac05 100644
--- a/FS/FS/cust_main/Packages.pm
+++ b/FS/FS/cust_main/Packages.pm
@@ -501,8 +501,8 @@ sub sort_packages {
     return 0  if !$a_num_cust_svc && !$b_num_cust_svc;
     return -1 if  $a_num_cust_svc && !$b_num_cust_svc;
     return 1  if !$a_num_cust_svc &&  $b_num_cust_svc;
-    my @a_cust_svc = $a->cust_svc;
-    my @b_cust_svc = $b->cust_svc;
+    my @a_cust_svc = $a->cust_svc_unsorted;
+    my @b_cust_svc = $b->cust_svc_unsorted;
     return 0  if !scalar(@a_cust_svc) && !scalar(@b_cust_svc);
     return -1 if  scalar(@a_cust_svc) && !scalar(@b_cust_svc);
     return 1  if !scalar(@a_cust_svc) &&  scalar(@b_cust_svc);

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

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




More information about the freeside-commits mailing list