Laravel-Backpack / CRUD

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;
         }
         $dbColumns = $this->getDbTableColumns();
@@ -115,7 +115,7 @@ 
 
         $dbColumnTypes = $this->getDbColumnTypes();
 
-        if (! isset($dbColumnTypes[$fieldName])) {
+        if (!isset($dbColumnTypes[$fieldName])) {
             return 'text';
         }
 
@@ -210,12 +210,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);
             }
         }

tests/Unit/CrudPanel/CrudPanelAutoSetTest.php

Spacing +1 added lines, -1 removed lines

@@ -848,7 +848,7 @@
 
     public function testSetDoctrineTypesMapping()
     {
-        if (! method_exists($this->crudPanel->getModel()->getConnection(), 'getDoctrineConnection')) {
+        if (!method_exists($this->crudPanel->getModel()->getConnection(), 'getDoctrineConnection')) {
             $this->markTestSkipped('This test is only for Laravel 10, Laravel 11 does not have dbal as a dependency anymore');
         }
 

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

Spacing +3 added lines, -3 removed lines

@@ -122,7 +122,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);
@@ -293,7 +293,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();
         }
 
@@ -338,7 +338,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();
         }

src/app/Models/Traits/HasRelationshipFields.php

Spacing +6 added lines, -6 removed lines

