Laravel-Backpack / CRUD

tests/BaseTestClass.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,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');
@@ -45,7 +45,7 @@ 
     protected function setupUserCreateRequest()
     {
         $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']);
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new RouteInstance('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request);
         });
         $this->crudPanel->setRequest($request);

tests/Unit/CrudPanel/CrudPanelQueryTest.php

Spacing +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
 
     public function testItCanAddAClauseToTheQueryUsingAClosure()
     {
-        $closure = function ($query) {
+        $closure = function($query) {
             $query->where('id', 1);
         };
 
@@ -51,7 +51,7 @@ 
 
     public function testItCanAddABaseClauseToTheQueryUsingAClosure()
     {
-        $closure = function ($query) {
+        $closure = function($query) {
             $query->where('id', 1);
         };
 
@@ -115,7 +115,7 @@ 
         $this->crudPanel->addColumn([
             'name' => 'name',
             'type' => 'text',
-            'orderLogic' => function ($query, $column, $direction) {
+            'orderLogic' => function($query, $column, $direction) {
                 $query->orderBy('name', $direction);
             },
         ]);

tests/Unit/CrudPanel/CrudPanelSaveActionsTest.php

Spacing +7 added lines, -7 removed lines

@@ -25,13 +25,13 @@ 
         $this->singleSaveAction = [
             'name' => 'save_action_one',
             'button_text' => 'custom',
-            'redirect' => function ($crud, $request, $itemId) {
+            'redirect' => function($crud, $request, $itemId) {
                 return 'https://backpackforlaravel.com';
             },
-            'referrer_url' => function ($crud, $request, $itemId) {
+            'referrer_url' => function($crud, $request, $itemId) {
                 return 'https://backpackforlaravel.com';
             },
-            'visible' => function ($crud) {
+            'visible' => function($crud) {
                 return true;
             },
         ];
@@ -39,19 +39,19 @@ 
         $this->multipleSaveActions = [
             [
                 'name' => 'save_action_one',
-                'redirect' => function ($crud, $request, $itemId) {
+                'redirect' => function($crud, $request, $itemId) {
                     return $crud->route;
                 },
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return true;
                 },
             ],
             [
                 'name' => 'save_action_two',
-                'redirect' => function ($crud, $request, $itemId) {
+                'redirect' => function($crud, $request, $itemId) {
                     return $crud->route;
                 },
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return true;
                 },
             ],

tests/Unit/CrudPanel/CrudPanelSearchTest.php

Spacing +3 added lines, -3 removed lines

@@ -90,7 +90,7 @@ 
         $this->crudPanel->addColumn([
             'name' => 'test',
             'type' => 'my_custom_type',
-            'searchLogic' => function ($query, $column, $searchTerm) {
+            'searchLogic' => function($query, $column, $searchTerm) {
                 $query->where($column['name'], 'like', "%{$searchTerm}%");
             },
             'tableColumn' => true,
@@ -225,7 +225,7 @@ 
             'name' => 'name',
             'type' => 'text',
             'tableColumn' => true,
-            'orderLogic' => function ($query, $column, $searchTerm) {
+            'orderLogic' => function($query, $column, $searchTerm) {
                 $query->orderBy('name', 'asc');
             },
         ]);
@@ -244,7 +244,7 @@ 
             'name' => 'name',
             'type' => 'text',
             'tableColumn' => true,
-            'orderLogic' => function ($query, $column, $searchTerm) {
+            'orderLogic' => function($query, $column, $searchTerm) {
                 $query->orderBy('id', 'asc');
             },
         ]);

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

Spacing +8 added lines, -9 removed lines

@@ -50,7 +50,7 @@ 
             return ['name' => Str::replace(' ', '', $column)];
         }
 
-        if (is_array($column) && ! isset($column['name'])) {
+        if (is_array($column) && !isset($column['name'])) {
             $column['name'] = 'anonymous_column_'.Str::random(5);
         }
 
@@ -68,7 +68,7 @@ 
      */
     protected function makeSureColumnHasLabel($column)
     {
-        if (! isset($column['label'])) {
+        if (!isset($column['label'])) {
             $column['label'] = mb_ucfirst($this->makeLabel($column['name']));
         }
 
@@ -153,7 +153,7 @@ 
      */
     protected function makeSureColumnHasKey($column)
     {
-        if (! isset($column['key'])) {
+        if (!isset($column['key'])) {
             $column['key'] = str_replace('.', '__', $column['name']);
         }
 
@@ -170,7 +170,7 @@ 
      */
     protected function makeSureColumnHasWrapper($column)
     {
-        if (! isset($column['wrapper'])) {
+        if (!isset($column['wrapper'])) {
             $column['wrapper'] = [];
         }
 
@@ -255,7 +255,7 @@ 
     {
         // if this is a relation type field and no corresponding model was specified,
         // get it from the relation method defined in the main model
-        if (isset($column['entity']) && $column['entity'] !== false && ! isset($column['model'])) {
+        if (isset($column['entity']) && $column['entity'] !== false && !isset($column['model'])) {
             $column['model'] = $this->getRelationModel($column['entity']);
         }
 
@@ -292,8 +292,7 @@ 
         $columnsArray = $this->columns();
 
         if (array_key_exists($targetColumnName, $columnsArray)) {
-            $targetColumnPosition = $before ? array_search($targetColumnName, array_keys($columnsArray)) :
-                array_search($targetColumnName, array_keys($columnsArray)) + 1;
+            $targetColumnPosition = $before ? array_search($targetColumnName, array_keys($columnsArray)) : array_search($targetColumnName, array_keys($columnsArray)) + 1;
 
             $element = array_pop($columnsArray);
 
@@ -322,7 +321,7 @@ 
     {
         static $cache = [];
 
-        if (! $this->driverIsSql()) {
+        if (!$this->driverIsSql()) {
             return true;
         }
 
@@ -338,7 +337,7 @@ 
     /**
      * Prepare the column attributes and add it to operation settings.
      */
-    private function prepareAttributesAndAddColumn(array|string $column): CrudColumn
+    private function prepareAttributesAndAddColumn(array | string $column): CrudColumn
     {
         $column = $this->makeSureColumnHasNeededAttributes($column);
         $this->addColumnToOperationSettings($column);

src/app/Console/Commands/Install.php

Spacing +16 added lines, -16 removed lines

@@ -83,7 +83,7 @@ 
 
         // Install Backpack Generators
         $this->progressBlock('Installing Generators');
-        if (! file_exists('vendor/backpack/generators/composer.json')) {
+        if (!file_exists('vendor/backpack/generators/composer.json')) {
             // only do this if Generators aren't already required
             $process = new Process(['composer', 'require', '--dev', 'backpack/generators']);
             $process->setTimeout(300);
@@ -97,7 +97,7 @@ 
         $this->closeProgressBlock();
 
         // Optional commands
-        if (! $this->option('no-interaction')) {
+        if (!$this->option('no-interaction')) {
             // Themes
             $this->installTheme();
 
@@ -106,7 +106,7 @@ 
 
             // Addons
             $this->installAddons();
-        } elseif (! $this->isAnyThemeInstalled()) {
+        } elseif (!$this->isAnyThemeInstalled()) {
             // Install default theme
             $this->progressBlock('Installing default theme');
             $this->executeArtisanProcess('backpack:require:theme-tabler');
@@ -189,7 +189,7 @@ 
 
         $this->deleteLines(3);
 
-        if (! $total) {
+        if (!$total) {
             $this->deleteLines();
             $this->note('Skipping creating an admin user.');
             $this->newLine();
@@ -198,14 +198,14 @@ 
 
     private function isEveryAddonInstalled()
     {
-        return collect($this->addons)->every(function ($addon) {
+        return collect($this->addons)->every(function($addon) {
             return file_exists($addon->path);
         });
     }
 
     private function updateAddonsStatus()
     {
-        $this->addons = $this->addons->each(function (&$addon) {
+        $this->addons = $this->addons->each(function(&$addon) {
             $isInstalled = file_exists($addon->path);
             $addon->status = $isInstalled ? 'installed' : 'not installed';
             $addon->statusColor = $isInstalled ? 'green' : 'yellow';
@@ -216,7 +216,7 @@ 
     {
         // map the addons
         $this->addons = collect($this->addons)
-            ->map(function ($class) {
+            ->map(function($class) {
                 return (object) $class::$addon;
             });
 
@@ -235,15 +235,15 @@ 
 
         // Calculate the printed line count
         $printedLines = $this->addons
-            ->map(function ($e) {
+            ->map(function($e) {
                 return count($e->description);
             })
-            ->reduce(function ($sum, $item) {
+            ->reduce(function($sum, $item) {
                 return $sum + $item + 2;
             }, 0);
 
         $total = 0;
-        while (! $this->isEveryAddonInstalled()) {
+        while (!$this->isEveryAddonInstalled()) {
             $input = (int) $this->listChoice('Would you like to install a premium Backpack add-on? <fg=gray>(enter an option number from above or press any other key to continue the installation)</>', $this->addons->toArray());
 
             if ($input < 1 || $input > $this->addons->count()) {
@@ -278,14 +278,14 @@ 
 
     private function isEveryThemeInstalled()
     {
-        return $this->themes()->every(function ($theme) {
+        return $this->themes()->every(function($theme) {
             return $theme->status == 'installed';
         });
     }
 
     private function isAnyThemeInstalled()
     {
-        return $this->themes()->filter(function ($theme) {
+        return $this->themes()->filter(function($theme) {
             return $theme->status == 'installed';
         })->count() > 0;
     }
@@ -303,10 +303,10 @@ 
 
         // Calculate the printed line count
         $printedLines = $this->themes()
-            ->map(function ($e) {
+            ->map(function($e) {
                 return count($e->description);
             })
-            ->reduce(function ($sum, $item) {
+            ->reduce(function($sum, $item) {
                 return $sum + $item + 2;
             }, 0);
 
@@ -342,9 +342,9 @@ 
     public function themes()
     {
         return collect($this->themes)
-            ->map(function ($class) {
+            ->map(function($class) {
                 return (object) $class::$addon;
-            })->each(function (&$theme) {
+            })->each(function(&$theme) {
                 $isInstalled = file_exists($theme->path);
                 $theme->status = $isInstalled ? 'installed' : 'not installed';
                 $theme->statusColor = $isInstalled ? 'green' : 'yellow';

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

Spacing +6 added lines, -6 removed lines

@@ -98,7 +98,7 @@ 
         }
 
         //remove fields that are not in the submitted form input
-        $relationFields = array_filter($relationFields, function ($field) use ($input) {
+        $relationFields = array_filter($relationFields, function($field) use ($input) {
             return Arr::has($input, $field['name']) || isset($input[$field['name']]) || Arr::has($input, Str::afterLast($field['name'], '.'));
         });
 
@@ -127,7 +127,7 @@ 
                     // the key used to store the values is the main relation key
                     $key = Str::beforeLast($this->getOnlyRelationEntity($field), '.');
 
-                    if (! isset($field['parentFieldName']) && isset($field['entity'])) {
+                    if (!isset($field['parentFieldName']) && isset($field['entity'])) {
                         $mainField = $field;
                         $mainField['entity'] = Str::beforeLast($field['entity'], '.');
 
@@ -185,7 +185,7 @@ 
 
             // when using dot notation if relationMethod is not set we are sure we want to exclude those relations.
             if ($this->getOnlyRelationEntity($field) !== $field['entity']) {
-                if (! $relationMethod) {
+                if (!$relationMethod) {
                     $excludedFields[] = $nameToExclude;
                 }
 
@@ -199,8 +199,8 @@ 
             }
         }
 
-        return Arr::where($input, function ($item, $key) use ($excludedFields) {
-            return ! in_array($key, $excludedFields);
+        return Arr::where($input, function($item, $key) use ($excludedFields) {
+            return !in_array($key, $excludedFields);
         });
     }
 
@@ -226,7 +226,7 @@ 
                 $jsonCastables = ['array', 'object', 'json'];
                 $fieldCasting = $casted_attributes[$field['name']];
 
-                if (in_array($fieldCasting, $jsonCastables) && isset($input[$field['name']]) && ! empty($input[$field['name']]) && ! is_array($input[$field['name']])) {
+                if (in_array($fieldCasting, $jsonCastables) && isset($input[$field['name']]) && !empty($input[$field['name']]) && !is_array($input[$field['name']])) {
                     try {
                         $input[$field['name']] = json_decode($input[$field['name']]);
                     } catch (\Exception $e) {

tests/Unit/CrudPanel/CrudPanelColumnsTest.php

Spacing +1 added lines, -1 removed lines

@@ -931,7 +931,7 @@
 
     public function testItCanInferFieldAttributesFromADynamicRelation()
     {
-        User::resolveRelationUsing('dynamicRelation', function ($user) {
+        User::resolveRelationUsing('dynamicRelation', function($user) {
             return $user->belongsTo(\Backpack\CRUD\Tests\config\Models\Bang::class);
         });
 

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'])) {