[freeside-commits] branch master updated. e08dc441d98b98643d5108ce8c189fcf12693e0c
Ivan
ivan at 420.am
Mon Jan 6 21:45:46 PST 2014
The branch, master has been updated
via e08dc441d98b98643d5108ce8c189fcf12693e0c (commit)
from fda4176a713fac94af01af665f6ac94bb14ea5c9 (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 e08dc441d98b98643d5108ce8c189fcf12693e0c
Author: Ivan Kohler <ivan at freeside.biz>
Date: Mon Jan 6 21:45:45 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 847ac05..9899bd0 100644
--- a/FS/FS/cust_main/Packages.pm
+++ b/FS/FS/cust_main/Packages.pm
@@ -501,6 +501,7 @@ 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;
+ return 0 if $a_num_cust_svc + $b_num_cust_svc > 20; #for perf, just give up
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);
-----------------------------------------------------------------------
Summary of changes:
FS/FS/cust_main/Packages.pm | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
More information about the freeside-commits
mailing list