[freeside-commits] branch master updated. dde3deba8c26a17e14a71786a0dcfa9d83b44771

Ivan Kohler ivan at freeside.biz
Thu Aug 29 15:35:03 PDT 2019


The branch, master has been updated
       via  dde3deba8c26a17e14a71786a0dcfa9d83b44771 (commit)
      from  c2c538bb32f8ad5263c41a0a8ca5ee7b8b12f0dc (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 dde3deba8c26a17e14a71786a0dcfa9d83b44771
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Thu Aug 29 15:34:28 2019 -0700

    these errors are fatal for now, and fix them, RT#83146, RT#40206

diff --git a/FS/bin/freeside-cdr-asterisk_sql b/FS/bin/freeside-cdr-asterisk_sql
index 97e475868..cd765deee 100755
--- a/FS/bin/freeside-cdr-asterisk_sql
+++ b/FS/bin/freeside-cdr-asterisk_sql
@@ -133,9 +133,9 @@ while ( my $row = $sth->fetchrow_hashref ) {
   $cdr->amaflags('') unless $cdr->amaflags =~ /^(\d+)$/;
 
   if ( my $error = $cdr->insert ) {
-    log_msg( error =>
-      sprintf 'Non-fatal failure to import acctid(%s) from table(%s): %s',
-        $row->acctid,
+    log_and_die( error =>
+      sprintf 'Fatal failure to import uniqueid(%s) from table(%s): %s',
+        $row->{'uniqueid'},
         $table,
         $error
     );
@@ -161,9 +161,9 @@ while ( my $row = $sth->fetchrow_hashref ) {
     if ( $dbi->do($usql, @args) ) {
       $updates++;
     } else {
-      log_msg( error =>
-        sprintf 'Non-fatal failure set status(done) acctid(%s) table(%s): %s',
-          $row->acctid,
+      log_and_die( error =>
+        sprintf 'Fatal failure set status(done) uniqueid(%s) table(%s): %s',
+          $row->{'uniqueid'},
           $table,
           $dbi->errstr
       );

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

Summary of changes:
 FS/bin/freeside-cdr-asterisk_sql | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)




More information about the freeside-commits mailing list