Merge pull request #217 from mailchimp/Issue216

closes #216
mailchimp · Feb 21, 2018 · 3130520 · 3130520
2 parents 65ac09d + ae228ab
commit 3130520
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/Model/Api/Result.php b/Model/Api/Result.php
@@ -66,10 +66,14 @@ public function processResponses($storeId, $isMailChimpStoreId = false, $mailchi
         foreach ($collection as $item) {
             try {
                 $files = $this->getBatchResponse($item->getBatchId(), $storeId);
-                if (count($files)) {
+                if (is_array($files)&&count($files)) {
                     $this->processEachResponseFile($files, $item->getBatchId(), $mailchimpStoreId, $storeId);
                     $item->setStatus('completed');
                     $item->getResource()->save($item);
+                } elseif($files === false) {
+                    $item->setStatus('canceled');
+                    $item->getResource()->save($item);
+                    continue;
                 }
                 $baseDir = $this->_helper->getBaseDir();
                 if (is_dir($baseDir . DIRECTORY_SEPARATOR . 'var' . DIRECTORY_SEPARATOR . self::MAILCHIMP_TEMP_DIR . DIRECTORY_SEPARATOR . $item->getBatchId())) {
@@ -118,6 +122,7 @@ public function getBatchResponse($batchId, $storeId = null)
             }
         } catch (\Mailchimp_Error $e) {
             $this->_helper->log($e->getMessage());
+            return false;
         } catch (\Exception $e) {
             $this->_helper->log($e->getMessage());
         }