handle exceptions on LoggableMigrations

irontec · Aug 11, 2022 · 169cfdb · 169cfdb
1 parent 52c2d81
commit 169cfdb
Showing 1 changed file with 14 additions and 3 deletions.
diff --git a/Infrastructure/Persistence/Doctrine/LoggableMigration.php b/Infrastructure/Persistence/Doctrine/LoggableMigration.php
@@ -15,16 +15,28 @@ abstract class LoggableMigration extends AbstractMigration
 
     public function postUp(Schema $schema): void
     {
-        $this->logChanges('up');
+        $this->logChangesAndHandleErrors('up');
         parent::postUp(...func_get_args());
     }
 
     public function postDown(Schema $schema): void
     {
-        $this->logChanges('down');
+        $this->logChangesAndHandleErrors('down');
         parent::postDown(...func_get_args());
     }
 
+    private function logChangesAndHandleErrors(string $direction)
+    {
+        try {
+            $this->logChanges($direction);
+        } catch (\Exception $e) {
+            $this->warnIf(
+                true,
+                $e->getMessage()
+            );
+        }
+    }
+
     private function logChanges(string $direction)
     {
         $event = new CommandWasExecuted(
@@ -76,7 +88,6 @@ private function logChanges(string $direction)
         }
     }
 
-
     /**
      * @param string $entity
      * @param array $commandData