Merge pull request #7 from flixtechs-labs/facade

feat: added the ability get permission names for a given model

Author: SlimGee

composer.json

@@ -18,21 +18,21 @@
     "require": {
         "php": "^8.1",
         "illuminate/contracts": "^9.0",
-        "spatie/laravel-package-tools": "^1.9.2",
-        "spatie/laravel-permission": "^5.5",
-        "symfony/filesystem": "^6.1"
+        "spatie/laravel-package-tools": "^1.12.1",
+        "spatie/laravel-permission": "^5.5.5",
+        "symfony/filesystem": "^6.1.3"
     },
     "require-dev": {
-        "laravel/pint": "^1.0",
-        "nunomaduro/collision": "^6.2",
-        "nunomaduro/larastan": "^2.0.1",
-        "orchestra/testbench": "^7.6",
-        "pestphp/pest": "^1.21",
-        "pestphp/pest-plugin-laravel": "^1.1",
+        "laravel/pint": "^1.1.1",
+        "nunomaduro/collision": "^6.2.1",
+        "nunomaduro/larastan": "^2.1.12",
+        "orchestra/testbench": "^7.7",
+        "pestphp/pest": "^1.21.3",
+        "pestphp/pest-plugin-laravel": "^1.2",
         "phpstan/extension-installer": "^1.1",
         "phpstan/phpstan-deprecation-rules": "^1.0",
-        "phpstan/phpstan-phpunit": "^1.0",
-        "phpunit/phpunit": "^9.5",
+        "phpstan/phpstan-phpunit": "^1.1.1",
+        "phpunit/phpunit": "^9.5.23",
         "roave/security-advisories": "dev-latest"
     },
     "autoload": {
@@ -69,7 +69,7 @@
                 "FlixtechsLabs\\LaravelAuthorizer\\LaravelAuthorizerServiceProvider"
             ],
             "aliases": {
-                "LaravelAuthorizer": "FlixtechsLabs\\LaravelAuthorizer\\Facades\\LaravelAuthorizer"
+                "Authorizer": "FlixtechsLabs\\LaravelAuthorizer\\Facades\\Authorizer"
             }
         }
     },

src/Commands/GeneratePermissionsCommand.php

@@ -3,8 +3,8 @@
 namespace FlixtechsLabs\LaravelAuthorizer\Commands;
 
 use FlixtechsLabs\LaravelAuthorizer\Commands\Traits\LocatesModels;
+use FlixtechsLabs\LaravelAuthorizer\Facades\Authorizer;
 use Illuminate\Console\Command;
-use Illuminate\Support\Str;
 use Spatie\Permission\Models\Permission;
 
 class GeneratePermissionsCommand extends Command
@@ -67,33 +67,12 @@ protected function generatePermissionsForAllModels(): void
      */
     public function generatePermissions(string $model): void
     {
-        $permissions = config('authorizer.permissions');
+        $permissions = Authorizer::getPermissionsFor($model);
 
         collect($permissions)->each(
-            fn (string $permission) => $this->generatePermission(
-                $model,
+            fn (string $permission) => Permission::findOrCreate(
                 $permission
             )
         );
     }
-
-    /**
-     * Generate a permission for a given model.
-     *
-     * @param  string  $model
-     * @param  string  $permission
-     * @return mixed
-     */
-    public function generatePermission(string $model, string $permission): mixed
-    {
-        if (Str::contains($permission, 'all')) {
-            return Permission::findOrCreate(
-                $permission.' '.Str::snake(Str::plural(Str::lower($model)))
-            );
-        }
-
-        return Permission::findOrCreate(
-            $permission.' '.Str::snake(Str::lower($model))
-        );
-    }
 }

src/Facades/LaravelAuthorizer.php renamed to src/Facades/Authorizer.php

@@ -7,7 +7,7 @@
 /**
  * @see \FlixtechsLabs\LaravelAuthorizer\LaravelAuthorizer
  */
-class LaravelAuthorizer extends Facade
+class Authorizer extends Facade
 {
     protected static function getFacadeAccessor()
     {

src/LaravelAuthorizer.php

@@ -2,6 +2,14 @@
 
 namespace FlixtechsLabs\LaravelAuthorizer;
 
+use Illuminate\Support\Str;
+
 class LaravelAuthorizer
 {
+    public function getPermissionsFor(string $model): array
+    {
+        return array_map(static function (string $permission) use ($model) {
+            return $permission.' '.(Str::contains($permission, 'all') ? Str::of($model)->snake()->plural()->lower() : Str::of($model)->snake()->lower());
+        }, config('authorizer.permissions'));
+    }
 }