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/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}) {

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

src/app/Library/Uploaders/Support/RegisterUploadEvents.php

Spacing +13 added lines, -13 removed lines

@@ -14,18 +14,18 @@ 
     private string $crudObjectType;
 
     public function __construct(
-        private readonly CrudField|CrudColumn $crudObject,
+        private readonly CrudField | CrudColumn $crudObject,
         private readonly array $uploaderConfiguration,
         private readonly string $macro
         ) {
         $this->crudObjectType = is_a($crudObject, CrudField::class) ? 'field' : (is_a($crudObject, CrudColumn::class) ? 'column' : null);
 
-        if (! $this->crudObjectType) {
+        if (!$this->crudObjectType) {
             abort(500, 'Upload handlers only work for CrudField and CrudColumn classes.');
         }
     }
 
-    public static function handle(CrudField|CrudColumn $crudObject, array $uploaderConfiguration, string $macro, ?array $subfield = null, ?bool $registerModelEvents = true): void
+    public static function handle(CrudField | CrudColumn $crudObject, array $uploaderConfiguration, string $macro, ?array $subfield = null, ?bool $registerModelEvents = true): void
     {
         $instance = new self($crudObject, $uploaderConfiguration, $macro);
 
@@ -35,9 +35,9 @@ 
     /*******************************
      * Private methods - implementation
      *******************************/
-    private function registerEvents(array|null $subfield = [], ?bool $registerModelEvents = true): void
+    private function registerEvents(array | null $subfield = [], ?bool $registerModelEvents = true): void
     {
-        if (! empty($subfield)) {
+        if (!empty($subfield)) {
             $this->registerSubfieldEvent($subfield, $registerModelEvents);
 
             return;
@@ -81,7 +81,7 @@ 
         }
 
         $subfields = collect($this->crudObject->getAttributes()['subfields']);
-        $subfields = $subfields->map(function ($item) use ($subfield, $uploader) {
+        $subfields = $subfields->map(function($item) use ($subfield, $uploader) {
             if ($item['name'] === $subfield['name']) {
                 $item['upload'] = true;
                 $item['disk'] = $uploader->getDisk();
@@ -111,7 +111,7 @@ 
         }
 
         if ($this->crudObjectType === 'field') {
-            $model::saving(function ($entry) use ($uploader) {
+            $model::saving(function($entry) use ($uploader) {
                 $entry = $uploader->storeUploadedFiles($entry);
             });
         }
@@ -124,7 +124,7 @@ 
             // is not called in pivot models when loading the relations.
             $retrieveModel = $this->getModelForRetrieveEvent($model, $uploader);
 
-            $retrieveModel::retrieved(function ($entry) use ($uploader) {
+            $retrieveModel::retrieved(function($entry) use ($uploader) {
                 if ($entry->translationEnabled()) {
                     $locale = request('_locale', app()->getLocale());
                     if (in_array($locale, array_keys($entry->getAvailableLocales()))) {
@@ -135,7 +135,7 @@ 
             });
         }
 
-        $model::deleting(function ($entry) use ($uploader) {
+        $model::deleting(function($entry) use ($uploader) {
             $uploader->deleteUploadedFiles($entry);
         });
 
@@ -157,13 +157,13 @@ 
         $customUploader = isset($uploaderConfiguration['uploader']) && class_exists($uploaderConfiguration['uploader']);
 
         if ($customUploader) {
-            return $uploaderConfiguration['uploader']::for($crudObject, $uploaderConfiguration);
+            return $uploaderConfiguration['uploader']::for ($crudObject, $uploaderConfiguration);
         }
 
         $uploader = app('UploadersRepository')->hasUploadFor($crudObject['type'], $this->macro);
 
         if ($uploader) {
-            return app('UploadersRepository')->getUploadFor($crudObject['type'], $this->macro)::for($crudObject, $uploaderConfiguration);
+            return app('UploadersRepository')->getUploadFor($crudObject['type'], $this->macro)::for ($crudObject, $uploaderConfiguration);
         }
 
         throw new Exception('Undefined upload type for '.$this->crudObjectType.' type: '.$crudObject['type']);
@@ -179,7 +179,7 @@ 
 
     private function getSubfieldModel(array $subfield, UploaderInterface $uploader)
     {
-        if (! $uploader->isRelationship()) {
+        if (!$uploader->isRelationship()) {
             return $subfield['baseModel'] ?? get_class(app('crud')->getModel());
         }
 
@@ -192,7 +192,7 @@ 
 
     private function getModelForRetrieveEvent(string $model, UploaderInterface $uploader)
     {
-        if (! $uploader->isRelationship()) {
+        if (!$uploader->isRelationship()) {
             return $model;
         }
 

src/app/Library/Uploaders/SingleBase64Image.php

Spacing +2 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
         $value = $value ?? CRUD::getRequest()->get($this->getName());
         $previousImage = $this->getPreviousFiles($entry);
 
-        if (! $value && $previousImage) {
+        if (!$value && $previousImage) {
             Storage::disk($this->getDisk())->delete($previousImage);
 
             return null;
@@ -67,6 +67,6 @@ 
 
     protected function shouldKeepPreviousValueUnchanged(Model $entry, $entryValue): bool
     {
-        return $entry->exists && is_string($entryValue) && ! Str::startsWith($entryValue, 'data:image');
+        return $entry->exists && is_string($entryValue) && !Str::startsWith($entryValue, 'data:image');
     }
 }

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

Spacing +13 added lines, -13 removed lines

@@ -115,7 +115,7 @@ 
             return ['name' => Str::replace(' ', '', $field)];
         }
 
-        if (is_array($field) && ! isset($field['name'])) {
+        if (is_array($field) && !isset($field['name'])) {
             abort(500, 'All fields must have their name defined');
         }
 
@@ -189,7 +189,7 @@ 
         }
         // if there's a model defined, but no attribute
         // guess an attribute using the identifiableAttribute functionality in CrudTrait
-        if (isset($field['model']) && ! isset($field['attribute']) && method_exists($field['model'], 'identifiableAttribute')) {
+        if (isset($field['model']) && !isset($field['attribute']) && method_exists($field['model'], 'identifiableAttribute')) {
             $field['attribute'] = (new $field['model']())->identifiableAttribute();
         }
 
@@ -205,7 +205,7 @@ 
      */
     protected function makeSureFieldHasLabel($field)
     {
-        if (! isset($field['label'])) {
+        if (!isset($field['label'])) {
             $name = str_replace(',', ' ', $field['name']);
             $name = str_replace('_id', '', $name);
             $field['label'] = mb_ucfirst(str_replace('_', ' ', $name));
@@ -223,7 +223,7 @@ 
      */
     protected function makeSureFieldHasType($field)
     {
-        if (! isset($field['type'])) {
+        if (!isset($field['type'])) {
             $field['type'] = isset($field['relation_type']) ? $this->inferFieldTypeFromRelationType($field['relation_type']) : $this->inferFieldTypeFromDbColumnType($field['name']);
         }
 
@@ -256,16 +256,16 @@ 
      */
     protected function makeSureSubfieldsHaveNecessaryAttributes($field)
     {
-        if (! isset($field['subfields']) || ! is_array($field['subfields'])) {
+        if (!isset($field['subfields']) || !is_array($field['subfields'])) {
             return $field;
         }
 
-        if (! is_multidimensional_array($field['subfields'], true)) {
+        if (!is_multidimensional_array($field['subfields'], true)) {
             abort(500, 'Subfields of «'.$field['name'].'» are malformed. Make sure you provide an array of subfields.');
         }
 
         foreach ($field['subfields'] as $key => $subfield) {
-            if (empty($subfield) || ! isset($subfield['name'])) {
+            if (empty($subfield) || !isset($subfield['name'])) {
                 abort(500, 'A subfield of «'.$field['name'].'» is malformed. Subfield attribute name can\'t be empty.');
             }
 
@@ -280,7 +280,7 @@ 
             $subfield['baseFieldName'] = is_array($subfield['name']) ? implode(',', $subfield['name']) : $subfield['name'];
             $subfield['baseFieldName'] = Str::afterLast($subfield['baseFieldName'], '.');
 
-            if (! isset($field['model'])) {
+            if (!isset($field['model'])) {
                 // we're inside a simple 'repeatable' with no model/relationship, so
                 // we assume all subfields are supposed to be text fields
                 $subfield['type'] = $subfield['type'] ?? 'text';
@@ -305,11 +305,11 @@ 
                 case 'BelongsToMany':
                     $pivotSelectorField = static::getPivotFieldStructure($field);
 
-                    $pivot = Arr::where($field['subfields'], function ($item) use ($pivotSelectorField) {
+                    $pivot = Arr::where($field['subfields'], function($item) use ($pivotSelectorField) {
                         return $item['name'] === $pivotSelectorField['name'];
                     });
 
-                    if (! empty($pivot)) {
+                    if (!empty($pivot)) {
                         break;
                     }
 
@@ -322,11 +322,11 @@ 
                     $entity = isset($field['baseEntity']) ? $field['baseEntity'].'.'.$field['entity'] : $field['entity'];
                     $relationInstance = $this->getRelationInstance(['entity' => $entity]);
 
-                    $localKeyField = Arr::where($field['subfields'], function ($item) use ($relationInstance) {
+                    $localKeyField = Arr::where($field['subfields'], function($item) use ($relationInstance) {
                         return $item['name'] === $relationInstance->getRelated()->getKeyName();
                     });
 
-                    if (! empty($localKeyField)) {
+                    if (!empty($localKeyField)) {
                         break;
                     }
 
@@ -351,7 +351,7 @@ 
     {
         // if a tab was mentioned, we should enable it
         if (isset($field['tab'])) {
-            if (! $this->tabsEnabled()) {
+            if (!$this->tabsEnabled()) {
                 $this->enableTabs();
             }
         }