add try catch on order saving to keep notifications out of in progress state

03f410c9 · Rik ter Beek · 8159aba3 · 03f410c9
Commit 03f410c9 authored Jan 08, 2019 by Rik ter Beek
Show whitespace changes
Inline Side-by-side

Showing with 7 additions and 2 deletions

Model/Cron.php Model/Cron.php +7 -2

No files found.
--- a/Model/Cron.php
+++ b/Model/Cron.php
@@ -416,8 +416,13 @@ class Cron
                $this->_processNotification();
            }
-            $this->_order->save();
+            try {
                // set done to true
+                $this->_order->save();
+            } catch (\Exception $e) {
+                $this->_adyenLogger->addAdyenNotificationCronjob($e->getMessage());
+            }
            $this->_updateNotification($notification, false, true);
            $this->_adyenLogger->addAdyenNotificationCronjob(
                sprintf("Notification %s is processed", $notification->getEntityId())