From: ivan <ivan>
Date: Mon, 9 May 2011 20:25:57 +0000 (+0000)
Subject: better retry behavior for non-found taqua CDR rewrites, RT#12181
X-Git-Tag: freeside_2_3_0~280
X-Git-Url: http://git.freeside.biz/gitweb/?p=freeside.git;a=commitdiff_plain;h=c0e4e71f6e2888fe83c6b288a84dcc4a4897c848

better retry behavior for non-found taqua CDR rewrites, RT#12181
---

diff --git a/FS/bin/freeside-cdrrewrited b/FS/bin/freeside-cdrrewrited
index a365d73db..09a2643e5 100644
--- a/FS/bin/freeside-cdrrewrited
+++ b/FS/bin/freeside-cdrrewrited
@@ -41,12 +41,12 @@ while (1) {
   # rewritten "skipped", i.e. why we're a daemon in the first place
   # instead of just doing this search like normal CDRs
 
+  #hmm :/
   my @recent = grep { ($accountcode_unmatch{$_} + $accountcode_retry) < time }
                  keys %accountcode_unmatch;
-  #hmm :/
-  my $extra_sql = '';
-  $extra_sql .= ' AND acctid NOT IN (', join(',', @recent). ') '
-    if @recent;
+  my $extra_sql = scalar(@recent)
+                    ? ' AND acctid NOT IN (', join(',', @recent). ') '
+                    : '';
 
   my $found = 0;
   foreach my $cdr (