From: Ivan Kohler <ivan@freeside.biz>
Date: Fri, 27 Oct 2017 23:34:13 +0000 (-0700)
Subject: don't abort upgrade if a customer doesn't have an echeck cust_main.paytype.  if it... 
X-Git-Url: http://git.freeside.biz/gitweb/?a=commitdiff_plain;h=20fd8098a709fe5c63e8aa418f0a5344d7df9c42;p=freeside.git

don't abort upgrade if a customer doesn't have an echeck cust_main.paytype.  if it's not there, it's not there.
---

diff --git a/FS/FS/cust_main.pm b/FS/FS/cust_main.pm
index 2b21cf135..d71ade04e 100644
--- a/FS/FS/cust_main.pm
+++ b/FS/FS/cust_main.pm
@@ -5773,15 +5773,17 @@ sub _upgrade_data { #class method
       # at the time we do this, also migrate paytype into cust_pay_batch
       # so that batches that are open before the migration can still be 
       # processed
-      my @cust_pay_batch = qsearch('cust_pay_batch', {
-          'custnum' => $cust_main->custnum,
-          'payby'   => 'CHEK',
-          'paytype' => '',
-      });
-      foreach my $cust_pay_batch (@cust_pay_batch) {
-        $cust_pay_batch->set('paytype', $cust_main->get('paytype'));
-        my $error = $cust_pay_batch->replace;
-        die "$error (setting cust_pay_batch.paytype)" if $error;
+      if ( $cust_main->get('paytype') ) {
+        my @cust_pay_batch = qsearch('cust_pay_batch', {
+            'custnum' => $cust_main->custnum,
+            'payby'   => 'CHEK',
+            'paytype' => '',
+        });
+        foreach my $cust_pay_batch (@cust_pay_batch) {
+          $cust_pay_batch->set('paytype', $cust_main->get('paytype'));
+          my $error = $cust_pay_batch->replace;
+          die "$error (setting cust_pay_batch.paytype)" if $error;
+        }
       }
 
       $cust_main->complimentary('Y') if $cust_main->payby eq 'COMP';