[freeside-commits] branch FREESIDE_2_3_BRANCH updated. 9bca6157d6ae81eaacb4eee93757d700240a6de7

[Ivan]

The branch, FREESIDE_2_3_BRANCH has been updated
       via  9bca6157d6ae81eaacb4eee93757d700240a6de7 (commit)
      from  5bb3332639ffadd71b00ad41fd2a7cbb038092ab (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 9bca6157d6ae81eaacb4eee93757d700240a6de7
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Mon Jan 6 21:30:51 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 57c4e52..3c96fc7 100644
--- a/FS/FS/cust_main/Packages.pm
+++ b/FS/FS/cust_main/Packages.pm
@@ -333,8 +333,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(-)