When the creation of the original record fails, an attempt to create the relations is still made - this, naturally, fails. By moving this block inside the check, the situation is prevented - at least the relations aren't attempted if the original fails.

1 files changed, 6 insertions, 7 deletions

diff --git a/src/Psecio/Gatekeeper/DataSource/Mysql.php b/src/Psecio/Gatekeeper/DataSource/Mysql.php
index 82a28b5..07ec6d8 100644
--- a/src/Psecio/Gatekeeper/DataSource/Mysql.php
+++ b/src/Psecio/Gatekeeper/DataSource/Mysql.php
@@ -117,13 +117,12 @@ class Mysql extends \Psecio\Gatekeeper\DataSource
         $result = $this->execute($sql, $data);
         if ($result !== false) {
             $model->id = $this->getDb()->lastInsertId();
-        }
-
-        // Now handle the relations - for each of them, get the model, make it and save it
-        foreach ($relations as $index => $item) {
-            $relation = $properties[$index];
-            $instance = new $relation['relation']['model']($this);
-            $instance->create($model, $item);
+            // Now handle the relations - for each of them, get the model, make it and save it
+            foreach ($relations as $index => $item) {
+                $relation = $properties[$index];
+                $instance = new $relation['relation']['model']($this);
+                $instance->create($model, $item);
+            }
         }
 
         return $result;