Fix the stupidest type check ever.

hirethunk · Feb 13, 2024 · 5895869 · 5895869
1 parent 3b782be
commit 5895869
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 2 deletions.
diff --git a/src/Lifecycle/Guards.php b/src/Lifecycle/Guards.php
@@ -67,8 +67,8 @@ protected function passesAuthorization(): bool
             if ($result instanceof Response) {
                 return $result->authorize();
             }
-
-            if ($result instanceof bool) {
+            
+            if (is_bool($result)) {
                 return $result;
             }
 

diff --git a/tests/Feature/PendingEventChecksTest.php b/tests/Feature/PendingEventChecksTest.php
@@ -22,6 +22,20 @@
     $this->assertFalse($event->isAllowed());
 });
 
+it('supports boolean authorization', function () {
+    $event = EventWithBooleanAuth::make([
+        'allowed' => true,
+    ]);
+
+    $this->assertTrue($event->isAllowed());
+
+    $event =  EventWithBooleanAuth::make([
+        'allowed' => false,
+    ]);
+
+    $this->assertFalse($event->isAllowed());
+});
+
 it('can test validation on a pending event', function () {
     SpecialState::factory()->create([
         'name' => 'daniel',
@@ -60,6 +74,17 @@
     $this->assertFalse($event->isValid());
 });
 
+
+class EventWithBooleanAuth extends Event
+{
+    public bool $allowed;
+
+    public function authorize()
+    {
+        return $this->allowed;
+    }
+}
+
 class EventWithMultipleStates extends Event
 {
     #[StateId(SpecialState::class)]