Laravel-Backpack / CRUD

src/routes/backpack/testing.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     (array) config('backpack.base.middleware_key', 'admin'),
     'prefix' => config('backpack.base.route_prefix', 'admin'),
 ],
-    function () {
+    function() {
         Route::crud('users', 'Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController');
     }
 );

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

Spacing +7 added lines, -7 removed lines

@@ -14,8 +14,8 @@ 
     {
         $this->getDbColumnTypes();
 
-        array_map(function ($field) use ($setFields, $setColumns) {
-            if ($setFields && ! isset($this->getCleanStateFields()[$field])) {
+        array_map(function($field) use ($setFields, $setColumns) {
+            if ($setFields && !isset($this->getCleanStateFields()[$field])) {
                 $this->addField([
                     'name' => $field,
                     'label' => $this->makeLabel($field),
@@ -28,7 +28,7 @@ 
                 ]);
             }
 
-            if ($setColumns && ! in_array($field, $this->model->getHidden()) && ! isset($this->columns()[$field])) {
+            if ($setColumns && !in_array($field, $this->model->getHidden()) && !isset($this->columns()[$field])) {
                 $this->addColumn([
                     'name' => $field,
                     'label' => $this->makeLabel($field),
@@ -50,7 +50,7 @@ 
     {
         $dbColumnTypes = [];
 
-        if (! $this->driverIsSql()) {
+        if (!$this->driverIsSql()) {
             return $dbColumnTypes;
         }
 
@@ -115,7 +115,7 @@ 
 
         $dbColumnTypes = $this->getDbColumnTypes();
 
-        if (! isset($dbColumnTypes[$fieldName])) {
+        if (!isset($dbColumnTypes[$fieldName])) {
             return 'text';
         }
 
@@ -212,12 +212,12 @@ 
     {
         $fillable = $this->model->getFillable();
 
-        if (! $this->driverIsSql()) {
+        if (!$this->driverIsSql()) {
             $columns = $fillable;
         } else {
             // Automatically-set columns should be both in the database, and in the $fillable variable on the Eloquent Model
             $columns = $this->model::getDbTableSchema()->getColumnsNames();
-            if (! empty($fillable)) {
+            if (!empty($fillable)) {
                 $columns = array_intersect($columns, $fillable);
             }
         }

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

Spacing +9 added lines, -9 removed lines

@@ -47,7 +47,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(),
@@ -72,12 +72,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']));
 
@@ -99,7 +99,7 @@ 
     {
         [$relatedModel, $relationMethod] = $this->getModelAndMethodFromEntity($model, $field);
 
-        if (! method_exists($relatedModel, $relationMethod)) {
+        if (!method_exists($relatedModel, $relationMethod)) {
             return $relatedModel->{$relationMethod};
         }
 
@@ -118,7 +118,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;
@@ -146,13 +146,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;
                 }
 
@@ -233,7 +233,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;
 
@@ -260,7 +260,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/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();
         });
     }

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

tests/Unit/CrudPanel/CrudPanelCreateTest.php

Spacing +2 added lines, -2 removed lines

@@ -605,7 +605,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',
@@ -741,7 +741,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',

tests/Unit/CrudPanel/CrudPanelValidationTest.php

Spacing +4 added lines, -4 removed lines

@@ -45,7 +45,7 @@ 
             'password' => 'test',
         ]);
 
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController', 'create']))->bind($request);
         });
 
@@ -85,7 +85,7 @@ 
             'email' => '',
         ]);
 
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController', 'create']))->bind($request);
         });
 
@@ -125,7 +125,7 @@ 
             'password' => 'test',
         ]);
 
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController', 'create']))->bind($request);
         });
 
@@ -247,7 +247,7 @@ 
             'name' => 'test',
         ]);
 
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController', 'create']))->bind($request);
         });
 

src/app/Library/Uploaders/Support/Traits/HandleRepeatableUploads.php

Spacing +15 added lines, -15 removed lines

