Laravel-Backpack / CRUD

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;
                 },
             ],

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

Spacing +15 added lines, -16 removed lines

@@ -28,7 +28,7 @@ 
         $higherAction = $this->getSaveActionByOrder(1);
 
         //if there is an higher action and that action is not the backpack default higher one `save_and_back` we return it.
-        if (! empty($higherAction) && key($higherAction) !== 'save_and_back') {
+        if (!empty($higherAction) && key($higherAction) !== 'save_and_back') {
             return key($higherAction);
         }
 
@@ -47,7 +47,7 @@ 
      */
     public function getSaveActionByOrder($order)
     {
-        return array_filter($this->getOperationSetting('save_actions') ?? [], function ($arr) use ($order) {
+        return array_filter($this->getOperationSetting('save_actions') ?? [], function($arr) use ($order) {
             return $arr['order'] == $order;
         });
     }
@@ -88,7 +88,7 @@ 
 
         $actions = $this->getOperationSetting('save_actions') ?? [];
 
-        if (! in_array($saveAction['name'], $actions)) {
+        if (!in_array($saveAction['name'], $actions)) {
             $actions[$saveAction['name']] = $saveAction;
         }
 
@@ -105,7 +105,7 @@ 
     public function orderSaveAction(string $saveAction, int $wantedOrder)
     {
         $actions = $this->getOperationSetting('save_actions') ?? [];
-        if (! empty($actions)) {
+        if (!empty($actions)) {
             $replaceOrder = isset($actions[$saveAction]) ? $actions[$saveAction]['order'] : count($actions) + 1;
 
             foreach ($actions as $key => $sv) {
@@ -199,7 +199,7 @@ 
     public function orderSaveActions(array $saveActions)
     {
         foreach ($saveActions as $sv => $order) {
-            if (! is_int($order)) {
+            if (!is_int($order)) {
                 $this->orderSaveAction($order, $sv + 1);
             } else {
                 $this->orderSaveAction($sv, $order);
@@ -216,7 +216,7 @@ 
     {
         $actions = $this->getOperationSetting('save_actions') ?? [];
 
-        uasort($actions, function ($a, $b) {
+        uasort($actions, function($a, $b) {
             return $a['order'] <=> $b['order'];
         });
 
@@ -238,7 +238,7 @@ 
             }
         }
 
-        return array_filter($actions, function ($action) {
+        return array_filter($actions, function($action) {
             return $action['visible'] == true;
         }, ARRAY_FILTER_USE_BOTH);
     }
@@ -300,8 +300,7 @@ 
      */
     public function setSaveAction($forceSaveAction = null)
     {
-        $saveAction = $forceSaveAction ?:
-            $this->getRequest()->input('_save_action', $this->getFallBackSaveAction());
+        $saveAction = $forceSaveAction ?: $this->getRequest()->input('_save_action', $this->getFallBackSaveAction());
 
         $showBubble = $this->getOperationSetting('showSaveActionChange') ?? config('backpack.crud.operations.'.$this->getCurrentOperation().'.showSaveActionChange') ?? true;
 
@@ -369,20 +368,20 @@ 
         $defaultSaveActions = [
             [
                 'name' => 'save_and_back',
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return $crud->hasAccess('list');
                 },
-                'redirect' => function ($crud, $request, $itemId = null) {
+                'redirect' => function($crud, $request, $itemId = null) {
                     return $request->has('_http_referrer') ? $request->get('_http_referrer') : $crud->route;
                 },
                 'button_text' => trans('backpack::crud.save_action_save_and_back'),
             ],
             [
                 'name' => 'save_and_edit',
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return $crud->hasAccess('update');
                 },
-                'redirect' => function ($crud, $request, $itemId = null) {
+                'redirect' => function($crud, $request, $itemId = null) {
                     $itemId = $itemId ?: $request->get('id');
                     $redirectUrl = $crud->route.'/'.$itemId.'/edit';
                     if ($request->has('_locale')) {
@@ -394,17 +393,17 @@ 
 
                     return $redirectUrl;
                 },
-                'referrer_url' => function ($crud, $request, $itemId) {
+                'referrer_url' => function($crud, $request, $itemId) {
                     return url($crud->route.'/'.$itemId.'/edit');
                 },
                 'button_text' => trans('backpack::crud.save_action_save_and_edit'),
             ],
             [
                 'name' => 'save_and_new',
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return $crud->hasAccess('create');
                 },
-                'redirect' => function ($crud, $request, $itemId = null) {
+                'redirect' => function($crud, $request, $itemId = null) {
                     return $this->route.'/create';
                 },
                 'button_text' => trans('backpack::crud.save_action_save_and_new'),

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';

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

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();
         }