Apply fixes from StyleCI

StyleCIBot · StyleCIBot · commit 4dbc3c49add9 · 2024-04-06T18:34:39.000Z
diff --git a/src/Relations/BelongsTo.php b/src/Relations/BelongsTo.php
@@ -26,9 +26,9 @@ public function beforeMutating(Model $model, Relation $relation, array $mutation
             case 'attach':
                 $this->resource()->authorizeToAttach($model, $toPerformActionModel);
                 break;
-            case 'detach';
-                $this->resource()->authorizeToDetach($model, $toPerformActionModel);
-                break;
+            case 'detach':
+            $this->resource()->authorizeToDetach($model, $toPerformActionModel);
+            break;
         }
 
         $model
diff --git a/src/Relations/BelongsToMany.php b/src/Relations/BelongsToMany.php
@@ -88,7 +88,6 @@ public function afterMutating(Model $model, Relation $relation, array $mutationR
                 foreach ($results['detached'] as $detached) {
                     $this->resource()->authorizeToDetach($model, $relation->resource()::$model::find($detached));
                 }
-
             } elseif ($mutationRelation['operation'] === 'sync') {
                 $results = $model
                     ->{$relation->relation}()
diff --git a/src/Relations/HasMany.php b/src/Relations/HasMany.php
@@ -34,9 +34,9 @@ public function afterMutating(Model $model, Relation $relation, array $mutationR
                 case 'attach':
                     $this->resource()->authorizeToAttach($model, $toPerformActionModel);
                     break;
-                case 'detach';
-                    $this->resource()->authorizeToDetach($model, $toPerformActionModel);
-                    break;
+                case 'detach':
+                $this->resource()->authorizeToDetach($model, $toPerformActionModel);
+                break;
             }
         }
     }
diff --git a/src/Relations/HasOne.php b/src/Relations/HasOne.php
@@ -30,9 +30,9 @@ public function afterMutating(Model $model, Relation $relation, array $mutationR
             case 'attach':
                 $this->resource()->authorizeToAttach($model, $toPerformActionModel);
                 break;
-            case 'detach';
-                $this->resource()->authorizeToDetach($model, $toPerformActionModel);
-                break;
+            case 'detach':
+            $this->resource()->authorizeToDetach($model, $toPerformActionModel);
+            break;
         }
     }
 }
diff --git a/src/Relations/HasOneOfMany.php b/src/Relations/HasOneOfMany.php
@@ -30,9 +30,9 @@ public function afterMutating(Model $model, Relation $relation, array $mutationR
             case 'attach':
                 $this->resource()->authorizeToAttach($model, $toPerformActionModel);
                 break;
-            case 'detach';
-                $this->resource()->authorizeToDetach($model, $toPerformActionModel);
-                break;
+            case 'detach':
+            $this->resource()->authorizeToDetach($model, $toPerformActionModel);
+            break;
         }
     }
 }
diff --git a/src/Relations/MorphMany.php b/src/Relations/MorphMany.php
@@ -35,9 +35,9 @@ public function afterMutating(Model $model, Relation $relation, array $mutationR
                 case 'attach':
                     $this->resource()->authorizeToAttach($model, $toPerformActionModel);
                     break;
-                case 'detach';
-                    $this->resource()->authorizeToDetach($model, $toPerformActionModel);
-                    break;
+                case 'detach':
+                $this->resource()->authorizeToDetach($model, $toPerformActionModel);
+                break;
             }
         }
     }
diff --git a/src/Relations/MorphOne.php b/src/Relations/MorphOne.php
@@ -31,9 +31,9 @@ public function afterMutating(Model $model, Relation $relation, array $mutationR
             case 'attach':
                 $this->resource()->authorizeToAttach($model, $toPerformActionModel);
                 break;
-            case 'detach';
-                $this->resource()->authorizeToDetach($model, $toPerformActionModel);
-                break;
+            case 'detach':
+            $this->resource()->authorizeToDetach($model, $toPerformActionModel);
+            break;
         }
     }
 }
