Laravel-Backpack / CRUD

src/app/Library/CrudPanel/CrudPanel.php

Spacing +18 added lines, -18 removed lines

@@ -109,11 +109,11 @@ 
      */
     public function setModel($model_namespace)
     {
-        if (! class_exists($model_namespace)) {
+        if (!class_exists($model_namespace)) {
             throw new Exception('The model does not exist.', 500);
         }
 
-        if (! method_exists($model_namespace, 'hasCrudTrait')) {
+        if (!method_exists($model_namespace, 'hasCrudTrait')) {
             throw new Exception('Please use CrudTrait on the model.', 500);
         }
 
@@ -206,7 +206,7 @@ 
 
         $complete_route = $route.'.index';
 
-        if (! \Route::has($complete_route)) {
+        if (!\Route::has($complete_route)) {
             throw new Exception('There are no routes for this route name.', 404);
         }
 
@@ -304,7 +304,7 @@ 
      */
     public function getFirstOfItsTypeInArray($type, $array)
     {
-        return Arr::first($array, function ($item) use ($type) {
+        return Arr::first($array, function($item) use ($type) {
             return $item['type'] == $type;
         });
     }
@@ -324,8 +324,8 @@ 
      */
     public function sync($type, $fields, $attributes)
     {
-        if (! empty($this->{$type})) {
-            $this->{$type} = array_map(function ($field) use ($fields, $attributes) {
+        if (!empty($this->{$type})) {
+            $this->{$type} = array_map(function($field) use ($fields, $attributes) {
                 if (in_array($field['name'], (array) $fields)) {
                     $field = array_merge($field, $attributes);
                 }
@@ -355,18 +355,18 @@ 
     {
         $relationArray = explode('.', $relationString);
 
-        if (! isset($length)) {
+        if (!isset($length)) {
             $length = count($relationArray);
         }
 
-        if (! isset($model)) {
+        if (!isset($model)) {
             $model = $this->model;
         }
 
-        $result = array_reduce(array_splice($relationArray, 0, $length), function ($obj, $method) {
+        $result = array_reduce(array_splice($relationArray, 0, $length), function($obj, $method) {
             try {
                 $result = $obj->$method();
-                if (! $result instanceof Relation) {
+                if (!$result instanceof Relation) {
                     throw new Exception('Not a relation');
                 }
 
@@ -402,7 +402,7 @@ 
             if (is_array($entries)) {
                 //if attribute does not exist in main array we have more than one entry OR the attribute
                 //is an accessor that is not in $appends property of model.
-                if (! isset($entries[$attribute])) {
+                if (!isset($entries[$attribute])) {
                     //we first check if we don't have the attribute because it's an accessor that is not in appends.
                     if ($model_instance->hasGetMutator($attribute) && isset($entries[$modelKey])) {
                         $entry_in_database = $model_instance->find($entries[$modelKey]);
@@ -441,21 +441,21 @@ 
      */
     public function parseTranslatableAttributes($model, $attribute, $value)
     {
-        if (! method_exists($model, 'isTranslatableAttribute')) {
+        if (!method_exists($model, 'isTranslatableAttribute')) {
             return $value;
         }
 
-        if (! $model->isTranslatableAttribute($attribute)) {
+        if (!$model->isTranslatableAttribute($attribute)) {
             return $value;
         }
 
-        if (! is_array($value)) {
+        if (!is_array($value)) {
             $decodedAttribute = json_decode($value, true) ?? ($value !== null ? [$value] : []);
         } else {
             $decodedAttribute = $value;
         }
 
-        if (is_array($decodedAttribute) && ! empty($decodedAttribute)) {
+        if (is_array($decodedAttribute) && !empty($decodedAttribute)) {
             if (isset($decodedAttribute[app()->getLocale()])) {
                 return $decodedAttribute[app()->getLocale()];
             } else {
@@ -473,7 +473,7 @@ 
         if (method_exists($model, 'translationEnabled') && $model->translationEnabled()) {
             $locale = $this->getRequest()->input('_locale', app()->getLocale());
             if (in_array($locale, array_keys($model->getAvailableLocales()))) {
-                $model->setLocale(! is_bool($useFallbackLocale) ? $useFallbackLocale : $locale);
+                $model->setLocale(!is_bool($useFallbackLocale) ? $useFallbackLocale : $locale);
                 $model->useFallbackLocale = (bool) $useFallbackLocale;
             }
         }
@@ -496,7 +496,7 @@ 
         $relation = $model->{$firstRelationName};
 
         $results = [];
-        if (! is_null($relation)) {
+        if (!is_null($relation)) {
             if ($relation instanceof Collection) {
                 $currentResults = $relation->all();
             } elseif (is_array($relation)) {
@@ -509,7 +509,7 @@ 
 
             array_shift($relationArray);
 
-            if (! empty($relationArray)) {
+            if (!empty($relationArray)) {
                 foreach ($currentResults as $currentResult) {
                     $results = array_merge_recursive($results, $this->getRelatedEntries($currentResult, implode('.', $relationArray)));
                 }

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

Spacing +14 added lines, -14 removed lines

@@ -24,7 +24,7 @@ 
             // here we are going to iterate through all relation parts to check
             foreach ($parts as $i => $part) {
                 $relation = $model->$part();
-                if (! is_a($relation, \Illuminate\Database\Eloquent\Relations\Relation::class, true)) {
+                if (!is_a($relation, \Illuminate\Database\Eloquent\Relations\Relation::class, true)) {
                     return $model;
                 }
                 $model = $relation->getRelated();
@@ -68,7 +68,7 @@ 
         $model = new ($this->getRelationModel($entity, -1));
         $lastSegmentAfterDot = Str::of($field['entity'])->afterLast('.')->value();
 
-        if (! $this->modelMethodIsRelationship($model, $lastSegmentAfterDot)) {
+        if (!$this->modelMethodIsRelationship($model, $lastSegmentAfterDot)) {
             return (string) Str::of($field['entity'])->beforeLast('.');
         }
 
@@ -89,7 +89,7 @@ 
 
         return collect($this->getCleanStateFields())
             ->whereIn('relation_type', $relation_types)
-            ->filter(function ($item) use ($nested) {
+            ->filter(function($item) use ($nested) {
                 if ($nested) {
                     return true;
                 }
@@ -133,7 +133,7 @@ 
      */
     private function getRelationFieldsWithoutRelationType($relations, $fields = [])
     {
-        if (! is_array($relations)) {
+        if (!is_array($relations)) {
             $relations = [$relations];
         }
 
@@ -142,8 +142,8 @@ 
         }
 
         foreach ($relations as $relation) {
-            $fields = array_filter($fields, function ($field) use ($relation) {
-                if (! isset($field['relation_type'])) {
+            $fields = array_filter($fields, function($field) use ($relation) {
+                if (!isset($field['relation_type'])) {
                     return false;
                 }
 
@@ -177,7 +177,7 @@ 
                     $fields = array_merge($field['subfields'], $fields);
                 }
             }
-            $fields = array_filter($fields, function ($field) {
+            $fields = array_filter($fields, function($field) {
                 return isset($field['relation_type']) && $field['relation_type'] === 'BelongsTo';
             });
         }
@@ -244,8 +244,8 @@ 
     {
         $all_relation_fields = $this->getRelationFields();
 
-        return Arr::where($all_relation_fields, function ($value, $key) {
-            return isset($value['pivot']) && ! $value['pivot'];
+        return Arr::where($all_relation_fields, function($value, $key) {
+            return isset($value['pivot']) && !$value['pivot'];
         });
     }
 
@@ -258,7 +258,7 @@ 
     {
         $all_relation_fields = $this->getRelationFields();
 
-        return Arr::where($all_relation_fields, function ($value, $key) {
+        return Arr::where($all_relation_fields, function($value, $key) {
             return isset($value['pivot']) && $value['pivot'];
         });
     }
@@ -328,7 +328,7 @@ 
      */
     private function modelMethodIsRelationship($model, $method)
     {
-        if (! method_exists($model, $method)) {
+        if (!method_exists($model, $method)) {
             if ($model->isRelation($method)) {
                 return $method;
             }
@@ -344,7 +344,7 @@ 
         }
 
         // relationships are always public methods.
-        if (! $methodReflection->isPublic()) {
+        if (!$methodReflection->isPublic()) {
             return false;
         }
 
@@ -371,7 +371,7 @@ 
      */
     public function isAttributeInRelationString(array $field): bool
     {
-        if (! str_contains($field['entity'], '.')) {
+        if (!str_contains($field['entity'], '.')) {
             return false;
         }
 
@@ -387,7 +387,7 @@ 
             try {
                 $model = $model->$part();
 
-                if (! is_a($model, \Illuminate\Database\Eloquent\Relations\Relation::class, true)) {
+                if (!is_a($model, \Illuminate\Database\Eloquent\Relations\Relation::class, true)) {
                     return true;
                 }
 

src/resources/views/crud/fields/checklist_dependency.blade.php

Spacing +3 added lines, -3 removed lines

@@ -20,7 +20,7 @@ 
     //all items with relation
     $dependencies = $primary_dependency['model']::with($primary_dependency['entity_secondary']);
 
-    if(isset($primary_dependency['options']) && $primary_dependency['options'] instanceof \Closure){
+    if (isset($primary_dependency['options']) && $primary_dependency['options'] instanceof \Closure) {
         $dependencies = $primary_dependency['options']($dependencies);
     }
 
@@ -76,7 +76,7 @@ 
 
     $primaryDependencyOptionQuery = $primary_dependency['model']::query();
 
-    if(isset($primary_dependency['options']) && $primary_dependency['options'] instanceof \Closure){
+    if (isset($primary_dependency['options']) && $primary_dependency['options'] instanceof \Closure) {
         $primaryDependencyOptionQuery = $primary_dependency['options']($primaryDependencyOptionQuery);
     }
 
@@ -84,7 +84,7 @@ 
 
     $secondaryDependencyOptionQuery = $secondary_dependency['model']::query();
 
-    if(isset($secondary_dependency['options']) && $secondary_dependency['options'] instanceof \Closure){
+    if (isset($secondary_dependency['options']) && $secondary_dependency['options'] instanceof \Closure) {
         $secondaryDependencyOptionQuery = $secondary_dependency['options']($secondaryDependencyOptionQuery);
     }
 

src/app/Http/Controllers/Operations/ReorderOperation.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
         $this->crud->set('reorder.enabled', true);
         $this->crud->allowAccess('reorder');
 
-        $this->crud->operation('reorder', function () {
+        $this->crud->operation('reorder', function() {
             $this->crud->loadDefaultOperationSettingsFromConfig();
             $this->crud->setOperationSetting('reorderColumnNames', [
                 'parent_id' => 'parent_id',
@@ -46,7 +46,7 @@ 
             ]);
         });
 
-        $this->crud->operation('list', function () {
+        $this->crud->operation('list', function() {
             $this->crud->addButton('top', 'reorder', 'view', 'crud::buttons.reorder');
         });
     }
@@ -62,7 +62,7 @@ 
     {
         $this->crud->hasAccessOrFail('reorder');
 
-        if (! $this->crud->isReorderEnabled()) {
+        if (!$this->crud->isReorderEnabled()) {
             abort(403, 'Reorder is disabled.');
         }
 

src/resources/views/crud/reorder.blade.php

Spacing +3 added lines, -3 removed lines

@@ -27,11 +27,11 @@ 
 
 @section('content')
 <?php
-if(!function_exists('tree_element')) {
+if (!function_exists('tree_element')) {
     function tree_element($entry, $key, $all_entries, $crud) {
         $columns = $crud->getOperationSetting('reorderColumnNames');
 
-        if (! isset($entry->tree_element_shown)) {
+        if (!isset($entry->tree_element_shown)) {
             // mark the element as shown
             $all_entries[$key]->tree_element_shown = true;
             $entry->tree_element_shown = true;
@@ -75,7 +75,7 @@ 
                 <ol class="sortable mt-0 mb-0">
                 <?php
                     $all_entries = collect($entries->all())->sortBy($columns['lft'])->keyBy($crud->getModel()->getKeyName());
-                    $root_entries = $all_entries->filter(function ($item) use ($columns) {
+                    $root_entries = $all_entries->filter(function($item) use ($columns) {
                         return $item->{$columns['parent_id']} == 0;
                     });
                     foreach ($root_entries as $key => $entry) {

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

Spacing +4 added lines, -4 removed lines

@@ -28,9 +28,9 @@ 
         $itemKeys = $this->model->query()->select($primaryKey)->get()->pluck($primaryKey);
 
         // filter the items that are not in the database and map the request
-        $reorderItems = collect($request)->filter(function ($item) use ($itemKeys) {
+        $reorderItems = collect($request)->filter(function($item) use ($itemKeys) {
             return $item['item_id'] !== '' && $item['item_id'] !== null && $itemKeys->contains($item['item_id']);
-        })->map(function ($item) use ($primaryKey, $columns) {
+        })->map(function($item) use ($primaryKey, $columns) {
             $item[$primaryKey] = $item['item_id'];
             $item[$columns['parent_id']] = empty($item['parent_id']) ? null : $item['parent_id'];
             $item[$columns['depth']] = empty($item['depth']) ? null : (int) $item['depth'];
@@ -59,12 +59,12 @@ 
 
         $sentIds = array_column($reorderItems, $primaryKey);
 
-        $itemKeys = $itemKeys->filter(function ($id) use ($sentIds) {
+        $itemKeys = $itemKeys->filter(function($id) use ($sentIds) {
             return in_array($id, $sentIds);
         });
 
         // wrap the queries in a transaction to avoid partial updates
-        DB::connection($this->model->getConnectionName())->transaction(function () use ($reorderItems, $primaryKey, $itemKeys, $columns) {
+        DB::connection($this->model->getConnectionName())->transaction(function() use ($reorderItems, $primaryKey, $itemKeys, $columns) {
             // create a string of ?,?,?,? to use as bind placeholders for item keys
             $reorderItemsBindString = implode(',', array_fill(0, count($reorderItems), '?'));
 

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