Merge pull request #216 from screencom/1.0

Add tags to the cache to easily purge ACL cache

Author: kodeine

src/Kodeine/Acl/Models/Eloquent/Role.php

@@ -48,7 +48,7 @@ public function users()
      */
     public function getPermissions()
     {
-        return \Cache::remember(
+        return \Cache::tags(config('acl.cacheTags', []))->remember(
             'acl.getPermissionsInheritedById_'.$this->id,
             config('acl.cacheMinutes'),
             function () {

src/Kodeine/Acl/Traits/HasPermission.php

@@ -34,8 +34,8 @@ public function permissions()
     public function getPermissions()
     {
         // user permissions overridden from role.
-        $permissions = \Cache::remember(
-            'acl.getPermissionsById_'.$this->id,
+        $permissions = \Cache::tags(config('acl.cacheTags', []))->remember(
+            'acl.getPermissionsById_' . $this->id,
             config('acl.cacheMinutes'),
             function () {
                 return $this->getPermissionsInherited();
@@ -48,10 +48,14 @@ function () {
         // true values.
         foreach ($this->roles as $role) {
             foreach ($role->getPermissions() as $slug => $array) {
-                if ( array_key_exists($slug, $permissions) ) {
+                if (array_key_exists($slug, $permissions)) {
                     foreach ($array as $clearance => $value) {
-                        if( !array_key_exists( $clearance, $permissions[$slug] ) ) {
+                        if (! array_key_exists($clearance, $permissions[$slug])) {
                             ! $value ?: $permissions[$slug][$clearance] = true;
+                        } else {
+                            if ($permissions[$slug][$clearance] != $value) {
+                                $permissions[$slug][$clearance] = $value;
+                            }
                         }
                     }
                 } else {
@@ -74,7 +78,7 @@ public function can($permission, $operator = null)
     {
         // user permissions including
         // all of user role permissions
-        $merge =  \Cache::remember(
+        $merge =  \Cache::tags(config('acl.cacheTags', []))->remember(
             'acl.getMergeById_'.$this->id,
             config('acl.cacheMinutes'),
             function () {
@@ -193,4 +197,4 @@ protected function parsePermissionId($permission)
 
         return (int) $permission;
     }
-}
+}

src/Kodeine/Acl/Traits/HasRole.php

@@ -26,7 +26,7 @@ public function roles()
     {
         $model = config('acl.role', 'Kodeine\Acl\Models\Eloquent\Role');
 
-        return $this->belongsToMany($model)->withTimestamps();
+        return $this->belongsToMany($model)->withTimestamps()->orderBy('roles.created_at');
     }
 
     /**
@@ -36,7 +36,7 @@ public function roles()
      */
     public function getRoles()
     {
-        $this_roles = \Cache::remember(
+        $this_roles = \Cache::tags(config('acl.cacheTags', []))->remember(
             'acl.getRolesById_'.$this->id,
             config('acl.cacheMinutes'),
             function () {

src/config/acl.php

@@ -27,4 +27,11 @@
      */
 
     'cacheMinutes' => 1,
+
+    /**
+     * Cache tags
+     *
+     * All cached entries are tagged to easily purge them on demand
+     */
+    'cacheTags' => ['acl']
 ];