Laravel-Backpack / CRUD

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

Spacing +11 added lines, -11 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'];
         });
     }
@@ -336,7 +336,7 @@ 
         }
 
         // relationships are always public methods.
-        if (! $methodReflection->isPublic()) {
+        if (!$methodReflection->isPublic()) {
             return false;
         }
 
@@ -363,7 +363,7 @@ 
      */
     public function isAttributeInRelationString(array $field): bool
     {
-        if (! str_contains($field['entity'], '.')) {
+        if (!str_contains($field['entity'], '.')) {
             return false;
         }
 

src/app/Library/CrudPanel/Traits/Support/MacroableWithAttributes.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
                                         ->filter(fn ($item) => isset($item[$macro]));
 
                 $subfieldsWithMacros->each(
-                    function ($item) use ($subfieldsWithMacros, $macro) {
+                    function($item) use ($subfieldsWithMacros, $macro) {
                         if ($subfieldsWithMacros->last() === $item) {
                             $this->{$macro}($item[$macro], $item);
                         } else {

tests/BaseTestClass.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
             (array) config('backpack.base.middleware_key', 'admin'),
             'prefix'     => config('backpack.base.route_prefix', 'admin'),
         ],
-            function () {
+            function() {
                 Route::get('articles/{id}/show/{detail}', ['as' => 'article.show.detail', 'action' => 'Backpack\CRUD\Tests\config\Http\Controllers\ArticleCrudController@detail']);
                 Route::crud('users', 'Backpack\CRUD\Tests\config\Http\Controllers\UserCrudController');
                 Route::crud('articles', 'Backpack\CRUD\Tests\config\Http\Controllers\ArticleCrudController');

src/macros.php

Spacing +19 added lines, -19 removed lines

@@ -19,8 +19,8 @@ 
  *      - when true: `address[street]`
  *      - when false: `[address][street]`
  */
-if (! Str::hasMacro('dotsToSquareBrackets')) {
-    Str::macro('dotsToSquareBrackets', function ($string, $ignore = [], $keyFirst = true) {
+if (!Str::hasMacro('dotsToSquareBrackets')) {
+    Str::macro('dotsToSquareBrackets', function($string, $ignore = [], $keyFirst = true) {
         $stringParts = explode('.', $string);
         $result = '';
 
@@ -34,8 +34,8 @@ 
         return $result;
     });
 }
-if (! CrudColumn::hasMacro('withFiles')) {
-    CrudColumn::macro('withFiles', function ($uploadDefinition = [], $subfield = null, $registerUploaderEvents = true) {
+if (!CrudColumn::hasMacro('withFiles')) {
+    CrudColumn::macro('withFiles', function($uploadDefinition = [], $subfield = null, $registerUploaderEvents = true) {
         $uploadDefinition = is_array($uploadDefinition) ? $uploadDefinition : [];
         /** @var CrudField|CrudColumn $this */
         RegisterUploadEvents::handle($this, $uploadDefinition, 'withFiles', $subfield, $registerUploaderEvents);
@@ -44,8 +44,8 @@ 
     });
 }
 
-if (! CrudField::hasMacro('withFiles')) {
-    CrudField::macro('withFiles', function ($uploadDefinition = [], $subfield = null, $registerUploaderEvents = true) {
+if (!CrudField::hasMacro('withFiles')) {
+    CrudField::macro('withFiles', function($uploadDefinition = [], $subfield = null, $registerUploaderEvents = true) {
         $uploadDefinition = is_array($uploadDefinition) ? $uploadDefinition : [];
         /** @var CrudField|CrudColumn $this */
         RegisterUploadEvents::handle($this, $uploadDefinition, 'withFiles', $subfield, $registerUploaderEvents);
@@ -54,8 +54,8 @@ 
     });
 }
 
-if (! CrudColumn::hasMacro('linkTo')) {
-    CrudColumn::macro('linkTo', function (string|array|Closure $routeOrConfiguration, ?array $parameters = []): static {
+if (!CrudColumn::hasMacro('linkTo')) {
+    CrudColumn::macro('linkTo', function(string | array | Closure $routeOrConfiguration, ?array $parameters = []): static {
         $wrapper = $this->attributes['wrapper'] ?? [];
 
         // parse the function input to get the actual route and parameters we'll be working with
@@ -68,7 +68,7 @@ 
 
         // if the route is a closure, we'll just call it
         if ($route instanceof Closure) {
-            $wrapper['href'] = function ($crud, $column, $entry, $related_key) use ($route) {
+            $wrapper['href'] = function($crud, $column, $entry, $related_key) use ($route) {
                 return $route($entry, $related_key, $column, $crud);
             };
             $this->wrapper($wrapper);
@@ -77,13 +77,13 @@ 
         }
 
         // if the route doesn't exist, we'll throw an exception
-        if (! $routeInstance = Route::getRoutes()->getByName($route)) {
+        if (!$routeInstance = Route::getRoutes()->getByName($route)) {
             throw new \Exception("Route [{$route}] not found while building the link for column [{$this->attributes['name']}].");
         }
 
         // calculate the parameters we'll be using for the route() call
         // (eg. if there's only one parameter and user didn't provide it, we'll assume it's the entry's related key)
-        $parameters = (function () use ($parameters, $routeInstance, $route) {
+        $parameters = (function() use ($parameters, $routeInstance, $route) {
             $expectedParameters = $routeInstance->parameterNames();
 
             if (count($expectedParameters) === 0) {
@@ -94,7 +94,7 @@ 
             if (count($autoInferedParameter) > 1) {
                 throw new \Exception("Route [{$route}] expects parameters [".implode(', ', $expectedParameters)."]. Insuficient parameters provided in column: [{$this->attributes['name']}].");
             }
-            $autoInferedParameter = current($autoInferedParameter) ? [current($autoInferedParameter) => function ($entry, $related_key, $column, $crud) {
+            $autoInferedParameter = current($autoInferedParameter) ? [current($autoInferedParameter) => function($entry, $related_key, $column, $crud) {
                 $entity = $crud->isAttributeInRelationString($column) ? Str::before($column['entity'], '.') : $column['entity'];
 
                 return $related_key ?? $entry->{$entity}?->getKey();
@@ -104,7 +104,7 @@ 
         })();
 
         // set up the wrapper href attribute
-        $wrapper['href'] = function ($crud, $column, $entry, $related_key) use ($route, $parameters) {
+        $wrapper['href'] = function($crud, $column, $entry, $related_key) use ($route, $parameters) {
             // if the parameter is callable, we'll call it
             $parameters = collect($parameters)->map(fn ($item) => is_callable($item) ? $item($entry, $related_key, $column, $crud) : $item)->toArray();
 
@@ -117,17 +117,17 @@ 
     });
 }
 
-if (! CrudColumn::hasMacro('linkToShow')) {
-    CrudColumn::macro('linkToShow', function (): static {
+if (!CrudColumn::hasMacro('linkToShow')) {
+    CrudColumn::macro('linkToShow', function(): static {
         $name = $this->attributes['name'];
         $entity = $this->attributes['entity'] ?? null;
         $route = "$entity.show";
 
-        if (! $entity) {
+        if (!$entity) {
             throw new \Exception("Entity not found while building the link for column [{$name}].");
         }
 
-        if (! Route::getRoutes()->getByName($route)) {
+        if (!Route::getRoutes()->getByName($route)) {
             throw new \Exception("Route '{$route}' not found while building the link for column [{$name}].");
         }
 
@@ -144,8 +144,8 @@ 
  *
  * It will go to the given CrudController and get the setupRoutes() method on it.
  */
-if (! Route::hasMacro('crud')) {
-    Route::macro('crud', function ($name, $controller) {
+if (!Route::hasMacro('crud')) {
+    Route::macro('crud', function($name, $controller) {
         // put together the route name prefix,
         // as passed to the Route::group() statements
         $routeName = '';

src/app/Http/Controllers/Operations/ListOperation.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
             'operation' => 'list',
         ]);
 
-        if (! isset($this->setupDetailsRowRoute) || $this->setupDetailsRowRoute === true) {
+        if (!isset($this->setupDetailsRowRoute) || $this->setupDetailsRowRoute === true) {
             Route::get($segment.'/{id}/details', [
                 'as'        => $routeName.'.showDetailsRow',
                 'uses'      => $controller.'@showDetailsRow',
@@ -43,7 +43,7 @@ 
     {
         $this->crud->allowAccess('list');
 
-        $this->crud->operation('list', function () {
+        $this->crud->operation('list', function() {
             $this->crud->loadDefaultOperationSettingsFromConfig();
         });
     }

src/app/Library/Uploaders/MultipleFiles.php

Spacing +4 added lines, -4 removed lines

@@ -10,7 +10,7 @@ 
 
 class MultipleFiles extends Uploader
 {
-    public static function for(array $field, $configuration): UploaderInterface
+    public static function for (array $field, $configuration): UploaderInterface
     {
         return (new self($field, $configuration))->multiple();
     }
@@ -25,7 +25,7 @@ 
         $value = $value ?? collect(CRUD::getRequest()->file($this->getRepeatableContainerName() ?? $this->getName()))->flatten()->toArray();
         $previousFiles = $this->getPreviousFiles($entry) ?? [];
 
-        if (! is_array($previousFiles) && is_string($previousFiles)) {
+        if (!is_array($previousFiles) && is_string($previousFiles)) {
             $previousFiles = json_decode($previousFiles, true);
         }
 
@@ -34,14 +34,14 @@ 
                 if (in_array($previousFile, $filesToDelete)) {
                     Storage::disk($this->getDisk())->delete($previousFile);
 
-                    $previousFiles = Arr::where($previousFiles, function ($value, $key) use ($previousFile) {
+                    $previousFiles = Arr::where($previousFiles, function($value, $key) use ($previousFile) {
                         return $value != $previousFile;
                     });
                 }
             }
         }
 
-        if (! is_array($value)) {
+        if (!is_array($value)) {
             $value = [];
         }
 

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

Spacing +10 added lines, -10 removed lines

@@ -27,7 +27,7 @@ 
         $item = $this->model->findOrFail($id);
 
         [$directInputs, $relationInputs] = $this->splitInputIntoDirectAndRelations($input);
-        if($this->get('update.useDatabaseTransactions')) {
+        if ($this->get('update.useDatabaseTransactions')) {
             return DB::transaction(fn() => $this->updateModelAndRelations($item, $directInputs, $relationInputs));
         }
         
@@ -56,7 +56,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(),
@@ -81,12 +81,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']));
 
@@ -108,7 +108,7 @@ 
     {
         [$relatedModel, $relationMethod] = $this->getModelAndMethodFromEntity($model, $field);
 
-        if (! method_exists($relatedModel, $relationMethod)) {
+        if (!method_exists($relatedModel, $relationMethod)) {
             return $relatedModel->{$relationMethod};
         }
 
@@ -127,7 +127,7 @@ 
                     $model = $this->setupRelatedModelLocale($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;
@@ -155,13 +155,13 @@ 
                 break;
             case 'HasOne':
             case 'MorphOne':
-                if (! method_exists($relatedModel, $relationMethod)) {
+                if (!method_exists($relatedModel, $relationMethod)) {
                     return;
                 }
 
                 $model = $relatedModel->{$relationMethod};
 
-                if (! $model) {
+                if (!$model) {
                     return;
                 }
 
@@ -242,7 +242,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;
 
@@ -269,7 +269,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}) {

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

Spacing +4 added lines, -4 removed lines

@@ -24,7 +24,7 @@ 
     {
         [$directInputs, $relationInputs] = $this->splitInputIntoDirectAndRelations($input);
 
-        if($this->get('create.useDatabaseTransactions')) {
+        if ($this->get('create.useDatabaseTransactions')) {
             return DB::transaction(fn()  => $this->createModelAndRelations($directInputs, $relationInputs));
         }
         
@@ -120,7 +120,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);
@@ -284,7 +284,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();
         }
 
@@ -329,7 +329,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();
         }