Make sure unguarded() does not change unguarded state on exception

laravel · Apr 16, 2016 · d256d9b · d256d9b
1 parent b45df80
commit d256d9b
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 3 deletions.
diff --git a/src/Illuminate/Database/Eloquent/Model.php b/src/Illuminate/Database/Eloquent/Model.php
@@ -2266,9 +2266,11 @@ public static function unguarded(callable $callback)
 
         static::unguard();
 
-        $result = $callback();
-
-        static::reguard();
+        try {
+            $result = $callback();
+        } finally {
+            static::reguard();
+        }
 
         return $result;
     }

diff --git a/tests/Database/DatabaseEloquentModelTest.php b/tests/Database/DatabaseEloquentModelTest.php
@@ -794,6 +794,18 @@ public function testUnguardedCallDoesNotChangeUnguardedState()
         Model::reguard();
     }
 
+    public function testUnguardedCallDoesNotChangeUnguardedStateOnException()
+    {
+        try {
+            Model::unguarded(function () {
+                throw new Exception;
+            });
+        } catch (Exception $e) {
+            // ignore the exception
+        }
+        $this->assertFalse(Model::isUnguarded());
+    }
+
     public function testHasOneCreatesProperRelation()
     {
         $model = new EloquentModelStub;