[freeside-commits] branch master updated. d56135c1e98194db7ad91cb12ef5944543180a0e

[Mark Wells]

The branch, master has been updated
       via  d56135c1e98194db7ad91cb12ef5944543180a0e (commit)
      from  b389ce7f754dc37af90f97e11f8ec5b0b6c09842 (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 d56135c1e98194db7ad91cb12ef5944543180a0e
Author: Mark Wells <mark at freeside.biz>
Date:   Wed Sep 4 14:59:30 2013 -0700

    optimize prepaid balance query, #24846

diff --git a/FS/FS/ClientAPI/PrepaidPhone.pm b/FS/FS/ClientAPI/PrepaidPhone.pm
index b05fb4f..2cea3c2 100644
--- a/FS/FS/ClientAPI/PrepaidPhone.pm
+++ b/FS/FS/ClientAPI/PrepaidPhone.pm
@@ -307,14 +307,8 @@ sub prepaid_phone_balance {
       foreach my $cust_svc ( @cust_svc ) {
         
         my $svc_x = $cust_svc->svc_x;
-
-        #XXX optimization: a single SQL query to return the total amount
-        my $cdr_search = $svc_x->psearch_cdrs(%options);
-        $cdr_search->limit(1000);
-        $cdr_search->increment(0);
-        while ( my $cdr = $cdr_search->fetch ) {
-          $balance -= $cdr->rated_price;
-        }
+        my $sum_cdr = $svc_x->sum_cdrs(%options);
+        $balance += $sum_cdr->rated_price;
 
       }
 

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

Summary of changes:
 FS/FS/ClientAPI/PrepaidPhone.pm |   10 ++--------
 1 files changed, 2 insertions(+), 8 deletions(-)