Added Events PermissionAttached, PermissionDetached, RoleAttached and RoleDetached

Author: sven-arztkonsultation

src/Events/PermissionAttached.php

@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Spatie\Permission\Events;
+
+use Illuminate\Broadcasting\InteractsWithSockets;
+use Illuminate\Database\Eloquent\Model;
+use Illuminate\Foundation\Bus\Dispatchable;
+use Illuminate\Queue\SerializesModels;
+
+class PermissionAttached
+{
+    use Dispatchable;
+    use InteractsWithSockets;
+    use SerializesModels;
+
+    public function __construct(public Model $model)
+    {}
+}

src/Events/PermissionDetached.php

@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Spatie\Permission\Events;
+
+use Illuminate\Broadcasting\InteractsWithSockets;
+use Illuminate\Database\Eloquent\Model;
+use Illuminate\Foundation\Bus\Dispatchable;
+use Illuminate\Queue\SerializesModels;
+
+class PermissionDetached
+{
+    use Dispatchable;
+    use InteractsWithSockets;
+    use SerializesModels;
+
+    public function __construct(public Model $model)
+    {}
+}

src/Events/RoleAttached.php

@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Spatie\Permission\Events;
+
+use Illuminate\Broadcasting\InteractsWithSockets;
+use Illuminate\Database\Eloquent\Model;
+use Illuminate\Foundation\Bus\Dispatchable;
+use Illuminate\Queue\SerializesModels;
+
+class RoleAttached
+{
+    use Dispatchable;
+    use InteractsWithSockets;
+    use SerializesModels;
+
+    public function __construct(public Model $model)
+    {}
+}

src/Events/RoleDetached.php

@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Spatie\Permission\Events;
+
+use Illuminate\Broadcasting\InteractsWithSockets;
+use Illuminate\Database\Eloquent\Model;
+use Illuminate\Foundation\Bus\Dispatchable;
+use Illuminate\Queue\SerializesModels;
+
+class RoleDetached
+{
+    use Dispatchable;
+    use InteractsWithSockets;
+    use SerializesModels;
+
+    public function __construct(public Model $model)
+    {}
+}

src/Traits/HasPermissions.php

@@ -9,6 +9,8 @@
 use Spatie\Permission\Contracts\Permission;
 use Spatie\Permission\Contracts\Role;
 use Spatie\Permission\Contracts\Wildcard;
+use Spatie\Permission\Events\PermissionAttached;
+use Spatie\Permission\Events\PermissionDetached;
 use Spatie\Permission\Exceptions\GuardDoesNotMatch;
 use Spatie\Permission\Exceptions\PermissionDoesNotExist;
 use Spatie\Permission\Exceptions\WildcardPermissionInvalidArgument;
@@ -422,6 +424,8 @@ function ($object) use ($permissions, $model, $teamPivot, &$saved) {
             $this->forgetCachedPermissions();
         }
 
+        event(new PermissionAttached($this->getModel()));
+
         $this->forgetWildcardPermissionIndex();
 
         return $this;
@@ -465,6 +469,8 @@ public function revokePermissionTo($permission)
             $this->forgetCachedPermissions();
         }
 
+        event(new PermissionDetached($this->getModel()));
+
         $this->forgetWildcardPermissionIndex();
 
         $this->unsetRelation('permissions');

src/Traits/HasRoles.php

@@ -8,6 +8,8 @@
 use Illuminate\Support\Collection;
 use Spatie\Permission\Contracts\Permission;
 use Spatie\Permission\Contracts\Role;
+use Spatie\Permission\Events\RoleAttached;
+use Spatie\Permission\Events\RoleDetached;
 use Spatie\Permission\PermissionRegistrar;
 
 trait HasRoles
@@ -179,6 +181,8 @@ function ($object) use ($roles, $model, $teamPivot, &$saved) {
             $this->forgetCachedPermissions();
         }
 
+        event(new RoleAttached($this->getModel()));
+
         return $this;
     }
 
@@ -197,6 +201,8 @@ public function removeRole($role)
             $this->forgetCachedPermissions();
         }
 
+        event(new RoleDetached($this->getModel()));
+
         return $this;
     }
 

tests/HasPermissionsTest.php

@@ -3,8 +3,11 @@
 namespace Spatie\Permission\Tests;
 
 use DB;
+use Illuminate\Support\Facades\Event;
 use Spatie\Permission\Contracts\Permission;
 use Spatie\Permission\Contracts\Role;
+use Spatie\Permission\Events\PermissionAttached;
+use Spatie\Permission\Events\RoleAttached;
 use Spatie\Permission\Exceptions\GuardDoesNotMatch;
 use Spatie\Permission\Exceptions\PermissionDoesNotExist;
 use Spatie\Permission\Tests\TestModels\SoftDeletingUser;
@@ -765,4 +768,16 @@ public function it_can_reject_permission_based_on_logged_in_user_guard()
             'status' => false,
         ]);
     }
+
+    /** @test */
+    public function it_fires_an_event_when_a_permission_is_added()
+    {
+        Event::fake();
+
+        $this->testUser->givePermissionTo('edit-news');
+
+        Event::assertDispatched(PermissionAttached::class, function ($event) {
+            return $event->model instanceof User && $event->model->hasPermissionTo('edit-news');
+        });
+    }
 }

tests/HasRolesTest.php

@@ -3,7 +3,9 @@
 namespace Spatie\Permission\Tests;
 
 use Illuminate\Support\Facades\DB;
+use Illuminate\Support\Facades\Event;
 use Spatie\Permission\Contracts\Role;
+use Spatie\Permission\Events\RoleAttached;
 use Spatie\Permission\Exceptions\GuardDoesNotMatch;
 use Spatie\Permission\Exceptions\RoleDoesNotExist;
 use Spatie\Permission\Tests\TestModels\Admin;
@@ -856,4 +858,16 @@ public function it_does_not_detach_roles_when_user_soft_deleting()
 
         $this->assertTrue($user->hasRole('testRole'));
     }
+
+    /** @test */
+    public function it_fires_an_event_when_a_role_is_added()
+    {
+        Event::fake();
+
+        $this->testUser->assignRole('testRole');
+
+        Event::assertDispatched(RoleAttached::class, function ($event) {
+            return $event->model instanceof User && $event->model->hasRole('testRole');
+        });
+    }
 }