Merge pull request #76 from mostafamaklad/v1.9

V1.9

Author: mostafamaklad

.scrutinizer.yml

@@ -9,9 +9,15 @@ build:
                 coverage:
                     file: 'coverage'
                     format: 'clover'
+    nodes:
+        analysis:
+            tests:
+                override:
+                - php-scrutinizer-run
 
 filter:
-    excluded_paths: [tests/*]
+    excluded_paths:
+        - "tests/"
 
 checks:
     php:

src/Commands/CreatePermission.php

@@ -19,7 +19,7 @@ class CreatePermission extends Command
 
     public function handle()
     {
-        $permissionClass = \app(Permission::class);
+        $permissionClass = \app(\config('permission.models.permission'));
 
         $permission = $permissionClass::create([
             'name'       => $this->argument('name'),

src/Commands/CreateRole.php

@@ -20,7 +20,7 @@ class CreateRole extends Command
 
     public function handle()
     {
-        $roleClass       = \app(Role::class);
+        $roleClass       = \app(\config('permission.models.role'));
 
         $name        = $this->argument('name');
         $guard       = $this->argument('guard');

src/Guard.php

@@ -21,18 +21,22 @@ class Guard
      */
     public function getNames($model) : Collection
     {
+        $guardName = null;
+        $class = null;
+
         if (\is_object($model)) {
             $guardName = $model->guard_name ?? null;
         }
 
-        if (! isset($guardName)) {
+        if ($guardName === null) {
             $class = \is_object($model) ? \get_class($model) : $model;
             $guardName = (new \ReflectionClass($class))->getDefaultProperties()['guard_name'] ?? null;
         }
 
         if ($guardName) {
             return collect($guardName);
         }
+
         return collect(config('auth.guards'))
             ->map(function ($guard) {
                 if (! isset($guard['provider'])) {

src/Helpers.php

@@ -119,6 +119,6 @@ public function getUserNotLoggedINMessage(): string
 
     public function isNotLumen(): bool
     {
-        return ! stripos(app()->version(), 'lumen');
+        return ! (stripos(app()->version(), 'lumen') !== false);
     }
 }

src/Models/Permission.php

@@ -113,7 +113,7 @@ public function roles(): BelongsToMany
      * A permission belongs to some users of the model associated with its guard.
      * @return BelongsToMany
      */
-    public function users(): BelongsToMany
+    public function users()
     {
         return $this->belongsToMany($this->helpers->getModelForGuard($this->attributes['guard_name']));
     }

src/PermissionRegistrar.php

@@ -48,7 +48,7 @@ public function forgetCachedPermissions()
     public function getPermissions(): Collection
     {
         return $this->cache->remember($this->cacheKey, \config('permission.cache_expiration_time'), function () {
-            return \app(Permission::class)->with('roles')->get();
+            return \app(config('permission.models.permission'))->with('roles')->get();
         });
     }
 }

src/Traits/HasPermissions.php

@@ -11,6 +11,7 @@
 use Maklad\Permission\Guard;
 use Maklad\Permission\Helpers;
 use Maklad\Permission\Models\Role;
+use Maklad\Permission\Models\PermissionRole;
 use Maklad\Permission\PermissionRegistrar;
 
 /**
@@ -55,7 +56,7 @@ public function users(): BelongsToMany
      * @return $this
      * @throws GuardDoesNotMatch
      */
-    public function givePermissionTo(...$permissions)
+    public function givePermissionTo(...$permissions): self
     {
         $permissions = collect($permissions)
             ->flatten()
@@ -82,7 +83,7 @@ public function givePermissionTo(...$permissions)
      * @return $this
      * @throws GuardDoesNotMatch
      */
-    public function syncPermissions(...$permissions)
+    public function syncPermissions(...$permissions): self
     {
         $this->permissions()->sync([]);
 
@@ -97,7 +98,7 @@ public function syncPermissions(...$permissions)
      * @return $this
      * @throws \Maklad\Permission\Exceptions\GuardDoesNotMatch
      */
-    public function revokePermissionTo(...$permissions)
+    public function revokePermissionTo(...$permissions): self
     {
         collect($permissions)
             ->flatten()
@@ -122,7 +123,7 @@ public function revokePermissionTo(...$permissions)
     protected function getStoredPermission($permission): Permission
     {
         if (\is_string($permission)) {
-            return \app(Permission::class)->findByName($permission, $this->getDefaultGuardName());
+            return \app(config('permission.models.permission'))->findByName($permission, $this->getDefaultGuardName());
         }
 
         return $permission;
@@ -214,6 +215,7 @@ public function getPermissionsViaRoles(): Collection
             ->roles->flatMap(function (Role $role) {
                 return $role->permissions;
             })->sort()->values();
+        //return \app(\config('permission.models.permission'))->whereIn('role_id', $this->role_ids)->get();
     }
 
     /**
@@ -239,7 +241,7 @@ public function getAllPermissions(): Collection
     public function hasPermissionTo($permission, $guardName = null): bool
     {
         if (\is_string($permission)) {
-            $permission = \app(Permission::class)->findByName(
+            $permission = \app(\config('permission.models.permission'))->findByName(
                 $permission,
                 $guardName ?? $this->getDefaultGuardName()
             );
@@ -294,7 +296,10 @@ protected function hasPermissionViaRole(Permission $permission): bool
     public function hasDirectPermission($permission): bool
     {
         if (\is_string($permission)) {
-            $permission = \app(Permission::class)->findByName($permission, $this->getDefaultGuardName());
+            $permission = \app(
+                \config('permission.models.permission')
+            )
+                    ->findByName($permission, $this->getDefaultGuardName());
         }
 
         return $this->permissions->contains('id', $permission->id);

src/Traits/HasRoles.php

@@ -5,7 +5,6 @@
 use Illuminate\Support\Collection;
 use Jenssegers\Mongodb\Eloquent\Builder;
 use Jenssegers\Mongodb\Eloquent\Model;
-use Jenssegers\Mongodb\Relations\BelongsToMany;
 use Maklad\Permission\Contracts\RoleInterface as Role;
 use ReflectionException;
 
@@ -31,9 +30,9 @@ public static function bootHasRoles()
     /**
      * A model may have multiple roles.
      */
-    public function roles(): BelongsToMany
+    public function roles()
     {
-        return $this->belongsToMany(\config('permission.models.role'))->withTimestamps();
+        return $this->belongsToMany(config('permission.models.role'));
     }
 
     /**
@@ -105,7 +104,7 @@ public function removeRole(...$roles)
      *
      * @param array ...$roles
      *
-     * @return $this
+     * @return array|Role|string
      */
     public function syncRoles(...$roles)
     {
@@ -185,7 +184,7 @@ public function hasAllRoles($roles): bool
     protected function getStoredRole($role): Role
     {
         if (\is_string($role)) {
-            return \app(Role::class)->findByName($role, $this->getDefaultGuardName());
+            return \app(\config('permission.models.role'))->findByName($role, $this->getDefaultGuardName());
         }
 
         return $role;

src/Traits/RefreshesPermissionCache.php

@@ -13,11 +13,11 @@ trait RefreshesPermissionCache
     public static function bootRefreshesPermissionCache()
     {
         static::saved(function () {
-            \app(PermissionRegistrar::class)->forgetCachedPermissions();
+            \app(\config('permission.models.permission'))->forgetCachedPermissions();
         });
 
         static::deleted(function () {
-            \app(PermissionRegistrar::class)->forgetCachedPermissions();
+            \app(\config('permission.models.permission'))->forgetCachedPermissions();
         });
     }
 }