Laravel-Backpack / CRUD

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

Spacing +7 added lines, -8 removed lines

@@ -49,7 +49,7 @@ 
             $column = ['name' => $column];
         }
 
-        if (is_array($column) && ! isset($column['name'])) {
+        if (is_array($column) && !isset($column['name'])) {
             $column['name'] = 'anonymous_column_'.Str::random(5);
         }
 
@@ -65,7 +65,7 @@ 
      */
     protected function makeSureColumnHasLabel($column)
     {
-        if (! isset($column['label'])) {
+        if (!isset($column['label'])) {
             $column['label'] = mb_ucfirst($this->makeLabel($column['name']));
         }
 
@@ -146,7 +146,7 @@ 
      */
     protected function makeSureColumnHasKey($column)
     {
-        if (! isset($column['key'])) {
+        if (!isset($column['key'])) {
             $column['key'] = str_replace('.', '__', $column['name']);
         }
 
@@ -164,7 +164,7 @@ 
      */
     protected function makeSureColumnHasWrapper($column)
     {
-        if (! isset($column['wrapper'])) {
+        if (!isset($column['wrapper'])) {
             $column['wrapper'] = [];
         }
 
@@ -255,7 +255,7 @@ 
     {
         // if this is a relation type field and no corresponding model was specified,
         // get it from the relation method defined in the main model
-        if (isset($column['entity']) && $column['entity'] !== false && ! isset($column['model'])) {
+        if (isset($column['entity']) && $column['entity'] !== false && !isset($column['model'])) {
             $column['model'] = $this->getRelationModel($column['entity']);
         }
 
@@ -276,8 +276,7 @@ 
         $columnsArray = $this->columns();
 
         if (array_key_exists($targetColumnName, $columnsArray)) {
-            $targetColumnPosition = $before ? array_search($targetColumnName, array_keys($columnsArray)) :
-                array_search($targetColumnName, array_keys($columnsArray)) + 1;
+            $targetColumnPosition = $before ? array_search($targetColumnName, array_keys($columnsArray)) : array_search($targetColumnName, array_keys($columnsArray)) + 1;
 
             $element = array_pop($columnsArray);
             $beginningPart = array_slice($columnsArray, 0, $targetColumnPosition, true);
@@ -299,7 +298,7 @@ 
     {
         static $cache = [];
 
-        if (! $this->driverIsSql()) {
+        if (!$this->driverIsSql()) {
             return true;
         }
 

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

Spacing +6 added lines, -6 removed lines

@@ -30,25 +30,25 @@
         $this->crud->allowAccess('show');
         $this->crud->setOperationSetting('setFromDb', true);
 
-        $this->crud->operation('show', function () {
+        $this->crud->operation('show', function() {
             $this->crud->loadDefaultOperationSettingsFromConfig();
 
-            if (! method_exists($this, 'setupShowOperation')) {
+            if (!method_exists($this, 'setupShowOperation')) {
                 $this->autoSetupShowOperation();
             }
         });
 
-        $this->crud->operation('list', function () {
+        $this->crud->operation('list', function() {
             $this->crud->addButton('line', 'show', 'view', 'crud::buttons.show', 'beginning');
         });
 
-        $this->crud->operation(['create', 'update'], function () {
+        $this->crud->operation(['create', 'update'], function() {
             $this->crud->addSaveAction([
                 'name' => 'save_and_preview',
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return $crud->hasAccess('show');
                 },
-                'redirect' => function ($crud, $request, $itemId = null) {
+                'redirect' => function($crud, $request, $itemId = null) {
                     $itemId = $itemId ?: $request->input('id');
                     $redirectUrl = $crud->route.'/'.$itemId.'/show';
                     if ($request->has('_locale')) {

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

Spacing +7 added lines, -7 removed lines

@@ -53,7 +53,7 @@ 
 
         foreach ($fields as &$field) {
             // set the value
-            if (! isset($field['value'])) {
+            if (!isset($field['value'])) {
                 if (isset($field['subfields'])) {
                     $field['value'] = [];
                     foreach ($field['subfields'] as $subfield) {
@@ -66,7 +66,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(),
@@ -108,7 +108,7 @@ 
     {
         [$related_model, $relation_method] = $this->getModelAndMethodFromEntity($model, $field);
 
-        if (! method_exists($related_model, $relation_method)) {
+        if (!method_exists($related_model, $relation_method)) {
             return $related_model->{$relation_method};
         }
 
@@ -119,11 +119,11 @@ 
             case 'HasMany':
             case 'BelongsToMany':
             case 'MorphToMany':
-                if (! isset($field['pivotFields'])) {
+                if (!isset($field['pivotFields'])) {
                     return $related_model->{$relation_method};
                 }
                 // we want to exclude the "self pivot field" since we already have it.
-                $pivot_fields = Arr::where($field['pivotFields'], function ($item) use ($field) {
+                $pivot_fields = Arr::where($field['pivotFields'], function($item) use ($field) {
                     return $field['name'] != $item['name'];
                 });
                 $related_models = $related_model->{$relation_method};
@@ -160,7 +160,7 @@ 
             break;
             case 'HasOne':
             case 'MorphOne':
-                if (! $related_model->{$relation_method}) {
+                if (!$related_model->{$relation_method}) {
                     return;
                 }
 
@@ -179,7 +179,7 @@ 
 
         $relation_array = explode('.', $relational_entity);
 
-        $related_model = array_reduce(array_splice($relation_array, 0, -1), function ($obj, $method) {
+        $related_model = array_reduce(array_splice($relation_array, 0, -1), function($obj, $method) {
             $method = Str::endsWith($method, '_id') ? Str::replaceLast('_id', '', $method) : $method;
 
             return $obj->{$method} ? $obj->{$method} : $obj;

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

Spacing +5 added lines, -5 removed lines

@@ -62,7 +62,7 @@ 
         $model = $this->getRelationModel($field['entity'], -1);
         $lastSegmentAfterDot = Str::of($field['entity'])->afterLast('.');
 
-        if (! method_exists($model, $lastSegmentAfterDot)) {
+        if (!method_exists($model, $lastSegmentAfterDot)) {
             return (string) Str::of($field['entity'])->beforeLast('.');
         }
 
@@ -83,7 +83,7 @@ 
         return collect($this->fields())
             ->where('model')
             ->whereIn('relation_type', $relation_types)
-            ->filter(function ($item) {
+            ->filter(function($item) {
                 $related_model = get_class($this->model->{Str::before($item['entity'], '.')}()->getRelated());
 
                 return Str::contains($item['entity'], '.') && $item['model'] !== $related_model ? false : true;
@@ -207,8 +207,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'];
         });
     }
 
@@ -221,7 +221,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'];
         });
     }

tests/config/database/migrations/2020_03_31_114745_create_pivotable_relations_tables.php

Spacing +10 added lines, -10 removed lines

@@ -13,14 +13,14 @@ 
      */
     public function up()
     {
-        Schema::create('comments', function (Blueprint $table) {
+        Schema::create('comments', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->string('text')->nullable();
             $table->morphs('commentable');
             $table->timestamps();
         });
 
-        Schema::create('recommendables', function (Blueprint $table) {
+        Schema::create('recommendables', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->string('text')->nullable();
             $table->morphs('recommendable');
@@ -28,7 +28,7 @@ 
             $table->timestamps();
         });
 
-        Schema::create('billables', function (Blueprint $table) {
+        Schema::create('billables', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->string('text')->nullable();
             $table->morphs('billable');
@@ -36,7 +36,7 @@ 
             $table->timestamps();
         });
 
-        Schema::create('articles_user', function (Blueprint $table) {
+        Schema::create('articles_user', function(Blueprint $table) {
             $table->increments('id');
             $table->integer('article_id')->unsigned();
             $table->integer('user_id')->unsigned();
@@ -44,38 +44,38 @@ 
             $table->nullableTimestamps();
         });
 
-        Schema::create('recommends', function (Blueprint $table) {
+        Schema::create('recommends', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->string('title')->nullable();
             $table->timestamps();
         });
 
-        Schema::create('bills', function (Blueprint $table) {
+        Schema::create('bills', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->string('title')->nullable();
             $table->timestamps();
         });
 
-        Schema::create('stars', function (Blueprint $table) {
+        Schema::create('stars', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->string('starable_type');
             $table->bigInteger('starable_id');
             $table->string('title')->nullable();
         });
 
-        Schema::create('universes', function (Blueprint $table) {
+        Schema::create('universes', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->bigInteger('user_id');
             $table->string('title')->nullable();
         });
 
-        Schema::create('planets', function (Blueprint $table) {
+        Schema::create('planets', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->bigInteger('user_id')->nullable();
             $table->string('title')->nullable();
         });
 
-        Schema::create('comets', function (Blueprint $table) {
+        Schema::create('comets', function(Blueprint $table) {
             $table->bigIncrements('id');
             $table->bigInteger('user_id')->default(0);
         });

src/app/Models/Traits/HasRelationshipFields.php

Spacing +3 added lines, -3 removed lines

@@ -67,7 +67,7 @@ 
         [$conn, $table] = self::getConnectionAndTable();
 
         // MongoDB columns are alway nullable
-        if (! in_array($conn->getConfig()['driver'], CRUD::getSqlDriverList())) {
+        if (!in_array($conn->getConfig()['driver'], CRUD::getSqlDriverList())) {
             return true;
         }
 
@@ -77,7 +77,7 @@ 
             // check for NOT NULL
             $notNull = $column->getNotnull();
             // return the value of nullable (aka the inverse of NOT NULL)
-            return ! $notNull;
+            return !$notNull;
         } catch (\Exception $e) {
             return true;
         }
@@ -94,7 +94,7 @@ 
         [$conn, $table] = self::getConnectionAndTable();
 
         // MongoDB columns don't have default values
-        if (! in_array($conn->getConfig()['driver'], CRUD::getSqlDriverList())) {
+        if (!in_array($conn->getConfig()['driver'], CRUD::getSqlDriverList())) {
             return false;
         }
 

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

Spacing +6 added lines, -6 removed lines

@@ -29,10 +29,10 @@ 
         // construct the validation rules array
         // (eg. ['name' => 'required|min:2'])
         $rules = collect($fields)
-                    ->filter(function ($value, $key) {
+                    ->filter(function($value, $key) {
                         // only keep fields where 'validationRules' attribute is defined
                         return array_key_exists('validationRules', $value);
-                    })->map(function ($item, $key) {
+                    })->map(function($item, $key) {
                         // only keep the rules, not the entire field definition
                         return $item['validationRules'];
                     })->toArray();
@@ -41,10 +41,10 @@ 
         // (eg. ['title.required' => 'You gotta write smth man.'])
         $messages = [];
         collect($fields)
-                    ->filter(function ($value, $key) {
+                    ->filter(function($value, $key) {
                         // only keep fields where 'validationMessages' attribute is defined
                         return array_key_exists('validationMessages', $value);
-                    })->each(function ($item, $key) use (&$messages) {
+                    })->each(function($item, $key) use (&$messages) {
                         foreach ($item['validationMessages'] as $rule => $message) {
                             $messages[$key.'.'.$rule] = $message;
                         }
@@ -74,7 +74,7 @@ 
      */
     public function setValidation($classOrRulesArray = false, $messages = [])
     {
-        if (! $classOrRulesArray) {
+        if (!$classOrRulesArray) {
             $this->setValidationFromFields();
         } elseif (is_array($classOrRulesArray)) {
             $this->setValidationFromArray($classOrRulesArray, $messages);
@@ -202,7 +202,7 @@ 
      */
     public function isRequired($inputKey)
     {
-        if (! $this->hasOperationSetting('requiredFields')) {
+        if (!$this->hasOperationSetting('requiredFields')) {
             return false;
         }
 

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

Spacing +8 added lines, -8 removed lines

@@ -101,7 +101,7 @@ 
         $fields_with_pivot = $this->getRelationFieldsWithPivot();
 
         //remove fields that are not in the submitted form input
-        $fields_with_pivot = array_filter($fields_with_pivot, function ($item) use ($input) {
+        $fields_with_pivot = array_filter($fields_with_pivot, function($item) use ($input) {
             return Arr::has($input, $item['name']);
         });
 
@@ -156,11 +156,11 @@ 
      */
     private function createRelationsForItem($item, $formattedRelations)
     {
-        if (! isset($formattedRelations['relations'])) {
+        if (!isset($formattedRelations['relations'])) {
             return false;
         }
         foreach ($formattedRelations['relations'] as $relationMethod => $relationDetails) {
-            if (! isset($relationDetails['model'])) {
+            if (!isset($relationDetails['model'])) {
                 continue;
             }
             $model = $relationDetails['model'];
@@ -243,7 +243,7 @@ 
             return $removed_entries->delete();
         }
 
-        if (! $relation_column_is_nullable && $model_instance->dbColumnHasDefault($relation_foreign_key)) {
+        if (!$relation_column_is_nullable && $model_instance->dbColumnHasDefault($relation_foreign_key)) {
             return $removed_entries->update([$relation_foreign_key => $model_instance->getDbColumnDefault($relation_foreign_key)]);
         }
 
@@ -266,7 +266,7 @@ 
         $created_ids = [];
 
         foreach ($items as $item) {
-            if (isset($item[$relation_local_key]) && ! empty($item[$relation_local_key])) {
+            if (isset($item[$relation_local_key]) && !empty($item[$relation_local_key])) {
                 $entry->{$relationMethod}()->updateOrCreate([$relation_local_key => $item[$relation_local_key]], Arr::except($item, $relation_local_key));
             } else {
                 $created_ids[] = $entry->{$relationMethod}()->create($item)->{$relation_local_key};
@@ -276,7 +276,7 @@ 
         // get from $items the sent ids, and merge the ones created.
         $relatedItemsSent = array_merge(array_filter(Arr::pluck($items, $relation_local_key)), $created_ids);
 
-        if (! empty($relatedItemsSent)) {
+        if (!empty($relatedItemsSent)) {
             // we perform the cleanup of removed database items
             $entry->{$relationMethod}()->whereNotIn($relation_local_key, $relatedItemsSent)->delete();
         }
@@ -306,12 +306,12 @@ 
         $relationFields = $this->getRelationFieldsWithoutPivot();
 
         //remove fields that are not in the submitted form input
-        $relationFields = array_filter($relationFields, function ($item) use ($input) {
+        $relationFields = array_filter($relationFields, function($item) use ($input) {
             return Arr::has($input, $item['name']);
         });
 
         // exclude the already attached belongs to relations in the main entry but include nested belongs to.
-        $relationFields = Arr::where($relationFields, function ($field, $key) {
+        $relationFields = Arr::where($relationFields, function($field, $key) {
             return $field['relation_type'] !== 'BelongsTo' || ($field['relation_type'] === 'BelongsTo' && Str::contains($field['name'], '.'));
         });