Laravel-Backpack / CRUD

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/Library/Uploaders/Uploader.php

Spacing +8 added lines, -8 removed lines

@@ -25,7 +25,7 @@ 
 
     private bool $deleteWhenEntryIsDeleted = true;
 
-    private bool|string $attachedToFakeField = false;
+    private bool | string $attachedToFakeField = false;
 
     /**
      * Cloud disks have the ability to generate temporary URLs to files, should we do it?
@@ -62,7 +62,7 @@ 
     /*******************************
      * Static methods
      *******************************/
-    public static function for(array $crudObject, array $definition): UploaderInterface
+    public static function for (array $crudObject, array $definition): UploaderInterface
     {
         return new static($crudObject, $definition);
     }
@@ -103,7 +103,7 @@ 
     public function deleteUploadedFiles(Model $entry): void
     {
         if ($this->deleteWhenEntryIsDeleted) {
-            if (! in_array(SoftDeletes::class, class_uses_recursive($entry), true)) {
+            if (!in_array(SoftDeletes::class, class_uses_recursive($entry), true)) {
                 $this->performFileDeletion($entry);
 
                 return;
@@ -179,7 +179,7 @@ 
 
     public function getPreviousFiles(Model $entry): mixed
     {
-        if (! $this->attachedToFakeField) {
+        if (!$this->attachedToFakeField) {
             return $this->getOriginalValue($entry);
         }
 
@@ -218,7 +218,7 @@ 
         $value = $entry->{$this->getAttributeName()};
 
         if ($this->handleMultipleFiles) {
-            if (! isset($entry->getCasts()[$this->getName()]) && is_string($value)) {
+            if (!isset($entry->getCasts()[$this->getName()]) && is_string($value)) {
                 $entry->{$this->getAttributeName()} = json_decode($value, true);
             }
 
@@ -250,7 +250,7 @@ 
 
         if ($this->handleMultipleFiles) {
             // ensure we have an array of values when field is not casted in model.
-            if (! isset($entry->getCasts()[$this->name]) && is_string($values)) {
+            if (!isset($entry->getCasts()[$this->name]) && is_string($values)) {
                 $values = json_decode($values, true);
             }
             foreach ($values ?? [] as $value) {
@@ -267,7 +267,7 @@ 
 
     private function performFileDeletion(Model $entry)
     {
-        if (! $this->handleRepeatableFiles) {
+        if (!$this->handleRepeatableFiles) {
             $this->deleteFiles($entry);
 
             return;
@@ -296,7 +296,7 @@ 
 
         $previousValue = $entry->getOriginal($field);
 
-        if (! $previousValue) {
+        if (!$previousValue) {
             return $previousValue;
         }
 

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/Database/DatabaseSchema.php

Spacing +6 added lines, -7 removed lines

@@ -26,7 +26,7 @@ 
     {
         $connection = $connection ?: config('database.default');
 
-        self::$schema[$connection] = LazyCollection::make(self::getCreateSchema($connection)->getTables())->mapWithKeys(function ($table, $key) use ($connection) {
+        self::$schema[$connection] = LazyCollection::make(self::getCreateSchema($connection)->getTables())->mapWithKeys(function($table, $key) use ($connection) {
             $tableName = is_array($table) ? $table['name'] : $table->getName();
 
             if ($existingTable = self::$schema[$connection][$tableName] ?? false) {
@@ -65,7 +65,7 @@ 
      */
     private static function generateDatabaseSchema(string $connection, string $table)
     {
-        if (! isset(self::$schema[$connection][$table])) {
+        if (!isset(self::$schema[$connection][$table])) {
             self::$schema[$connection][$table] = self::mapTable($connection, $table);
         }
     }
@@ -74,20 +74,19 @@ 
     {
         try {
             $table = method_exists(self::getCreateSchema($connection), 'getTable') ?
-                        self::getCreateSchema($connection)->getTable($tableName) :
-                        self::getCreateSchema($connection)->getColumns($tableName);
+                        self::getCreateSchema($connection)->getTable($tableName) : self::getCreateSchema($connection)->getColumns($tableName);
         } catch (\Exception $e) {
             return new Table($tableName, []);
         }
 
-        if (! is_array($table) || empty($table)) {
+        if (!is_array($table) || empty($table)) {
             return $table;
         }
 
         $schemaManager = self::getSchemaManager($connection);
         $indexes = $schemaManager->getIndexes($tableName);
 
-        $indexes = array_map(function ($index) {
+        $indexes = array_map(function($index) {
             return $index['columns'];
         }, $indexes);
 
@@ -105,7 +104,7 @@ 
 
         $indexes = self::$schema[$connection][$table]->getIndexes();
 
-        $indexes = \Illuminate\Support\Arr::flatten(array_map(function ($index) {
+        $indexes = \Illuminate\Support\Arr::flatten(array_map(function($index) {
             return is_string($index) ? $index : $index->getColumns();
         }, $indexes));
 

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

Spacing +14 added lines, -14 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.');
             }
 
@@ -278,7 +278,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';
@@ -305,15 +305,15 @@ 
                 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;
                     }
 
-                    if($field['allow_multiple_pivots'] ?? false) {
+                    if ($field['allow_multiple_pivots'] ?? false) {
                         $pivotSelectorField['allow_multiple_pivots'] = true;
                         $field['subfields'] = Arr::prepend($field['subfields'], [
                                 'name' => $field['pivot_key_name'] ?? 'id',
@@ -336,11 +336,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;
                     }
 
@@ -365,7 +365,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/Create.php

Indentation +1 added lines, -1 removed lines

@@ -154,7 +154,7 @@
                                 if(isset($value[$keyName])) {
                                     $relation->newPivot()->where($keyName, $value[$keyName])->update($this->preparePivotAttributesForUpdate($value, $relation));
                                 } else {
-                                   $relation->newPivot()->create($this->preparePivotAttributesForCreate($value, $relation, $item->getKey()));
+                                    $relation->newPivot()->create($this->preparePivotAttributesForCreate($value, $relation, $item->getKey()));
                                 }
                             }
                             break;

Spacing +8 added lines, -8 removed lines

@@ -123,7 +123,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);
@@ -144,14 +144,14 @@ 
                             $dbValues = $relation->get()->pluck($keyName)->all();
                             $toDelete = array_diff($dbValues, $sentIds);
 
-                            if (! empty($toDelete)) {
-                                foreach($toDelete as $id) {
+                            if (!empty($toDelete)) {
+                                foreach ($toDelete as $id) {
                                     $relation->newPivot()->where($keyName, $id)->delete();
                                 }
                             }
-                            foreach($values as $value) { 
+                            foreach ($values as $value) { 
                                 // if it's an existing pivot, update it
-                                if(isset($value[$keyName])) {
+                                if (isset($value[$keyName])) {
                                     $relation->newPivot()->where($keyName, $value[$keyName])->update($this->preparePivotAttributesForUpdate($value, $relation));
                                 } else {
                                    $relation->newPivot()->create($this->preparePivotAttributesForCreate($value, $relation, $item->getKey()));
@@ -184,7 +184,7 @@ 
         }
     }
 
-    private function preparePivotAttributesForCreate(array $attributes, BelongsToMany $relation, string|int $relatedItemKey) {
+    private function preparePivotAttributesForCreate(array $attributes, BelongsToMany $relation, string | int $relatedItemKey) {
         $attributes[$relation->getForeignPivotKeyName()] = $relatedItemKey;
         $attributes[$relation->getRelatedPivotKeyName()] = $attributes[$relation->getRelationName()];
         $pivotKeyName = $attributes['pivot_key_name'] ?? 'id';
@@ -331,7 +331,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();
         }
 
@@ -376,7 +376,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();
         }

tests/Unit/CrudPanel/CrudPanelCreateTest.php

Spacing +3 added lines, -3 removed lines

@@ -759,7 +759,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',
@@ -895,7 +895,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',
@@ -1645,7 +1645,7 @@ 
     {
         $faker = Factory::create();
 
-        if($initCrud) {
+        if ($initCrud) {
             $this->crudPanel->setModel(User::class);
             $this->crudPanel->addFields($this->userInputFieldsNoRelationships);
             $this->crudPanel->addField([