Laravel-Backpack / CRUD

tests/Unit/CrudPanel/CrudPanelFieldsTest.php

Spacing +4 added lines, -4 removed lines

@@ -613,7 +613,7 @@ 
     {
         $this->crudPanel->setOperationSetting(
             'strippedRequest',
-            static function (Request $request) {
+            static function(Request $request) {
                 return $request->toArray();
             },
             'update'
@@ -748,7 +748,7 @@ 
                         ->validationMessages(['required' => 'is_required', 'min' => 'min_2'])
                         ->store_in('some')
                         ->size(6)
-                        ->on('created', function () {
+                        ->on('created', function() {
                         })
                         ->subfields([['name' => 'sub_1']])
                         ->entity('bang');
@@ -780,7 +780,7 @@ 
                 'class' => 'form-group col-md-6 mb-3',
             ],
             'events' => [
-                'created' => function () {
+                'created' => function() {
                 },
             ],
             'subfields' => [
@@ -982,7 +982,7 @@ 
 
     public function testItCanInferFieldAttributesFromADynamicRelation()
     {
-        User::resolveRelationUsing('dynamicRelation', function ($user) {
+        User::resolveRelationUsing('dynamicRelation', function($user) {
             return $user->hasOne(\Backpack\CRUD\Tests\config\Models\AccountDetails::class);
         });
 

src/app/Library/Widget.php

Spacing +5 added lines, -5 removed lines

@@ -37,7 +37,7 @@ 
         // if that widget name already exists in the widgets collection
         // then pick up all widget attributes from that entry
         // and overwrite them with the ones passed in $attributes
-        if ($existingItem = self::collection()->filter(function ($item) use ($attributes) {
+        if ($existingItem = self::collection()->filter(function($item) use ($attributes) {
             return $item->attributes['name'] === $attributes['name'];
         })->first()) {
             $attributes = array_merge($existingItem->attributes, $attributes);
@@ -159,7 +159,7 @@ 
             }
         }
         $type = $this->attributes['type'];
-        $paths = array_map(function ($item) use ($type) {
+        $paths = array_map(function($item) use ($type) {
             return $item.'.'.$type;
         }, ViewNamespaces::getWithFallbackFor('widgets', 'backpack.ui.component_view_namespaces.widgets'));
 
@@ -169,7 +169,7 @@ 
             }
         }
         // if no view exists, in any of the directories above... no bueno
-        if (! backpack_pro()) {
+        if (!backpack_pro()) {
             throw new BackpackProRequiredException('Cannot find the widget view: '.$this->attributes['type'].'. Please check for typos.'.(backpack_pro() ? '' : ' If you are trying to use a PRO widget, please first purchase and install the backpack/pro addon from backpackforlaravel.com'), 1);
         }
         abort(500, 'Cannot find the view for «'.$this->attributes['type'].'» widget type. Please check for typos.', ['developer-error-exception']);
@@ -253,10 +253,10 @@ 
      */
     private function save()
     {
-        $itemExists = $this->collection()->filter(function ($item) {
+        $itemExists = $this->collection()->filter(function($item) {
             return $item->attributes['name'] === $this->attributes['name'];
         })->isNotEmpty();
-        if (! $itemExists) {
+        if (!$itemExists) {
             $this->collection()->put($this->attributes['name'], $this);
         } else {
             $this->collection()[$this->attributes['name']] = $this;

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.', ['developer-error-exception']);
         }
     }
 
-    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']);
@@ -184,7 +184,7 @@ 
 
     private function getSubfieldModel(array $subfield, UploaderInterface $uploader)
     {
-        if (! $uploader->isRelationship()) {
+        if (!$uploader->isRelationship()) {
             return $subfield['baseModel'] ?? get_class(app('crud')->getModel());
         }
 
@@ -197,7 +197,7 @@ 
 
     private function getModelForRetrieveEvent(string $model, UploaderInterface $uploader)
     {
-        if (! $uploader->isRelationship()) {
+        if (!$uploader->isRelationship()) {
             return $model;
         }
 

src/app/Library/CrudPanel/CrudColumn.php

Spacing +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
      */
     public function key(string $key)
     {
-        if (! isset($this->attributes['name'])) {
+        if (!isset($this->attributes['name'])) {
             abort(500, 'Column name must be defined before changing the key.', ['developer-error-exception']);
         }
 

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

Spacing +10 added lines, -10 removed lines

@@ -72,7 +72,7 @@ 
         $this->enableFilters();
 
         // check if another filter with the same name exists
-        if (! isset($options['name'])) {
+        if (!isset($options['name'])) {
             abort(500, 'All your filters need names.', ['developer-error-exception']);
         }
 
@@ -120,7 +120,7 @@ 
     {
         $unappliedFilters = $this->filters()->where('applied', false);
         if ($unappliedFilters->count()) {
-            $unappliedFilters->each(function ($filter) {
+            $unappliedFilters->each(function($filter) {
                 $filter->apply();
             });
         }
@@ -167,7 +167,7 @@ 
     {
         $filter = $this->filters()->firstWhere('name', $name);
 
-        if (! $filter) {
+        if (!$filter) {
             abort(500, 'CRUD Filter "'.$name.'" not found. Please check the filter exists before you modify it.', ['developer-error-exception']);
         }
 
@@ -182,7 +182,7 @@ 
 
     public function replaceFilter($name, $newFilter)
     {
-        $newFilters = $this->filters()->map(function ($filter, $key) use ($name, $newFilter) {
+        $newFilters = $this->filters()->map(function($filter, $key) use ($name, $newFilter) {
             if ($filter->name != $name) {
                 return $filter;
             }
@@ -195,7 +195,7 @@ 
 
     public function removeFilter($name)
     {
-        $strippedCollection = $this->filters()->reject(function ($filter) use ($name) {
+        $strippedCollection = $this->filters()->reject(function($filter) use ($name) {
             return $filter->name == $name;
         });
 
@@ -238,7 +238,7 @@ 
      */
     public function makeFirstFilter()
     {
-        if (! $this->filters()) {
+        if (!$this->filters()) {
             return false;
         }
 
@@ -268,15 +268,15 @@ 
         $destinationFilter = $this->firstFilterWhere('name', $destination);
         $destinationKey = $this->getFilterKey($destination);
         $newDestinationKey = ($where == 'before' ? $destinationKey : $destinationKey + 1);
-        $newFilters = $this->filters()->filter(function ($value, $key) use ($target) {
+        $newFilters = $this->filters()->filter(function($value, $key) use ($target) {
             return $value->name != $target;
         });
 
-        if (! $targetFilter) {
+        if (!$targetFilter) {
             return;
         }
 
-        if (! $destinationFilter) {
+        if (!$destinationFilter) {
             return;
         }
 
@@ -284,7 +284,7 @@ 
         $lastSlice = $newFilters->slice($newDestinationKey, null);
 
         $newFilters = $firstSlice->push($targetFilter);
-        $lastSlice->each(function ($item, $key) use ($newFilters) {
+        $lastSlice->each(function($item, $key) use ($newFilters) {
             $newFilters->push($item);
         });
 

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', ['developer-error-exception']);
         }
 
@@ -193,7 +193,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();
         }
 
@@ -209,7 +209,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));
@@ -227,7 +227,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']);
         }
 
@@ -260,16 +260,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.', ['developer-error-exception']);
         }
 
         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.', ['developer-error-exception']);
             }
 
@@ -282,7 +282,7 @@ 
 
             $subfield['parentFieldName'] = $field['name'];
 
-            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';
@@ -309,11 +309,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;
                     }
 
@@ -337,11 +337,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;
                     }
 
@@ -366,7 +366,7 @@ 
     {
         // if a tab was mentioned, we should enable it
         if (isset($field['tab'])) {
-            if (! $this->tabsEnabled()) {
+            if (!$this->tabsEnabled()) {
                 $this->enableTabs();
             }
         }

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

Spacing +14 added lines, -14 removed lines

@@ -24,7 +24,7 @@ 
         $otherButtons = collect([]);
 
         // we get the buttons that belong to the specified stack
-        $stackButtons = $this->buttons()->reject(function ($item) use ($stack, $otherButtons) {
+        $stackButtons = $this->buttons()->reject(function($item) use ($stack, $otherButtons) {
             if ($item->stack != $stack) {
                 // if the button does not belong to this stack we just add it for merging later
                 $otherButtons->push($item);
@@ -36,8 +36,8 @@ 
         });
 
         // we parse the ordered buttons
-        collect($order)->each(function ($btnKey) use ($newButtons, $stackButtons) {
-            if (! $button = $stackButtons->where('name', $btnKey)->first()) {
+        collect($order)->each(function($btnKey) use ($newButtons, $stackButtons) {
+            if (!$button = $stackButtons->where('name', $btnKey)->first()) {
                 abort(500, 'Button name [«'.$btnKey.'»] not found.', ['developer-error-exception']);
             }
             $newButtons->push($button);
@@ -47,7 +47,7 @@ 
         // we add the remaining buttons to the end of the ordered ones
         if (count($newButtons) < count($stackButtons)) {
             foreach ($stackButtons as $button) {
-                if (! $newButtons->where('name', $button->name)->first()) {
+                if (!$newButtons->where('name', $button->name)->first()) {
                     $newButtons->push($button);
                 }
             }
@@ -116,7 +116,7 @@ 
          */
         $button = $this->buttons()->firstWhere('name', $name);
 
-        if (! $button) {
+        if (!$button) {
             abort(500, 'CRUD Button "'.$name.'" not found. Please ensure the button exists before you modify it.', ['developer-error-exception']);
         }
 
@@ -137,7 +137,7 @@ 
      */
     public function removeButton($name, $stack = null)
     {
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($name, $stack) {
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($name, $stack) {
             return $stack == null ? $button->name == $name : ($button->stack == $stack) && ($button->name == $name);
         }));
     }
@@ -148,7 +148,7 @@ 
      */
     public function removeButtons($names, $stack = null)
     {
-        if (! empty($names)) {
+        if (!empty($names)) {
             foreach ($names as $name) {
                 $this->removeButton($name, $stack);
             }
@@ -162,14 +162,14 @@ 
 
     public function removeAllButtonsFromStack($stack)
     {
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($stack) {
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($stack) {
             return $button->stack == $stack;
         }));
     }
 
     public function removeButtonFromStack($name, $stack)
     {
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($name, $stack) {
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($name, $stack) {
             return $button->name == $name && $button->stack == $stack;
         }));
     }
@@ -190,15 +190,15 @@ 
         $destinationKey = $this->getButtonKey($destination);
         $newDestinationKey = ($where == 'before' ? $destinationKey : $destinationKey + 1);
 
-        $newButtons = $this->buttons()->filter(function ($value, $key) use ($target) {
+        $newButtons = $this->buttons()->filter(function($value, $key) use ($target) {
             return $value->name != $target;
         });
 
-        if (! $targetButton) {
+        if (!$targetButton) {
             return;
         }
 
-        if (! $destinationButton) {
+        if (!$destinationButton) {
             return;
         }
 
@@ -207,7 +207,7 @@ 
 
         $newButtons = $firstSlice->push($targetButton);
 
-        $lastSlice->each(function ($item, $key) use ($newButtons) {
+        $lastSlice->each(function($item, $key) use ($newButtons) {
             $newButtons->push($item);
         });
 
@@ -266,7 +266,7 @@ 
     /**
      * Add a new button to the current CRUD operation.
      */
-    public function button(string|array $nameOrAttributes): CrudButton
+    public function button(string | array $nameOrAttributes): CrudButton
     {
         return new CrudButton($nameOrAttributes);
     }

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

Spacing +9 added lines, -9 removed lines

@@ -22,9 +22,9 @@ 
      */
     public function applySearchTerm($searchTerm)
     {
-        return $this->query->where(function ($query) use ($searchTerm) {
+        return $this->query->where(function($query) use ($searchTerm) {
             foreach ($this->columns() as $column) {
-                if (! isset($column['type'])) {
+                if (!isset($column['type'])) {
                     abort(500, 'Missing column type when trying to apply search term.', ['developer-error-exception']);
                 }
 
@@ -84,7 +84,7 @@ 
 
                 case 'select':
                 case 'select_multiple':
-                    $query->orWhereHas($column['entity'], function ($q) use ($column, $searchTerm, $searchOperator) {
+                    $query->orWhereHas($column['entity'], function($q) use ($column, $searchTerm, $searchOperator) {
                         $q->where($this->getColumnWithTableNamePrefixed($q, $column['attribute']), $searchOperator, '%'.$searchTerm.'%');
                     });
                     break;
@@ -108,7 +108,7 @@ 
                 $column_direction = (strtolower((string) $order['dir']) == 'asc' ? 'ASC' : 'DESC');
                 $column = $this->findColumnById($column_number);
 
-                if ($column['tableColumn'] && ! isset($column['orderLogic'])) {
+                if ($column['tableColumn'] && !isset($column['orderLogic'])) {
                     if (method_exists($this->model, 'translationEnabled') &&
                         $this->model->translationEnabled() &&
                         $this->model->isTranslatableAttribute($column['name']) &&
@@ -136,12 +136,12 @@ 
         $key = $this->model->getKeyName();
         $groupBy = $this->query->toBase()->groups;
 
-        $hasOrderByPrimaryKey = collect($orderBy)->some(function ($item) use ($key, $table) {
+        $hasOrderByPrimaryKey = collect($orderBy)->some(function($item) use ($key, $table) {
             return (isset($item['column']) && $item['column'] === $key)
                 || (isset($item['sql']) && str_contains($item['sql'], "$table.$key"));
         });
 
-        if (! $hasOrderByPrimaryKey && empty($groupBy)) {
+        if (!$hasOrderByPrimaryKey && empty($groupBy)) {
             $this->orderByWithPrefix($key, 'DESC');
         }
     }
@@ -320,13 +320,13 @@ 
         if (isset($column['type'])) {
             // create a list of paths to column blade views
             // including the configured view_namespaces
-            $columnPaths = array_map(function ($item) use ($column) {
+            $columnPaths = array_map(function($item) use ($column) {
                 return $item.'.'.$column['type'];
             }, ViewNamespaces::getFor('columns'));
 
             // but always fall back to the stock 'text' column
             // if a view doesn't exist
-            if (! in_array('crud::columns.text', $columnPaths)) {
+            if (!in_array('crud::columns.text', $columnPaths)) {
                 $columnPaths[] = 'crud::columns.text';
             }
 
@@ -365,7 +365,7 @@ 
      */
     private function renderCellView($view, $column, $entry, $rowNumber = false)
     {
-        if (! view()->exists($view)) {
+        if (!view()->exists($view)) {
             $view = 'crud::columns.text'; // fallback to text column
         }
 

src/app/Library/CrudPanel/Hooks/BackpackHooks.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 {
     public array $hooks = [];
 
-    public static function register(string $hook, string|array $operations, callable $callback): void
+    public static function register(string $hook, string | array $operations, callable $callback): void
     {
         $operations = is_array($operations) ? $operations : [$operations];