diff --git a/src/Relations/MorphOneOfMany.php b/src/Relations/MorphOneOfMany.php
@@ -31,9 +31,9 @@ public function afterMutating(Model $model, Relation $relation, array $mutationR
             case 'attach':
                 $this->resource()->authorizeToAttach($model, $toPerformActionModel);
                 break;
-            case 'detach';
-                $this->resource()->authorizeToDetach($model, $toPerformActionModel);
-                break;
+            case 'detach':
+            $this->resource()->authorizeToDetach($model, $toPerformActionModel);
+            break;
         }
     }
 }
diff --git a/src/Relations/MorphTo.php b/src/Relations/MorphTo.php
@@ -26,9 +26,9 @@ public function beforeMutating(Model $model, Relation $relation, array $mutation
             case 'attach':
                 $this->resource()->authorizeToAttach($model, $toPerformActionModel);
                 break;
-            case 'detach';
-                $this->resource()->authorizeToDetach($model, $toPerformActionModel);
-                break;
+            case 'detach':
+            $this->resource()->authorizeToDetach($model, $toPerformActionModel);
+            break;
         }
 
         $model
diff --git a/src/Relations/MorphToMany.php b/src/Relations/MorphToMany.php
@@ -88,7 +88,6 @@ public function afterMutating(Model $model, Relation $relation, array $mutationR
                 foreach ($results['detached'] as $detached) {
                     $this->resource()->authorizeToDetach($model, $relation->resource()::$model::find($detached));
                 }
-
             } elseif ($mutationRelation['operation'] === 'sync') {
                 $results = $model
                     ->{$relation->relation}()
diff --git a/tests/Feature/Controllers/MutateNotAuthorizedMorphsRelationsOperationsTest.php b/tests/Feature/Controllers/MutateNotAuthorizedMorphsRelationsOperationsTest.php
@@ -3,35 +3,22 @@
 namespace Controllers;
 
 use Illuminate\Support\Facades\Gate;
-use Lomkit\Rest\Concerns\Authorizable;
-use Lomkit\Rest\Http\Resource;
-use Lomkit\Rest\Relations\MorphToMany;
 use Lomkit\Rest\Tests\Feature\TestCase;
-use Lomkit\Rest\Tests\Support\Database\Factories\MorphOneOfManyRelationFactory;
-use Lomkit\Rest\Tests\Support\Database\Factories\MorphToManyRelationFactory;
-use Lomkit\Rest\Tests\Support\Database\Factories\MorphToRelationFactory;
-use Lomkit\Rest\Tests\Support\Database\Factories\HasManyRelationFactory;
-use Lomkit\Rest\Tests\Support\Database\Factories\HasOneOfManyRelationFactory;
-use Lomkit\Rest\Tests\Support\Database\Factories\HasOneRelationFactory;
 use Lomkit\Rest\Tests\Support\Database\Factories\ModelFactory;
 use Lomkit\Rest\Tests\Support\Database\Factories\MorphManyRelationFactory;
+use Lomkit\Rest\Tests\Support\Database\Factories\MorphOneOfManyRelationFactory;
 use Lomkit\Rest\Tests\Support\Database\Factories\MorphOneRelationFactory;
+use Lomkit\Rest\Tests\Support\Database\Factories\MorphToManyRelationFactory;
+use Lomkit\Rest\Tests\Support\Database\Factories\MorphToRelationFactory;
 use Lomkit\Rest\Tests\Support\Database\Factories\NoRelationshipAuthorizationModelFactory;
-use Lomkit\Rest\Tests\Support\Models\MorphToManyRelation;
-use Lomkit\Rest\Tests\Support\Models\MorphToRelation;
-use Lomkit\Rest\Tests\Support\Models\HasManyRelation;
-use Lomkit\Rest\Tests\Support\Models\HasOneOfManyRelation;
-use Lomkit\Rest\Tests\Support\Models\HasOneRelation;
-use Lomkit\Rest\Tests\Support\Models\Model;
 use Lomkit\Rest\Tests\Support\Models\MorphManyRelation;
 use Lomkit\Rest\Tests\Support\Models\MorphOneOfManyRelation;
 use Lomkit\Rest\Tests\Support\Models\MorphOneRelation;
+use Lomkit\Rest\Tests\Support\Models\MorphToManyRelation;
+use Lomkit\Rest\Tests\Support\Models\MorphToRelation;
 use Lomkit\Rest\Tests\Support\Models\NoRelationshipAuthorizedModel;
 use Lomkit\Rest\Tests\Support\Policies\GreenPolicy;
 use Lomkit\Rest\Tests\Support\Policies\NoRelationshipAuthorizationModelPolicy;
-use Lomkit\Rest\Tests\Support\Rest\Resources\NoRelationshipAuthorizationModelResource;
-use Mockery;
-use Mockery\MockInterface;
 
 class MutateNotAuthorizedMorphsRelationsOperationsTest extends TestCase
 {
@@ -452,15 +439,15 @@ public function test_creating_a_resource_with_updating_not_authorized_morph_to_r
             [
                 'mutate' => [
                     [
-                        'operation' => 'create',
+                        'operation'  => 'create',
                         'attributes' => [
-                            'name' => $modelToCreate->name,
+                            'name'   => $modelToCreate->name,
                             'number' => $modelToCreate->number,
                         ],
                         'relations' => [
                             'morphToRelation' => [
-                                'operation' => 'update',
-                                'key' => $morphToRelationToUpdate->getKey(),
+                                'operation'  => 'update',
+                                'key'        => $morphToRelationToUpdate->getKey(),
                                 'attributes' => ['number' => 5001], // 5001 because with factory it can't exceed 5000
                             ],
                         ],
diff --git a/tests/Feature/Controllers/MutateNotAuthorizedRelationsOperationsTest.php b/tests/Feature/Controllers/MutateNotAuthorizedRelationsOperationsTest.php
@@ -3,9 +3,6 @@
 namespace Controllers;
 
 use Illuminate\Support\Facades\Gate;
-use Lomkit\Rest\Concerns\Authorizable;
-use Lomkit\Rest\Http\Resource;
-use Lomkit\Rest\Relations\BelongsToMany;
 use Lomkit\Rest\Tests\Feature\TestCase;
 use Lomkit\Rest\Tests\Support\Database\Factories\BelongsToManyRelationFactory;
 use Lomkit\Rest\Tests\Support\Database\Factories\BelongsToRelationFactory;
@@ -19,13 +16,9 @@
 use Lomkit\Rest\Tests\Support\Models\HasManyRelation;
 use Lomkit\Rest\Tests\Support\Models\HasOneOfManyRelation;
 use Lomkit\Rest\Tests\Support\Models\HasOneRelation;
-use Lomkit\Rest\Tests\Support\Models\Model;
 use Lomkit\Rest\Tests\Support\Models\NoRelationshipAuthorizedModel;
 use Lomkit\Rest\Tests\Support\Policies\GreenPolicy;
 use Lomkit\Rest\Tests\Support\Policies\NoRelationshipAuthorizationModelPolicy;
-use Lomkit\Rest\Tests\Support\Rest\Resources\NoRelationshipAuthorizationModelResource;
-use Mockery;
-use Mockery\MockInterface;
 
 class MutateNotAuthorizedRelationsOperationsTest extends TestCase
 {
@@ -446,15 +439,15 @@ public function test_creating_a_resource_with_updating_not_authorized_belongs_to
             [
                 'mutate' => [
                     [
-                        'operation' => 'create',
+                        'operation'  => 'create',
                         'attributes' => [
-                            'name' => $modelToCreate->name,
+                            'name'   => $modelToCreate->name,
                             'number' => $modelToCreate->number,
                         ],
                         'relations' => [
                             'belongsToRelation' => [
-                                'operation' => 'update',
-                                'key' => $belongsToRelationToUpdate->getKey(),
+                                'operation'  => 'update',
+                                'key'        => $belongsToRelationToUpdate->getKey(),
                                 'attributes' => ['number' => 5001], // 5001 because with factory it can't exceed 5000
                             ],
                         ],
diff --git a/tests/Support/Policies/NoRelationshipAuthorizationModelPolicy.php b/tests/Support/Policies/NoRelationshipAuthorizationModelPolicy.php
@@ -60,9 +60,8 @@ public function detachHasManyRelation($user, Model $model, Model $toDetachModel)
     }
 
     /**
-     * MORPHS
+     * MORPHS.
      */
-
     public function attachMorphToRelation($user, Model $model, Model $toAttachModel)
     {
         return false;

Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,6 @@ public function afterMutating(Model $model, Relation $relation, array $mutationR`
`88`	`88`	`foreach ($results['detached'] as $detached) {`
`89`	`89`	`$this->resource()->authorizeToDetach($model, $relation->resource()::$model::find($detached));`
`90`	`90`	`}`
`91`		`-`
`92`	`91`	`} elseif ($mutationRelation['operation'] === 'sync') {`
`93`	`92`	`$results = $model`
`94`	`93`	`->{$relation->relation}()`
Original file line number	Diff line number	Diff line change
`@@ -34,9 +34,9 @@ public function afterMutating(Model $model, Relation $relation, array $mutationR`
`34`	`34`	`case 'attach':`
`35`	`35`	`$this->resource()->authorizeToAttach($model, $toPerformActionModel);`
`36`	`36`	`break;`
`37`		`- case 'detach';`
`38`		`- $this->resource()->authorizeToDetach($model, $toPerformActionModel);`
`39`		`- break;`
	`37`	`+ case 'detach':`
	`38`	`+ $this->resource()->authorizeToDetach($model, $toPerformActionModel);`
	`39`	`+ break;`
`40`	`40`	`}`
`41`	`41`	`}`
`42`	`42`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,9 +30,9 @@ public function afterMutating(Model $model, Relation $relation, array $mutationR`
`30`	`30`	`case 'attach':`
`31`	`31`	`$this->resource()->authorizeToAttach($model, $toPerformActionModel);`
`32`	`32`	`break;`
`33`		`- case 'detach';`
`34`		`- $this->resource()->authorizeToDetach($model, $toPerformActionModel);`
`35`		`- break;`
	`33`	`+ case 'detach':`
	`34`	`+ $this->resource()->authorizeToDetach($model, $toPerformActionModel);`
	`35`	`+ break;`
`36`	`36`	`}`
`37`	`37`	`}`
`38`	`38`	`}`
Original file line number	Diff line number	Diff line change
`@@ -35,9 +35,9 @@ public function afterMutating(Model $model, Relation $relation, array $mutationR`
`35`	`35`	`case 'attach':`
`36`	`36`	`$this->resource()->authorizeToAttach($model, $toPerformActionModel);`
`37`	`37`	`break;`
`38`		`- case 'detach';`
`39`		`- $this->resource()->authorizeToDetach($model, $toPerformActionModel);`
`40`		`- break;`
	`38`	`+ case 'detach':`
	`39`	`+ $this->resource()->authorizeToDetach($model, $toPerformActionModel);`
	`40`	`+ break;`
`41`	`41`	`}`
`42`	`42`	`}`
`43`	`43`	`}`
Original file line number	Diff line number	Diff line change
`@@ -31,9 +31,9 @@ public function afterMutating(Model $model, Relation $relation, array $mutationR`
`31`	`31`	`case 'attach':`
`32`	`32`	`$this->resource()->authorizeToAttach($model, $toPerformActionModel);`
`33`	`33`	`break;`
`34`		`- case 'detach';`
`35`		`- $this->resource()->authorizeToDetach($model, $toPerformActionModel);`
`36`		`- break;`
	`34`	`+ case 'detach':`
	`35`	`+ $this->resource()->authorizeToDetach($model, $toPerformActionModel);`
	`36`	`+ break;`
`37`	`37`	`}`
`38`	`38`	`}`
`39`	`39`	`}`