Laravel-Backpack / CRUD

tests/config/database/migrations/2017_09_08_000001_create_account_details_table.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      */
     public function up()
     {
-        Schema::create('account_details', function (Blueprint $table) {
+        Schema::create('account_details', function(Blueprint $table) {
             $table->increments('id');
             $table->integer('user_id')->unsigned();
             $table->string('nickname');

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

Spacing +9 added lines, -10 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']));
         }
 
@@ -82,11 +82,11 @@ 
     {
         // if it's got a method on the model with the same name
         // then it should be a relationship
-        if (! isset($column['type']) && method_exists($this->model, $column['name'])) {
+        if (!isset($column['type']) && method_exists($this->model, $column['name'])) {
             $column['type'] = 'relationship';
         }
 
-        if (! isset($column['type'])) {
+        if (!isset($column['type'])) {
             $column['type'] = 'text';
         }
 
@@ -103,7 +103,7 @@ 
      */
     protected function makeSureColumnHasKey($column)
     {
-        if (! isset($column['key'])) {
+        if (!isset($column['key'])) {
             $column['key'] = str_replace('.', '__', $column['name']);
         }
 
@@ -121,7 +121,7 @@ 
      */
     protected function makeSureColumnHasWrapper($column)
     {
-        if (! isset($column['wrapper'])) {
+        if (!isset($column['wrapper'])) {
             $column['wrapper'] = [];
         }
 
@@ -139,7 +139,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']) && ! isset($column['model'])) {
+        if (isset($column['entity']) && !isset($column['model'])) {
             $column['model'] = $this->getRelationModel($column['entity']);
         }
 
@@ -160,8 +160,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);
@@ -184,7 +183,7 @@ 
     {
         static $cache = [];
 
-        if (! $this->driverIsSql()) {
+        if (!$this->driverIsSql()) {
             return true;
         }
 

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

Spacing +7 added lines, -7 removed lines

@@ -16,7 +16,7 @@ 
             $this->getDbColumnTypes();
         }
 
-        array_map(function ($field) {
+        array_map(function($field) {
             $new_field = [
                 'name'       => $field,
                 'label'      => $this->makeLabel($field),
@@ -28,11 +28,11 @@ 
                 'autoset'    => true,
             ];
 
-            if (! isset($this->fields()[$field])) {
+            if (!isset($this->fields()[$field])) {
                 $this->addField($new_field);
             }
 
-            if (! in_array($field, $this->model->getHidden()) && ! in_array($field, $this->columns())) {
+            if (!in_array($field, $this->model->getHidden()) && !in_array($field, $this->columns())) {
                 $this->addColumn([
                     'name'    => $field,
                     'label'   => $this->makeLabel($field),
@@ -110,7 +110,7 @@ 
 
         $dbColumnTypes = $this->getDbColumnTypes();
 
-        if (! isset($dbColumnTypes[$fieldName])) {
+        if (!isset($dbColumnTypes[$fieldName])) {
             return 'text';
         }
 
@@ -168,7 +168,7 @@ 
         $types = ['enum' => 'string'];
         $platform = $this->getSchema()->getConnection()->getDoctrineConnection()->getDatabasePlatform();
         foreach ($types as $type_key => $type_value) {
-            if (! $platform->hasDoctrineTypeMappingFor($type_key)) {
+            if (!$platform->hasDoctrineTypeMappingFor($type_key)) {
                 $platform->registerDoctrineTypeMapping($type_key, $type_value);
             }
         }
@@ -221,13 +221,13 @@ 
     {
         $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->getConnection()->getSchemaBuilder()->getColumnListing($this->model->getTable());
 
-            if (! empty($fillable)) {
+            if (!empty($fillable)) {
                 $columns = array_intersect($columns, $fillable);
             }
         }

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

Spacing +4 added lines, -4 removed lines

@@ -41,7 +41,7 @@ 
     {
         $this->crud->allowAccess('list');
 
-        $this->crud->operation('list', function () {
+        $this->crud->operation('list', function() {
             $this->crud->loadDefaultOperationSettingsFromConfig();
         });
     }
@@ -114,8 +114,8 @@ 
         // if there was an order set, this will be the last one (after all others were applied)
         $orderBy = $this->crud->query->getQuery()->orders;
         $hasOrderByPrimaryKey = false;
-        collect($orderBy)->each(function ($item, $key) use ($hasOrderByPrimaryKey) {
-            if (! isset($item['column'])) {
+        collect($orderBy)->each(function($item, $key) use ($hasOrderByPrimaryKey) {
+            if (!isset($item['column'])) {
                 return false;
             }
 
@@ -125,7 +125,7 @@ 
                 return false;
             }
         });
-        if (! $hasOrderByPrimaryKey) {
+        if (!$hasOrderByPrimaryKey) {
             $this->crud->orderByWithPrefix($this->crud->model->getKeyName(), 'DESC');
         }
 

src/app/Models/Traits/HasRelationshipFields.php

Spacing +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
         $table = $instance->getTableWithPrefix();
 
         // MongoDB columns are alway nullable
-        if (! in_array($conn->getConfig()['driver'], CRUD::getSqlDriverList())) {
+        if (!in_array($conn->getConfig()['driver'], CRUD::getSqlDriverList())) {
             return true;
         }
 
@@ -82,7 +82,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;
         }

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

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
         return collect($this->fields())
             ->where('model')
             ->whereIn('relation_type', $relation_types)
-            ->filter(function ($item) {
+            ->filter(function($item) {
                 $related_model = get_class($this->model->{Arr::first(explode('.', $item['entity']))}()->getRelated());
 
                 return Str::contains($item['entity'], '.') && $item['model'] !== $related_model ? false : true;

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

Spacing +5 added lines, -5 removed lines

@@ -54,7 +54,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) {
@@ -67,7 +67,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(),
@@ -93,7 +93,7 @@ 
 
             $relation_array = explode('.', $relational_entity);
 
-            $relatedModel = array_reduce(array_splice($relation_array, 0, -1), function ($obj, $method) {
+            $relatedModel = array_reduce(array_splice($relation_array, 0, -1), function($obj, $method) {
                 return $obj->{$method} ? $obj->{$method} : $obj;
             }, $model);
 
@@ -110,7 +110,7 @@ 
                     case 'HasMany':
                     case 'MorphMany':
                         if (isset($field['pivotFields']) && is_array($field['pivotFields'])) {
-                            $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 = $relatedModel->{$relationMethod};
@@ -136,7 +136,7 @@ 
                         if (isset($field['pivot']) && $field['pivot'] && isset($field['pivotFields']) && is_array($field['pivotFields'])) {
 
                             // we remove our current relation from the pivotFields.
-                            $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'];
                             });
 

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

Indentation +1 added lines, -1 removed lines

@@ -385,7 +385,7 @@
         if (! is_null($relation_values) && $relationData['values'][$relationMethod][0] !== null) {
             //we add the new values into the relation
             $model_instance->whereIn($model_instance->getKeyName(), $relation_values)
-           ->update([$relation_foreign_key => $item->{$relation_local_key}]);
+            ->update([$relation_foreign_key => $item->{$relation_local_key}]);
 
             //we clear up any values that were removed from model relation.
             //if developer provided a fallback id, we use it

Spacing +20 added lines, -20 removed lines

@@ -91,7 +91,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'];
         });
     }
@@ -150,7 +150,7 @@ 
                             $field_data = json_decode($data[$field['name']], true);
 
                             //we grab from the parsed data the specific values for this pivot
-                            $pivot_data = Arr::first(Arr::where($field_data, function ($item) use ($pivot_id, $field) {
+                            $pivot_data = Arr::first(Arr::where($field_data, function($item) use ($pivot_id, $field) {
                                 return $item[$field['name']] === $pivot_id;
                             }));
 
@@ -194,12 +194,12 @@ 
      */
     private function createRelationsForItem($item, $formattedData)
     {
-        if (! isset($formattedData['relations'])) {
+        if (!isset($formattedData['relations'])) {
             return false;
         }
 
         foreach ($formattedData['relations'] as $relationMethod => $relationData) {
-            if (! isset($relationData['model'])) {
+            if (!isset($relationData['model'])) {
                 continue;
             }
 
@@ -211,14 +211,14 @@ 
                 case 'MorphOne':
                     // we first check if there are relations of the relation
                     if (isset($relationData['relations'])) {
-                        $belongsToRelations = Arr::where($relationData['relations'], function ($relation_data) {
+                        $belongsToRelations = Arr::where($relationData['relations'], function($relation_data) {
                             return $relation_data['relation_type'] == 'BelongsTo';
                         });
                         // adds the values of the BelongsTo relations of this entity to the array of values that will
                         // be saved at the same time like we do in parent entity belongs to relations
                         $valuesWithRelations = $this->associateHasOneBelongsTo($belongsToRelations, $relationData['values'], $relation->getModel());
 
-                        $relationData['relations'] = Arr::where($relationData['relations'], function ($item) {
+                        $relationData['relations'] = Arr::where($relationData['relations'], function($item) {
                             return $item['relation_type'] != 'BelongsTo';
                         });
 
@@ -231,14 +231,14 @@ 
 
             if ($relation instanceof HasOne || $relation instanceof MorphOne) {
                 if (isset($relationData['relations'])) {
-                    $belongsToRelations = Arr::where($relationData['relations'], function ($relation_data) {
+                    $belongsToRelations = Arr::where($relationData['relations'], function($relation_data) {
                         return $relation_data['relation_type'] == 'BelongsTo';
                     });
                     // adds the values of the BelongsTo relations of this entity to the array of values that will
                     // be saved at the same time like we do in parent entity belongs to relations
                     $valuesWithRelations = $this->associateHasOneBelongsTo($belongsToRelations, $relationData['values'], $relation->getModel());
 
-                    $relationData['relations'] = Arr::where($relationData['relations'], function ($item) {
+                    $relationData['relations'] = Arr::where($relationData['relations'], function($item) {
                         return $item['relation_type'] != 'BelongsTo';
                     });
 
@@ -297,7 +297,7 @@ 
     private function getRelationDataFromFormData($data)
     {
         // exclude the already attached belongs to relations but include nested belongs to.
-        $relation_fields = Arr::where($this->getRelationFields(), function ($field, $key) {
+        $relation_fields = Arr::where($this->getRelationFields(), function($field, $key) {
             return $field['relation_type'] !== 'BelongsTo' || $this->isNestedRelation($field);
         });
 
@@ -308,10 +308,10 @@ 
             if (isset($relation_field['pivot']) && $relation_field['pivot'] !== true) {
                 $key = implode('.relations.', explode('.', $this->getOnlyRelationEntity($relation_field)));
                 $fieldData = Arr::get($relationData, 'relations.'.$key, []);
-                if (! array_key_exists('model', $fieldData)) {
+                if (!array_key_exists('model', $fieldData)) {
                     $fieldData['model'] = $relation_field['model'];
                 }
-                if (! array_key_exists('parent', $fieldData)) {
+                if (!array_key_exists('parent', $fieldData)) {
                     $fieldData['parent'] = $this->getRelationModel($attributeKey, -1);
                 }
 
@@ -325,7 +325,7 @@ 
                 // this allow developers to override that behavior and force deletion from database
                 $fieldData['force_delete'] = $relation_field['force_delete'] ?? false;
 
-                if (! array_key_exists('relation_type', $fieldData)) {
+                if (!array_key_exists('relation_type', $fieldData)) {
                     $fieldData['relation_type'] = $relation_field['relation_type'];
                 }
                 $relatedAttribute = Arr::last(explode('.', $attributeKey));
@@ -353,7 +353,7 @@ 
 
         $related_method = Arr::last($entity_array);
 
-        if (! method_exists($relation_model, $related_method)) {
+        if (!method_exists($relation_model, $related_method)) {
             if (count($entity_array) <= 1) {
                 return $relation_field['entity'];
             } else {
@@ -382,7 +382,7 @@ 
 
         $relation_column_is_nullable = $model_instance->isColumnNullable($relation_foreign_key);
 
-        if (! is_null($relation_values) && $relationData['values'][$relationMethod][0] !== null) {
+        if (!is_null($relation_values) && $relationData['values'][$relationMethod][0] !== null) {
             //we add the new values into the relation
             $model_instance->whereIn($model_instance->getKeyName(), $relation_values)
            ->update([$relation_foreign_key => $item->{$relation_local_key}]);
@@ -396,7 +396,7 @@ 
                             ->where($relation_foreign_key, $item->{$relation_local_key})
                             ->update([$relation_foreign_key => $relationData['fallback_id']]);
             } else {
-                if (! $relation_column_is_nullable || $force_delete) {
+                if (!$relation_column_is_nullable || $force_delete) {
                     $model_instance->whereNotIn($model_instance->getKeyName(), $relation_values)
                             ->where($relation_foreign_key, $item->{$relation_local_key})
                             ->delete();
@@ -413,7 +413,7 @@ 
                 $model_instance->where($relation_foreign_key, $item->{$relation_local_key})
                             ->update([$relation_foreign_key => $relationData['fallback_id']]);
             } else {
-                if (! $relation_column_is_nullable || $force_delete) {
+                if (!$relation_column_is_nullable || $force_delete) {
                     $model_instance->where($relation_foreign_key, $item->{$relation_local_key})->delete();
                 } else {
                     $model_instance->where($relation_foreign_key, $item->{$relation_local_key})
@@ -439,7 +439,7 @@ 
         if ($items->isEmpty()) {
             $entry->{$relationMethod}()->sync([]);
         } else {
-            $items->each(function (&$item, $key) use ($relatedModel, $entry, $relationMethod) {
+            $items->each(function(&$item, $key) use ($relatedModel, $entry, $relationMethod) {
                 if (isset($item[$relatedModel->getKeyName()])) {
                     $entry->{$relationMethod}()->updateOrCreate([$relatedModel->getKeyName() => $item[$relatedModel->getKeyName()]], $item);
                 } else {
@@ -449,11 +449,11 @@ 
 
             $relatedItemsSent = $items->pluck($relatedModel->getKeyName());
 
-            if (! $relatedItemsSent->isEmpty()) {
+            if (!$relatedItemsSent->isEmpty()) {
                 $itemsInDatabase = $entry->{$relationMethod};
                 //we perform the cleanup of removed database items
-                $itemsInDatabase->each(function ($item, $key) use ($relatedItemsSent) {
-                    if (! $relatedItemsSent->contains($item->getKey())) {
+                $itemsInDatabase->each(function($item, $key) use ($relatedItemsSent) {
+                    if (!$relatedItemsSent->contains($item->getKey())) {
                         $item->delete();
                     }
                 });