Apply fixes from StyleCI

Author: StyleCIBot

src/Controls/Control.php

@@ -24,7 +24,7 @@ class Control
     public static $namespace = 'App\\Access\\Controls\\';
 
     /**
-     * Get the perimeters for the current control
+     * Get the perimeters for the current control.
      *
      * @return array
      */
@@ -36,21 +36,22 @@ protected function perimeters(): array
     /**
      * Specify the callback that should be invoked to guess control names.
      *
-     * @param  callable(class-string<\Illuminate\Database\Eloquent\Model>): class-string<\Lomkit\Access\Controls\Control>  $callback
+     * @param callable(class-string<\Illuminate\Database\Eloquent\Model>): class-string<\Lomkit\Access\Controls\Control> $callback
+     *
      * @return void
      */
     public static function guessControlNamesUsing(callable $callback)
     {
         static::$controlNameResolver = $callback;
     }
 
-
     /**
      * Get a new control instance for the given model name.
      *
      * @template TClass of \Illuminate\Database\Eloquent\Model
      *
-     * @param  class-string<TClass>  $modelName
+     * @param class-string<TClass> $modelName
+     *
      * @return \Lomkit\Access\Controls\Control<TClass>
      */
     public static function controlForModel(string $modelName)
@@ -69,15 +70,16 @@ public static function controlForModel(string $modelName)
      */
     public static function new()
     {
-        return (new static);
+        return new static();
     }
 
     /**
      * Get the control name for the given model name.
      *
      * @template TClass of \Illuminate\Database\Eloquent\Model
      *
-     * @param  class-string<TClass>  $modelName
+     * @param class-string<TClass> $modelName
+     *
      * @return class-string<\Lomkit\Access\Controls\Control<TClass>>
      */
     public static function resolveControlName(string $modelName)
@@ -110,4 +112,4 @@ protected static function appNamespace()
             return 'App\\';
         }
     }
-}
+}

src/Controls/HasControl.php

@@ -3,8 +3,6 @@
 namespace Lomkit\Access\Controls;
 
 use Illuminate\Database\Eloquent\Factories\Factory;
-use Illuminate\Support\Facades\App;
-use Illuminate\Support\Str;
 
 trait HasControl
 {
@@ -15,7 +13,7 @@ trait HasControl
      */
     public static function bootHasControl()
     {
-        static::addGlobalScope(new HasControlScope);
+        static::addGlobalScope(new HasControlScope());
     }
 
     /**
@@ -39,4 +37,4 @@ protected static function newControl(): Control|null
     {
         return static::$control::new() ?? null;
     }
-}
+}

src/Perimeters/Perimeter.php

@@ -21,6 +21,7 @@ public function query(Closure $queryCallback): self
     {
         // @TODO: ok mais pas possible de le déclarer globalement alors, seems ok for me
         $this->queryCallback = $queryCallback;
+
         return $this;
     }
 
@@ -31,6 +32,6 @@ public function query(Closure $queryCallback): self
      */
     public static function new()
     {
-        return (new static);
+        return new static();
     }
-}
+}

tests/Feature/PerimetersTest.php

@@ -2,30 +2,24 @@
 
 namespace Lomkit\Access\Tests\Feature;
 
-use Illuminate\Database\Eloquent\Factories\Sequence;
 use Illuminate\Support\Facades\Auth;
-use Illuminate\Support\Facades\Cache;
-use Lomkit\Access\Exceptions\QueryNotImplemented;
 use Lomkit\Access\Tests\Support\Access\Perimeters\ClientPerimeter;
-use Lomkit\Access\Tests\Support\Database\Factories\ModelFactory;
 use Lomkit\Access\Tests\Support\Models\Model;
-use Lomkit\Access\Tests\Support\Models\NotImplementedQueryModel;
-use Lomkit\Access\Tests\Support\Models\User;
 
 class PerimetersTest extends TestCase
 {
     public function test_should_client_perimeter(): void
     {
         Auth::user()->update(['should_client' => true]);
 
-        $this->assertTrue((new ClientPerimeter())->should(Auth::user(), 'create', new Model));
+        $this->assertTrue((new ClientPerimeter())->should(Auth::user(), 'create', new Model()));
     }
 
     public function test_should_not_client_perimeter(): void
     {
         Auth::user()->update(['should_client' => false]);
 
-        $this->assertFalse((new ClientPerimeter())->should(Auth::user(), 'create', new Model));
+        $this->assertFalse((new ClientPerimeter())->should(Auth::user(), 'create', new Model()));
     }
 
     // @TODO: other perimeters

tests/Support/Access/Controls/ModelControl.php

@@ -25,6 +25,6 @@ protected function perimeters(): array
                 ->query(function (Builder $query) {
                     $query->where('is_own', true);
                 }),
-            ];
+        ];
     }
-}
+}

tests/Support/Access/Perimeters/ClientPerimeter.php

@@ -12,4 +12,4 @@ public function should(Authenticatable $user, string $method, Model $model): boo
     {
         return $user->should_client;
     }
-}
+}

tests/Support/Access/Perimeters/GlobalPerimeter.php

@@ -12,4 +12,4 @@ public function should(Authenticatable $user, string $method, Model $model): boo
     {
         return $user->should_global;
     }
-}
+}

tests/Support/Access/Perimeters/OwnPerimeter.php

@@ -12,4 +12,4 @@ public function should(Authenticatable $user, string $method, Model $model): boo
     {
         return $user->should_own;
     }
-}
+}

tests/Support/Access/Perimeters/SharedPerimeter.php

@@ -12,4 +12,4 @@ public function should(Authenticatable $user, string $method, Model $model): boo
     {
         return $user->should_shared;
     }
-}
+}

tests/Support/Database/Factories/UserFactory.php

@@ -33,7 +33,7 @@ public function definition(): array
             'remember_token'    => Str::random(10),
             'should_shared'     => false,
             'should_global'     => false,
-            'should_own'     => false,
+            'should_own'        => false,
             'should_client'     => false,
         ];
     }