@@ -13,7 +13,7 @@ 
 {
     public bool $handleRepeatableFiles = false;
 
-    public null|string $repeatableContainerName = null;
+    public null | string $repeatableContainerName = null;
 
     /*******************************
      * Setters - fluently configure the uploader
@@ -30,7 +30,7 @@ 
     /*******************************
      * Getters
      *******************************/
-    public function getRepeatableContainerName(): null|string
+    public function getRepeatableContainerName(): null | string
     {
         return $this->repeatableContainerName;
     }
@@ -76,7 +76,7 @@ 
         foreach (app('UploadersRepository')->getRepeatableUploadersFor($this->getRepeatableContainerName()) as $uploader) {
             $uploadedValues = $uploader->uploadRepeatableFiles($values->pluck($uploader->getAttributeName())->toArray(), $this->getPreviousRepeatableValues($entry, $uploader));
 
-            $values = $values->map(function ($item, $key) use ($uploadedValues, $uploader) {
+            $values = $values->map(function($item, $key) use ($uploadedValues, $uploader) {
                 $item[$uploader->getAttributeName()] = $uploadedValues[$key] ?? null;
 
                 return $item;
@@ -96,7 +96,7 @@ 
 
         $values = $entry->{$this->getRepeatableContainerName()};
         $values = is_string($values) ? json_decode($values, true) : $values;
-        $values = array_map(function ($item) use ($repeatableUploaders) {
+        $values = array_map(function($item) use ($repeatableUploaders) {
             foreach ($repeatableUploaders as $upload) {
                 $item[$upload->getAttributeName()] = $this->getValuesWithPathStripped($item, $upload);
             }
@@ -111,7 +111,7 @@ 
 
     private function retrieveRepeatableRelationFiles(Model $entry)
     {
-        switch($this->getRepeatableRelationType()) {
+        switch ($this->getRepeatableRelationType()) {
             case 'BelongsToMany':
             case 'MorphToMany':
                 $pivotClass = app('crud')->getModel()->{$this->getUploaderSubfield()['baseEntity']}()->getPivotClass();
@@ -157,12 +157,12 @@ 
 
         $repeatableValues = collect($entry->{$this->getName()});
         foreach (app('UploadersRepository')->getRepeatableUploadersFor($this->getRepeatableContainerName()) as $upload) {
-            if (! $upload->shouldDeleteFiles()) {
+            if (!$upload->shouldDeleteFiles()) {
                 continue;
             }
             $values = $repeatableValues->pluck($upload->getName())->toArray();
             foreach ($values as $value) {
-                if (! $value) {
+                if (!$value) {
                     continue;
                 }
 
@@ -185,7 +185,7 @@ 
     /**
      * Given two multidimensional arrays/collections, merge them recursively.
      */
-    protected function mergeValuesRecursive(array|Collection $array1, array|Collection $array2): array|Collection
+    protected function mergeValuesRecursive(array | Collection $array1, array | Collection $array2): array | Collection
     {
         $merged = $array1;
         foreach ($array2 as $key => &$value) {
@@ -207,7 +207,7 @@ 
     {
         $items = CRUD::getRequest()->input('_order_'.$this->getRepeatableContainerName()) ?? [];
 
-        array_walk($items, function (&$key, $value) {
+        array_walk($items, function(&$key, $value) {
             $requestValue = $key[$this->getName()] ?? null;
             $key = $this->handleMultipleFiles ? (is_string($requestValue) ? explode(',', $requestValue) : $requestValue) : $requestValue;
         });
@@ -219,18 +219,18 @@ 
     {
         $previousValues = json_decode($entry->getOriginal($uploader->getRepeatableContainerName()), true);
 
-        if (! empty($previousValues)) {
+        if (!empty($previousValues)) {
             $previousValues = array_column($previousValues, $uploader->getName());
         }
 
         return $previousValues ?? [];
     }
 
-    private function getValuesWithPathStripped(array|string|null $item, UploaderInterface $upload)
+    private function getValuesWithPathStripped(array | string | null $item, UploaderInterface $upload)
     {
         $uploadedValues = $item[$upload->getName()] ?? null;
         if (is_array($uploadedValues)) {
-            return array_map(function ($value) use ($upload) {
+            return array_map(function($value) use ($upload) {
                 return Str::after($value, $upload->getPath());
             }, $uploadedValues);
         }
@@ -242,19 +242,19 @@ 
     {
         if (in_array($this->getRepeatableRelationType(), ['BelongsToMany', 'MorphToMany'])) {
             $pivotAttributes = $entry->getAttributes();
-            $connectedPivot = $entry->pivotParent->{$this->getRepeatableContainerName()}->where(function ($item) use ($pivotAttributes) {
+            $connectedPivot = $entry->pivotParent->{$this->getRepeatableContainerName()}->where(function($item) use ($pivotAttributes) {
                 $itemPivotAttributes = $item->pivot->only(array_keys($pivotAttributes));
 
                 return $itemPivotAttributes === $pivotAttributes;
             })->first();
 
-            if (! $connectedPivot) {
+            if (!$connectedPivot) {
                 return;
             }
 
             $files = $connectedPivot->getOriginal()['pivot_'.$this->getAttributeName()];
 
-            if (! $files) {
+            if (!$files) {
                 return;
             }
 

src/app/Library/Uploaders/SingleFile.php

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
             return $this->getPath().$fileName;
         }
 
-        if (! $value && CrudPanelFacade::getRequest()->has($this->getNameForRequest()) && $previousFile) {
+        if (!$value && CrudPanelFacade::getRequest()->has($this->getNameForRequest()) && $previousFile) {
             Storage::disk($this->getDisk())->delete($previousFile);
 
             return null;
@@ -47,7 +47,7 @@ 
         }
 
         foreach ($previousRepeatableValues as $row => $file) {
-            if ($file && ! isset($orderedFiles[$row])) {
+            if ($file && !isset($orderedFiles[$row])) {
                 $orderedFiles[$row] = null;
                 Storage::disk($this->getDisk())->delete($file);
             }