Laravel-Backpack / CRUD

src/app/Library/CrudPanel/Traits/Update.php

Spacing +9 added lines, -9 removed lines

@@ -58,7 +58,7 @@ 
         }
 
         // always have a hidden input for the entry id
-        if (! array_key_exists('id', $fields)) {
+        if (!array_key_exists('id', $fields)) {
             $fields['id'] = [
                 'name' => $entry->getKeyName(),
                 'value' => $entry->getKey(),
@@ -83,12 +83,12 @@ 
         $fieldEntity = $field['entity'] ?? false;
         $fakeField = $field['fake'] ?? false;
 
-        if ($fieldEntity && ! $fakeField) {
+        if ($fieldEntity && !$fakeField) {
             return $this->getModelAttributeValueFromRelationship($model, $field);
         }
 
         if ($this->holdsMultipleInputs($field['name'])) {
-            $result = array_map(function ($item) use ($model) {
+            $result = array_map(function($item) use ($model) {
                 return $model->{$item};
             }, explode(',', $field['name']));
 
@@ -110,7 +110,7 @@ 
     {
         [$relatedModel, $relationMethod] = $this->getModelAndMethodFromEntity($model, $field);
 
-        if (! method_exists($relatedModel, $relationMethod) && ! $relatedModel->isRelation($relationMethod)) {
+        if (!method_exists($relatedModel, $relationMethod) && !$relatedModel->isRelation($relationMethod)) {
             return $relatedModel->{$relationMethod};
         }
 
@@ -129,7 +129,7 @@ 
                     $model = $this->setLocaleOnModel($model);
                     // when subfields are NOT set we don't need to get any more values
                     // we just return the plain models as we only need the ids
-                    if (! isset($field['subfields'])) {
+                    if (!isset($field['subfields'])) {
                         $result->push($model);
 
                         continue;
@@ -157,13 +157,13 @@ 
                 break;
             case 'HasOne':
             case 'MorphOne':
-                if (! method_exists($relatedModel, $relationMethod) && ! $relatedModel->isRelation($relationMethod)) {
+                if (!method_exists($relatedModel, $relationMethod) && !$relatedModel->isRelation($relationMethod)) {
                     return;
                 }
 
                 $model = $relatedModel->{$relationMethod};
 
-                if (! $model) {
+                if (!$model) {
                     return;
                 }
 
@@ -226,7 +226,7 @@ 
 
         $relationArray = explode('.', $relationEntity);
 
-        $relatedModel = array_reduce(array_splice($relationArray, 0, -1), function ($obj, $method) {
+        $relatedModel = array_reduce(array_splice($relationArray, 0, -1), function($obj, $method) {
             // if the string ends with `_id` we strip it out
             $method = Str::endsWith($method, '_id') ? Str::replaceLast('_id', '', $method) : $method;
 
@@ -253,7 +253,7 @@ 
             // if the subfield name does not contain a dot we just need to check
             // if it has subfields and return the result accordingly.
             foreach ((array) $subfield['name'] as $name) {
-                if (! Str::contains($name, '.')) {
+                if (!Str::contains($name, '.')) {
                     // when subfields are present, $relatedModel->{$name} returns a model instance
                     // otherwise returns the model attribute.
                     if ($relatedModel->{$name} && isset($subfield['subfields'])) {

src/app/Library/CrudPanel/Traits/Relationships.php

Spacing +12 added lines, -12 removed lines

@@ -65,7 +65,7 @@ 
         $model = $this->getRelationModel($entity, -1);
         $lastSegmentAfterDot = Str::of($field['entity'])->afterLast('.');
 
-        if (! method_exists($model, $lastSegmentAfterDot)) {
+        if (!method_exists($model, $lastSegmentAfterDot)) {
             return (string) Str::of($field['entity'])->beforeLast('.');
         }
 
@@ -86,7 +86,7 @@ 
 
         return collect($this->getCleanStateFields())
             ->whereIn('relation_type', $relation_types)
-            ->filter(function ($item) use ($nested) {
+            ->filter(function($item) use ($nested) {
                 if ($nested) {
                     return true;
                 }
@@ -130,7 +130,7 @@ 
      */
     private function getRelationFieldsWithoutRelationType($relations, $fields = [])
     {
-        if (! is_array($relations)) {
+        if (!is_array($relations)) {
             $relations = [$relations];
         }
 
@@ -139,8 +139,8 @@ 
         }
 
         foreach ($relations as $relation) {
-            $fields = array_filter($fields, function ($field) use ($relation) {
-                if (! isset($field['relation_type'])) {
+            $fields = array_filter($fields, function($field) use ($relation) {
+                if (!isset($field['relation_type'])) {
                     return false;
                 }
 
@@ -174,7 +174,7 @@ 
                     $fields = array_merge($field['subfields'], $fields);
                 }
             }
-            $fields = array_filter($fields, function ($field) {
+            $fields = array_filter($fields, function($field) {
                 return isset($field['relation_type']) && $field['relation_type'] === 'BelongsTo';
             });
         }
@@ -241,8 +241,8 @@ 
     {
         $all_relation_fields = $this->getRelationFields();
 
-        return Arr::where($all_relation_fields, function ($value, $key) {
-            return isset($value['pivot']) && ! $value['pivot'];
+        return Arr::where($all_relation_fields, function($value, $key) {
+            return isset($value['pivot']) && !$value['pivot'];
         });
     }
 
@@ -255,7 +255,7 @@ 
     {
         $all_relation_fields = $this->getRelationFields();
 
-        return Arr::where($all_relation_fields, function ($value, $key) {
+        return Arr::where($all_relation_fields, function($value, $key) {
             return isset($value['pivot']) && $value['pivot'];
         });
     }
@@ -325,7 +325,7 @@ 
      */
     private function modelMethodIsRelationship($model, $method)
     {
-        if (! method_exists($model, $method) && $model->isRelation($method)) {
+        if (!method_exists($model, $method) && $model->isRelation($method)) {
             return $method;
         }
 
@@ -337,7 +337,7 @@ 
         }
 
         // relationships are always public methods.
-        if (! $methodReflection->isPublic()) {
+        if (!$methodReflection->isPublic()) {
             return false;
         }
 
@@ -364,7 +364,7 @@ 
      */
     public function isAttributeInRelationString(array $field): bool
     {
-        if (! str_contains($field['entity'], '.')) {
+        if (!str_contains($field['entity'], '.')) {
             return false;
         }
 

tests/Unit/CrudPanel/CrudPanelCreateTest.php

Spacing +16 added lines, -16 removed lines

@@ -169,7 +169,7 @@ 
 
     public function testCreateWithOneToOneDynamicRelationship()
     {
-        User::resolveRelationUsing('dynamicRelation', function ($user) {
+        User::resolveRelationUsing('dynamicRelation', function($user) {
             return $user->hasOne(\Backpack\CRUD\Tests\config\Models\AccountDetails::class);
         });
         $this->crudPanel->setModel(User::class);
@@ -280,7 +280,7 @@ 
 
     public function testCreateWithOneToManyDynamicRelationship()
     {
-        Article::resolveRelationUsing('dynamicRelation', function ($article) {
+        Article::resolveRelationUsing('dynamicRelation', function($article) {
             return $article->belongsTo(\Backpack\CRUD\Tests\config\Models\User::class, 'user_id');
         });
 
@@ -343,7 +343,7 @@ 
 
     public function testCreateWithManyToManyDynamicRelationship()
     {
-        User::resolveRelationUsing('dynamicRelation', function ($user) {
+        User::resolveRelationUsing('dynamicRelation', function($user) {
             return $user->belongsToMany(\Backpack\CRUD\Tests\config\Models\Role::class, 'user_role');
         });
 
@@ -685,7 +685,7 @@ 
 
     public function testBelongsToManyWithPivotDataDynamicRelationship()
     {
-        User::resolveRelationUsing('dynamicRelation', function ($user) {
+        User::resolveRelationUsing('dynamicRelation', function($user) {
             return $user->belongsToMany(\Backpack\CRUD\Tests\config\Models\Article::class, 'articles_user')->withPivot(['notes', 'start_date', 'end_date']);
         });
 
@@ -794,7 +794,7 @@ 
 
     public function testBelongsToManyWithMultipleSameRelationIdAndPivotDataDynamicRelationship()
     {
-        User::resolveRelationUsing('dynamicRelation', function ($user) {
+        User::resolveRelationUsing('dynamicRelation', function($user) {
             return $user->belongsToMany(\Backpack\CRUD\Tests\config\Models\Article::class, 'articles_user')->withPivot(['notes', 'start_date', 'end_date', 'id'])->using('Backpack\CRUD\Tests\config\Models\SuperArticlePivot');
         });
 
@@ -1046,7 +1046,7 @@ 
 
     public function testCreateBelongsToFake()
     {
-        $belongsToField = [   // select_grouped
+        $belongsToField = [// select_grouped
             'label' => 'Select_grouped',
             'type' => 'select_grouped', //https://github.com/Laravel-Backpack/CRUD/issues/502
             'name' => 'bang_relation_field',
@@ -1182,7 +1182,7 @@ 
                 'nickname' => 'i_have_has_one',
                 'profile_picture' => 'ohh my picture 1.jpg',
                 'article' => 1,
-                'addresses' => [ // HasOne is tested in other test function
+                'addresses' => [// HasOne is tested in other test function
                     [
                         'city' => 2,
                         'street' => 'test',
@@ -1974,23 +1974,23 @@ 
         $this->crudPanel->addField([
             'name' => 'name',
             'events' => [
-                'created' => function ($entry) {
+                'created' => function($entry) {
                 },
-                'creating' => function ($entry) {
+                'creating' => function($entry) {
                     $entry->email = 'backpack@laravel.com';
                     $entry->password = Hash::make('password');
                 },
-                'saving' => function ($entry) {
+                'saving' => function($entry) {
                 },
-                'saved' => function ($entry) {
+                'saved' => function($entry) {
                 },
-                'updating' => function ($entry) {
+                'updating' => function($entry) {
                 },
-                'updated' => function ($entry) {
+                'updated' => function($entry) {
                 },
-                'deleting' => function ($entry) {
+                'deleting' => function($entry) {
                 },
-                'deleted' => function ($entry) {
+                'deleted' => function($entry) {
                 },
             ],
         ]);
@@ -2013,7 +2013,7 @@ 
 
     public function testItCanCreateDynamicRelationships()
     {
-        User::resolveRelationUsing('dynamicRelation', function ($user) {
+        User::resolveRelationUsing('dynamicRelation', function($user) {
             return $user->belongsTo(\Backpack\CRUD\Tests\config\Models\Bang::class, 'bang_relation_field');
         });
 

src/app/Library/CrudPanel/Traits/Create.php

Spacing +5 added lines, -5 removed lines

@@ -124,7 +124,7 @@ 
                     // - a single dimensional array: [1,2,3]
                     // - an array of arrays: [[1][2][3]]
                     // if is as single dimensional array we can only attach.
-                    if ($relationValues === null || ! is_multidimensional_array($relationValues)) {
+                    if ($relationValues === null || !is_multidimensional_array($relationValues)) {
                         $this->attachManyRelation($item, $relation, $relationDetails, $relationValues);
                     } else {
                         $this->createManyEntries($item, $relation, $relationMethod, $relationDetails);
@@ -146,7 +146,7 @@ 
 
                             $toDelete = array_diff($dbValues, $sentIds);
 
-                            if (! empty($toDelete)) {
+                            if (!empty($toDelete)) {
                                 foreach ($toDelete as $id) {
                                     $relation->newPivot()->where($keyName, $id)->delete();
                                 }
@@ -188,7 +188,7 @@ 
         }
     }
 
-    private function preparePivotAttributesForSave(array $attributes, BelongsToMany|MorphToMany $relation, string|int $relatedItemKey, $pivotKeyName, $relationMethod): array
+    private function preparePivotAttributesForSave(array $attributes, BelongsToMany | MorphToMany $relation, string | int $relatedItemKey, $pivotKeyName, $relationMethod): array
     {
         $attributes[$relation->getForeignPivotKeyName()] = $relatedItemKey;
         $attributes[$relation->getRelatedPivotKeyName()] = $attributes[$relationMethod];
@@ -334,7 +334,7 @@ 
         }
         // if column is not nullable in database, and there is no column default (null),
         // we will delete the entry from the database, otherwise it will throw and ugly DB error.
-        if (! $relationColumnIsNullable && $dbColumnDefault === null) {
+        if (!$relationColumnIsNullable && $dbColumnDefault === null) {
             return $removedEntries->lazy()->each->delete();
         }
 
@@ -379,7 +379,7 @@ 
         }
 
         // use the collection of sent ids to match against database ids, delete the ones not found in the submitted ids.
-        if (! empty($relatedItemsSent)) {
+        if (!empty($relatedItemsSent)) {
             // we perform the cleanup of removed database items
             $entry->{$relationMethod}()->whereNotIn($relatedModelLocalKey, $relatedItemsSent)->lazy()->each->delete();
         }

src/app/Library/CrudPanel/Traits/Access.php

Spacing +10 added lines, -10 removed lines

@@ -10,7 +10,7 @@ 
     /**
      * Set an operation as having access using the Settings API.
      */
-    public function allowAccess(array|string $operation): bool
+    public function allowAccess(array | string $operation): bool
     {
         foreach ((array) $operation as $op) {
             $this->set($op.'.access', true);
@@ -22,13 +22,13 @@ 
     /**
      * Disable the access to a certain operation, or the current one.
      */
-    public function denyAccess(array|string $operation): bool
+    public function denyAccess(array | string $operation): bool
     {
         foreach ((array) $operation as $op) {
             $this->set($op.'.access', false);
         }
 
-        return ! $this->hasAccessToAny($operation);
+        return !$this->hasAccessToAny($operation);
     }
 
     /**
@@ -52,7 +52,7 @@ 
     /**
      * Check if any operations are allowed for a Crud Panel. Return false if not.
      */
-    public function hasAccessToAny(array|string $operation_array, ?Model $entry = null): bool
+    public function hasAccessToAny(array | string $operation_array, ?Model $entry = null): bool
     {
         foreach ((array) $operation_array as $key => $operation) {
             if ($this->hasAccess($operation, $entry) == true) {
@@ -66,10 +66,10 @@ 
     /**
      * Check if all operations are allowed for a Crud Panel. Return false if not.
      */
-    public function hasAccessToAll(array|string $operation_array, ?Model $entry = null): bool
+    public function hasAccessToAll(array | string $operation_array, ?Model $entry = null): bool
     {
         foreach ((array) $operation_array as $key => $operation) {
-            if (! $this->hasAccess($operation, $entry)) {
+            if (!$this->hasAccess($operation, $entry)) {
                 return false;
             }
         }
@@ -84,7 +84,7 @@ 
      */
     public function hasAccessOrFail(string $operation, ?Model $entry = null): bool
     {
-        if (! $this->hasAccess($operation, $entry)) {
+        if (!$this->hasAccess($operation, $entry)) {
             throw new AccessDeniedException(trans('backpack::crud.unauthorized_access', ['access' => $operation]), 403);
         }
 
@@ -95,7 +95,7 @@ 
      * Get an operation's access condition, if set. A condition
      * can be anything, but usually a boolean or a callable.
      */
-    public function getAccessCondition(string $operation): bool|callable|null
+    public function getAccessCondition(string $operation): bool | callable | null
     {
         return $this->get($operation.'.access');
     }
@@ -103,7 +103,7 @@ 
     /**
      * Set the condition under which an operation is allowed for a Crud Panel.
      */
-    public function setAccessCondition(array|string $operation, bool|callable|null $condition): void
+    public function setAccessCondition(array | string $operation, bool | callable | null $condition): void
     {
         foreach ((array) $operation as $op) {
             $this->set($op.'.access', $condition);
@@ -131,7 +131,7 @@ 
      * Allow access only to operations in the array.
      * By denying access to all other operations.
      */
-    public function allowAccessOnlyTo(array|string $operation): void
+    public function allowAccessOnlyTo(array | string $operation): void
     {
         $this->denyAllAccess();
         $this->allowAccess($operation);