@@ -22,7 +22,7 @@ 
     {
         $connection = DB::connection($this->getConnectionName());
 
-        if (! method_exists($connection, 'getDoctrineSchemaManager')) {
+        if (!method_exists($connection, 'getDoctrineSchemaManager')) {
             return $connection;
         }
 
@@ -35,7 +35,7 @@ 
         if (self::isSqlConnection()) {
             $platform = $connection->getDoctrineSchemaManager()->getDatabasePlatform();
             foreach ($types as $type_key => $type_value) {
-                if (! $platform->hasDoctrineTypeMappingFor($type_key)) {
+                if (!$platform->hasDoctrineTypeMappingFor($type_key)) {
                     $platform->registerDoctrineTypeMapping($type_key, $type_value);
                 }
             }
@@ -65,7 +65,7 @@ 
      */
     public function getColumnType($columnName)
     {
-        if (! self::isSqlConnection()) {
+        if (!self::isSqlConnection()) {
             return 'text';
         }
 
@@ -80,7 +80,7 @@ 
      */
     public static function isColumnNullable($columnName)
     {
-        if (! self::isSqlConnection()) {
+        if (!self::isSqlConnection()) {
             return true;
         }
 
@@ -95,7 +95,7 @@ 
      */
     public static function dbColumnHasDefault($columnName)
     {
-        if (! self::isSqlConnection()) {
+        if (!self::isSqlConnection()) {
             return false;
         }
 
@@ -110,7 +110,7 @@ 
      */
     public static function getDbColumnDefault($columnName)
     {
-        if (! self::isSqlConnection()) {
+        if (!self::isSqlConnection()) {
             return false;
         }
 

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.');
@@ -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/Http/Controllers/Operations/ListOperation.php

Spacing +3 added lines, -3 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();
         });
     }
@@ -80,7 +80,7 @@ 
         $search = request()->input('search');
 
         // check if length is allowed by developer
-        if ($length && ! in_array($length, $this->crud->getPageLengthMenu()[0])) {
+        if ($length && !in_array($length, $this->crud->getPageLengthMenu()[0])) {
             return response()->json([
                 'error' => 'Unknown page length.',
             ], 400);

src/app/Library/Database/TableSchema.php

Spacing +8 added lines, -8 removed lines

@@ -20,7 +20,7 @@ 
     public function getColumnsNames()
     {
         return array_values(
-            array_map(function ($item) {
+            array_map(function($item) {
                 return $item->getName();
             }, $this->getColumns())
         );
@@ -34,7 +34,7 @@ 
      */
     public function getColumnType(string $columnName)
     {
-        if (! $this->schemaExists() || ! $this->schema->hasColumn($columnName)) {
+        if (!$this->schemaExists() || !$this->schema->hasColumn($columnName)) {
             return 'varchar';
         }
 
@@ -51,7 +51,7 @@ 
      */
     public function hasColumn($columnName)
     {
-        if (! $this->schemaExists()) {
+        if (!$this->schemaExists()) {
             return false;
         }
 
@@ -66,7 +66,7 @@ 
      */
     public function columnIsNullable($columnName)
     {
-        if (! $this->hasColumn($columnName)) {
+        if (!$this->hasColumn($columnName)) {
             return true;
         }
 
@@ -83,7 +83,7 @@ 
      */
     public function columnHasDefault($columnName)
     {
-        if (! $this->hasColumn($columnName)) {
+        if (!$this->hasColumn($columnName)) {
             return false;
         }
 
@@ -100,7 +100,7 @@ 
      */
     public function getColumnDefault($columnName)
     {
-        if (! $this->hasColumn($columnName)) {
+        if (!$this->hasColumn($columnName)) {
             return false;
         }
 
@@ -116,7 +116,7 @@ 
      */
     public function getColumns()
     {
-        if (! $this->schemaExists()) {
+        if (!$this->schemaExists()) {
             return [];
         }
 
@@ -130,7 +130,7 @@ 
      */
     private function schemaExists()
     {
-        if (! empty($this->schema)) {
+        if (!empty($this->schema)) {
             return true;
         }
 

src/app/Library/Validation/Rules/BackpackCustomRule.php

Spacing +15 added lines, -16 removed lines

@@ -29,13 +29,13 @@ 
 
     public bool $implicit = true;
 
-    public static function field(string|array|ValidationRule|Rule $rules = []): self
+    public static function field(string | array | ValidationRule | Rule $rules = []): self
     {
         $instance = new static();
         $instance->fieldRules = self::getRulesAsArray($rules);
 
         if ($instance->validatesArrays()) {
-            if (! in_array('array', $instance->getFieldRules())) {
+            if (!in_array('array', $instance->getFieldRules())) {
                 $instance->fieldRules[] = 'array';
             }
         }
@@ -95,7 +95,7 @@ 
 
     public function getFieldRules(): array
     {
-        return tap($this->fieldRules, function ($rule) {
+        return tap($this->fieldRules, function($rule) {
             if (is_a($rule, BackpackCustomRule::class, true)) {
                 $rule = $rule->getFieldRules();
             }
@@ -110,7 +110,7 @@ 
             $rules = explode('|', $rules);
         }
 
-        if (! is_array($rules)) {
+        if (!is_array($rules)) {
             $rules = [$rules];
         }
 
@@ -119,10 +119,10 @@ 
 
     protected function ensureValueIsValid($value)
     {
-        if ($this->validatesArrays() && ! is_array($value)) {
+        if ($this->validatesArrays() && !is_array($value)) {
             try {
                 $value = json_decode($value, true) ?? [];
-            } catch(\Exception $e) {
+            } catch (\Exception $e) {
                 return false;
             }
         }
@@ -141,12 +141,12 @@ 
             $attribute => $rules,
         ], $this->validator->customMessages, $this->getValidatorCustomAttributes($attribute));
 
-        return $validator->errors()->messages()[$attribute] ?? (! empty($validator->errors()->messages()) ? current($validator->errors()->messages()) : []);
+        return $validator->errors()->messages()[$attribute] ?? (!empty($validator->errors()->messages()) ? current($validator->errors()->messages()) : []);
     }
 
     private function getValidatorCustomAttributes(string $attribute): array
     {
-        if (! is_a($this, ValidGenericAjaxEndpoint::class) && ! Str::contains($attribute, '.*.')) {
+        if (!is_a($this, ValidGenericAjaxEndpoint::class) && !Str::contains($attribute, '.*.')) {
             return $this->validator->customAttributes;
         }
 
@@ -157,8 +157,7 @@ 
     protected function getValidationAttributeString(string $attribute)
     {
         return Str::substrCount($attribute, '.') > 1 ?
-                Str::before($attribute, '.').'.*.'.Str::afterLast($attribute, '.') :
-                $attribute;
+                Str::before($attribute, '.').'.*.'.Str::afterLast($attribute, '.') : $attribute;
     }
 
     protected function validateOnSubmit(string $attribute, mixed $value): array
@@ -166,7 +165,7 @@ 
         return $this->validateRules($attribute, $value);
     }
 
-    protected function validateFieldAndFile(string $attribute, null|array $data = null, array|null $customRules = null): array
+    protected function validateFieldAndFile(string $attribute, null | array $data = null, array | null $customRules = null): array
     {
         $fieldErrors = $this->validateFieldRules($attribute, $data, $customRules);
 
@@ -178,7 +177,7 @@ 
     /**
      * Implementation.
      */
-    public function validateFieldRules(string $attribute, null|array|string|UploadedFile $data = null, array|null $customRules = null): array
+    public function validateFieldRules(string $attribute, null | array | string | UploadedFile $data = null, array | null $customRules = null): array
     {
         $data = $data ?? $this->data;
         $validationRuleAttribute = $this->getValidationAttributeString($attribute);
@@ -187,16 +186,16 @@ 
         return $this->validateAndGetErrors($validationRuleAttribute, $data, $customRules ?? $this->getFieldRules());
     }
 
-    protected function prepareValidatorData(array|string|UploadedFile $data, string $attribute): array
+    protected function prepareValidatorData(array | string | UploadedFile $data, string $attribute): array
     {
-        if ($this->validatesArrays() && is_array($data) && ! Str::contains($attribute, '.')) {
+        if ($this->validatesArrays() && is_array($data) && !Str::contains($attribute, '.')) {
             return Arr::has($data, $attribute) ? $data : [$attribute => $data];
         }
 
         if (Str::contains($attribute, '.')) {
             $validData = [];
 
-            Arr::set($validData, $attribute, ! is_array($data) ? $data : Arr::get($data, $attribute));
+            Arr::set($validData, $attribute, !is_array($data) ? $data : Arr::get($data, $attribute));
 
             return $validData;
         }
@@ -211,7 +210,7 @@ 
         $validationRuleAttribute = $this->getValidationAttributeString($attribute);
 
         $filesToValidate = Arr::get($items, $attribute);
-        $filesToValidate = is_array($filesToValidate) ? array_filter($filesToValidate, function ($item) {
+        $filesToValidate = is_array($filesToValidate) ? array_filter($filesToValidate, function($item) {
             return $item instanceof UploadedFile;
         }) : (is_a($filesToValidate, UploadedFile::class, true) ? [$filesToValidate] : []);
 

src/app/Library/Uploaders/MultipleFiles.php

Spacing +7 added lines, -7 removed lines

@@ -10,7 +10,7 @@ 
 
 class MultipleFiles extends Uploader
 {
-    public static function for(array $field, $configuration): UploaderInterface
+    public static function for (array $field, $configuration): UploaderInterface
     {
         return (new self($field, $configuration))->multiple();
     }
@@ -29,7 +29,7 @@ 
             $previousFiles = [];
         }
 
-        if (! is_array($previousFiles) && is_string($previousFiles)) {
+        if (!is_array($previousFiles) && is_string($previousFiles)) {
             $previousFiles = json_decode($previousFiles, true);
         }
 
@@ -38,14 +38,14 @@ 
                 if (in_array($previousFile, $filesToDelete)) {
                     Storage::disk($this->getDisk())->delete($previousFile);
 
-                    $previousFiles = Arr::where($previousFiles, function ($value, $key) use ($previousFile) {
+                    $previousFiles = Arr::where($previousFiles, function($value, $key) use ($previousFile) {
                         return $value != $previousFile;
                     });
                 }
             }
         }
 
-        if (! is_array($value)) {
+        if (!is_array($value)) {
             $value = [];
         }
 
@@ -82,13 +82,13 @@ 
         // create a temporary variable that we can unset keys
         // everytime one is found. That way we avoid iterating
         // already handled keys (notice we do a deep array copy)
-        $tempFileOrder = array_map(function ($item) {
+        $tempFileOrder = array_map(function($item) {
             return $item;
         }, $fileOrder);
 
         foreach ($previousRepeatableValues as $previousRow => $previousFiles) {
             foreach ($previousFiles ?? [] as $key => $file) {
-                $previousFileInArray = array_filter($tempFileOrder, function ($items, $key) use ($file, $tempFileOrder) {
+                $previousFileInArray = array_filter($tempFileOrder, function($items, $key) use ($file, $tempFileOrder) {
                     $found = array_search($file, $items ?? [], true);
                     if ($found !== false) {
                         Arr::forget($tempFileOrder, $key.'.'.$found);
@@ -98,7 +98,7 @@ 
 
                     return false;
                 }, ARRAY_FILTER_USE_BOTH);
-                if ($file && ! $previousFileInArray) {
+                if ($file && !$previousFileInArray) {
                     Storage::disk($this->getDisk())->delete($file);
                 }
             }