Laravel-Backpack / CRUD

src/helpers.php

Spacing +16 added lines, -16 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (! function_exists('backpack_url')) {
+if (!function_exists('backpack_url')) {
     /**
      * Appends the configured backpack prefix and returns
      * the URL using the standard Laravel helpers.
@@ -11,13 +11,13 @@ 
      */
     function backpack_url($path = null, $parameters = [], $secure = null)
     {
-        $path = ! $path || (substr($path, 0, 1) == '/') ? $path : '/'.$path;
+        $path = !$path || (substr($path, 0, 1) == '/') ? $path : '/'.$path;
 
         return url(config('backpack.base.route_prefix', 'admin').$path, $parameters, $secure);
     }
 }
 
-if (! function_exists('backpack_authentication_column')) {
+if (!function_exists('backpack_authentication_column')) {
     /**
      * Return the username column name.
      * The Laravel default (and Backpack default) is 'email'.
@@ -30,7 +30,7 @@ 
     }
 }
 
-if (! function_exists('backpack_form_input')) {
+if (!function_exists('backpack_form_input')) {
     /**
      * Parse the submitted input in request('form') to an usable array.
      * Joins the multiple[] fields in a single key and transform the dot notation fields into arrayed ones.
@@ -59,7 +59,7 @@ 
             }
 
             if (is_null($input_arg)) {
-                if (! isset($result[$input_key])) {
+                if (!isset($result[$input_key])) {
                     $result[$input_key] = $start ? [$row['value']] : $row['value'];
                 } else {
                     array_push($result[$input_key], $row['value']);
@@ -73,7 +73,7 @@ 
     }
 }
 
-if (! function_exists('backpack_users_have_email')) {
+if (!function_exists('backpack_users_have_email')) {
     /**
      * Check if the email column is present on the user table.
      *
@@ -88,7 +88,7 @@ 
     }
 }
 
-if (! function_exists('backpack_avatar_url')) {
+if (!function_exists('backpack_avatar_url')) {
     /**
      * Returns the avatar URL of a user.
      *
@@ -121,7 +121,7 @@ 
     }
 }
 
-if (! function_exists('backpack_middleware')) {
+if (!function_exists('backpack_middleware')) {
     /**
      * Return the key of the middleware used across Backpack.
      * That middleware checks if the visitor is an admin.
@@ -136,7 +136,7 @@ 
     }
 }
 
-if (! function_exists('backpack_guard_name')) {
+if (!function_exists('backpack_guard_name')) {
     /*
      * Returns the name of the guard defined
      * by the application config
@@ -147,7 +147,7 @@ 
     }
 }
 
-if (! function_exists('backpack_auth')) {
+if (!function_exists('backpack_auth')) {
     /*
      * Returns the user instance if it exists
      * of the currently authenticated admin
@@ -159,7 +159,7 @@ 
     }
 }
 
-if (! function_exists('backpack_user')) {
+if (!function_exists('backpack_user')) {
     /*
      * Returns back a user instance without
      * the admin guard, however allows you
@@ -171,7 +171,7 @@ 
     }
 }
 
-if (! function_exists('mb_ucfirst')) {
+if (!function_exists('mb_ucfirst')) {
     /**
      * Capitalize the first letter of a string,
      * even if that string is multi-byte (non-latin alphabet).
@@ -193,7 +193,7 @@ 
     }
 }
 
-if (! function_exists('backpack_view')) {
+if (!function_exists('backpack_view')) {
     /**
      * Returns a new displayable view based on the configured backpack view namespace.
      * If that view doesn't exist, it will load the one from the original theme.
@@ -213,7 +213,7 @@ 
 
         $returnView = $theme.$view;
 
-        if (! view()->exists($returnView)) {
+        if (!view()->exists($returnView)) {
             $returnView = $originalTheme.$view;
         }
 
@@ -221,7 +221,7 @@ 
     }
 }
 
-if (! function_exists('square_brackets_to_dots')) {
+if (!function_exists('square_brackets_to_dots')) {
     /**
      * Turns a string from bracket-type array to dot-notation array.
      * Ex: array[0][property] turns into array.0.property.
@@ -238,7 +238,7 @@ 
     }
 }
 
-if (! function_exists('is_countable')) {
+if (!function_exists('is_countable')) {
     /**
      * We need this because is_countable was only introduced in PHP 7.3,
      * and in PHP 7.2 you should check if count() argument is really countable.

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

Switch Indentation +8 added lines, -8 removed lines

@@ -184,15 +184,15 @@
                         $item[$related_model->getKeyName()] = $related_model->getKey();
                         $return[] = $item;
             break;
-            case BelongsToMany::class:
-            case MorphToMany::class:
+                case BelongsToMany::class:
+                case MorphToMany::class:
 
-                    $item[$field['name']] = $related_model->getKey();
-                    //for any given related model, we get the pivot fields.
-                    foreach ($pivot_fields as $pivot_field) {
-                        $item[$pivot_field['name']] = $related_model->pivot->{$pivot_field['name']};
-                    }
-                    $return[] = $item;
+                        $item[$field['name']] = $related_model->getKey();
+                        //for any given related model, we get the pivot fields.
+                        foreach ($pivot_fields as $pivot_field) {
+                            $item[$pivot_field['name']] = $related_model->pivot->{$pivot_field['name']};
+                        }
+                        $return[] = $item;
             }
         }
         // we return the json encoded result as expected by repeatable field.

Spacing +5 added lines, -5 removed lines

@@ -60,7 +60,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) {
@@ -73,7 +73,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(),
@@ -99,7 +99,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);
 
@@ -162,11 +162,11 @@ 
      */
     private function getManyRelationAttributeValue($model, $relation_method, $field, $relation_type)
     {
-        if (! isset($field['pivotFields']) || ! is_array($field['pivotFields'])) {
+        if (!isset($field['pivotFields']) || !is_array($field['pivotFields'])) {
             return false;
         }
 
-        $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 = $model->{$relation_method};

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

Spacing +8 added lines, -8 removed lines

@@ -114,7 +114,7 @@ 
      */
     public function makeFirstColumn()
     {
-        if (! $this->columns()) {
+        if (!$this->columns()) {
             return false;
         }
 
@@ -159,7 +159,7 @@ 
      */
     public function removeColumns($columns)
     {
-        if (! empty($columns)) {
+        if (!empty($columns)) {
             foreach ($columns as $columnKey) {
                 $this->removeColumn($columnKey);
             }
@@ -252,8 +252,8 @@ 
     {
         $columns = $this->columns();
 
-        return collect($columns)->pluck('entity')->reject(function ($value, $key) {
-            return ! $value;
+        return collect($columns)->pluck('entity')->reject(function($value, $key) {
+            return !$value;
         })->toArray();
     }
 
@@ -329,7 +329,7 @@ 
      */
     public function hasColumnWhere($attribute, $value)
     {
-        $match = Arr::first($this->columns(), function ($column, $columnKey) use ($attribute, $value) {
+        $match = Arr::first($this->columns(), function($column, $columnKey) use ($attribute, $value) {
             return isset($column[$attribute]) && $column[$attribute] == $value;
         });
 
@@ -345,7 +345,7 @@ 
      */
     public function firstColumnWhere($attribute, $value)
     {
-        return Arr::first($this->columns(), function ($column, $columnKey) use ($attribute, $value) {
+        return Arr::first($this->columns(), function($column, $columnKey) use ($attribute, $value) {
             return isset($column[$attribute]) && $column[$attribute] == $value;
         });
     }
@@ -395,8 +395,8 @@ 
      */
     public function countColumnsWithoutActions()
     {
-        return collect($this->columns())->filter(function ($column, $key) {
-            return ! isset($column['hasActions']) || $column['hasActions'] == false;
+        return collect($this->columns())->filter(function($column, $key) {
+            return !isset($column['hasActions']) || $column['hasActions'] == false;
         })->count();
     }
 

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()->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);
             }
         }
@@ -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/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 @@ 
     {
         $could_be_relation = isset($column['entity']) && $column['entity'] !== false;
 
-        if (! isset($column['type']) && $could_be_relation) {
+        if (!isset($column['type']) && $could_be_relation) {
             $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'] = [];
         }
 
@@ -185,7 +185,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']);
         }
 
@@ -206,8 +206,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);
@@ -230,7 +229,7 @@ 
     {
         static $cache = [];
 
-        if (! $this->driverIsSql()) {
+        if (!$this->driverIsSql()) {
             return true;
         }
 

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

Spacing +6 added lines, -8 removed lines

@@ -46,8 +46,8 @@ 
         // otherwise assume the arguments are actually the configuration array
         $config = [];
 
-        if (! is_array($arg)) {
-            if (! class_exists($arg)) {
+        if (!is_array($arg)) {
+            if (!class_exists($arg)) {
                 return response()->json(['error' => 'Class: '.$arg.' does not exists'], 500);
             }
             $config['model'] = $arg;
@@ -77,8 +77,7 @@ 
 
         if ($search_string === false) {
             return ($config['paginate'] !== false) ?
-            $config['query']->paginate($config['paginate']) :
-            $config['query']->get();
+            $config['query']->paginate($config['paginate']) : $config['query']->get();
         }
 
         $textColumnTypes = ['string', 'json_string', 'text', 'longText', 'json_array', 'json'];
@@ -89,8 +88,8 @@ 
         // .... 'query' => function($model) { return $model->where('active', 1); }
         // So it reads: SELECT ... WHERE active = 1 AND (XXX = x OR YYY = y) and not SELECT ... WHERE active = 1 AND XXX = x OR YYY = y;
 
-        if (! empty($config['query']->getQuery()->wheres)) {
-            $config['query'] = $config['query']->where(function ($query) use ($model_instance, $config, $search_string, $textColumnTypes) {
+        if (!empty($config['query']->getQuery()->wheres)) {
+            $config['query'] = $config['query']->where(function($query) use ($model_instance, $config, $search_string, $textColumnTypes) {
                 foreach ((array) $config['searchable_attributes'] as $k => $searchColumn) {
                     $operation = ($k == 0) ? 'where' : 'orWhere';
                     $columnType = $model_instance->getColumnType($searchColumn);
@@ -122,7 +121,6 @@ 
 
         // return the results with or without pagination
         return ($config['paginate'] !== false) ?
-                    $config['query']->paginate($config['paginate']) :
-                    $config['query']->get();
+                    $config['query']->paginate($config['paginate']) : $config['query']->get();
     }
 }

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

Spacing +2 added lines, -2 removed lines

@@ -98,7 +98,7 @@ 
                         // Convert dot to array notation
                         $key = \Str::of($key)
                             ->explode('.')
-                            ->map(function ($value, $key) {
+                            ->map(function($value, $key) {
                                 return $key ? "[$value]" : $value;
                             })
                             ->join('');
@@ -123,7 +123,7 @@ 
      */
     public function isRequired($inputKey)
     {
-        if (! $this->hasOperationSetting('requiredFields')) {
+        if (!$this->hasOperationSetting('requiredFields')) {
             return false;
         }
 

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

Indentation +1 added lines, -1 removed lines

@@ -375,7 +375,7 @@
         if ($relation_values !== null && $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 +14 added lines, -14 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'];
         });
     }
@@ -155,7 +155,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($field_data, function ($item) use ($pivot_id, $field) {
+                            $pivot_data = Arr::first($field_data, function($item) use ($pivot_id, $field) {
                                 return $item[$field['name']] === $pivot_id;
                             });
 
@@ -188,12 +188,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;
             }
 
@@ -206,7 +206,7 @@ 
                     // we first check if there are relations of the relation
                     if (isset($relationData['relations'])) {
                         // if there are nested relations, we first add the BelongsTo like in main entry
-                        $belongsToRelations = Arr::where($relationData['relations'], function ($relation_data) {
+                        $belongsToRelations = Arr::where($relationData['relations'], function($relation_data) {
                             return $relation_data['relation_type'] == 'BelongsTo';
                         });
 
@@ -215,7 +215,7 @@ 
                         $valuesWithRelations = $this->associateHasOneBelongsTo($belongsToRelations, $relationData['values'], $relation->getModel());
 
                         // remove previously added BelongsTo relations from relation data.
-                        $relationData['relations'] = Arr::where($relationData['relations'], function ($item) {
+                        $relationData['relations'] = Arr::where($relationData['relations'], function($item) {
                             return $item['relation_type'] != 'BelongsTo';
                         });
 
@@ -315,7 +315,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);
         });
 
@@ -326,10 +326,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);
                 }
 
@@ -343,7 +343,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));
@@ -386,7 +386,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();
@@ -403,7 +403,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})
@@ -433,7 +433,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]], $item);
                 } else {
                     $created_ids[] = $entry->{$relationMethod}()->create($item)->{$relation_local_key};
@@ -443,7 +443,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();
             }