Laravel-Backpack / CRUD

src/app/Library/CrudPanel/CrudButton.php

Spacing +2 added lines, -2 removed lines

@@ -275,7 +275,7 @@ 
     private function getViewPathsWithFallbacks()
     {
         $type = $this->name;
-        $paths = array_map(function ($item) use ($type) {
+        $paths = array_map(function($item) use ($type) {
             return $item.'.'.$type;
         }, ViewNamespaces::getFor('buttons'));
 
@@ -441,7 +441,7 @@ 
     {
         $itemExists = $this->collection()->contains('name', $this->name);
 
-        if (! $itemExists) {
+        if (!$itemExists) {
             if ($this->position == 'beginning') {
                 $this->collection()->prepend($this);
             } else {

src/app/Library/CrudPanel/CrudColumn.php

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
      */
     public function key(string $key)
     {
-        if (! isset($this->attributes['name'])) {
+        if (!isset($this->attributes['name'])) {
             abort(500, 'Column name must be defined before changing the key.');
         }
 

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

Spacing +8 added lines, -8 removed lines

@@ -47,7 +47,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(),
@@ -72,7 +72,7 @@ 
         $fieldEntity = $field['entity'] ?? false;
         $fakeField = $field['fake'] ?? false;
 
-        if ($fieldEntity && ! $fakeField) {
+        if ($fieldEntity && !$fakeField) {
             return $this->getModelAttributeValueFromRelationship($model, $field);
         }
 
@@ -102,7 +102,7 @@ 
     {
         [$relatedModel, $relationMethod] = $this->getModelAndMethodFromEntity($model, $field);
 
-        if (! method_exists($relatedModel, $relationMethod)) {
+        if (!method_exists($relatedModel, $relationMethod)) {
             return $relatedModel->{$relationMethod};
         }
 
@@ -121,7 +121,7 @@ 
                     $model = $this->setupRelatedModelLocale($model);
                     // when subfields are NOT set we don't need to get any more values
                     // we just return the plain models as we only need the ids
-                    if (! isset($field['subfields'])) {
+                    if (!isset($field['subfields'])) {
                         $result->push($model);
                         continue;
                     }
@@ -148,13 +148,13 @@ 
                 break;
             case 'HasOne':
             case 'MorphOne':
-                if (! method_exists($relatedModel, $relationMethod)) {
+                if (!method_exists($relatedModel, $relationMethod)) {
                     return;
                 }
 
                 $model = $relatedModel->{$relationMethod};
 
-                if (! $model) {
+                if (!$model) {
                     return;
                 }
 
@@ -235,7 +235,7 @@ 
 
         $relationArray = explode('.', $relationEntity);
 
-        $relatedModel = array_reduce(array_splice($relationArray, 0, -1), function ($obj, $method) {
+        $relatedModel = array_reduce(array_splice($relationArray, 0, -1), function($obj, $method) {
             // if the string ends with `_id` we strip it out
             $method = Str::endsWith($method, '_id') ? Str::replaceLast('_id', '', $method) : $method;
 
@@ -262,7 +262,7 @@ 
             // if the subfield name does not contain a dot we just need to check
             // if it has subfields and return the result accordingly.
             foreach ((array) $subfield['name'] as $name) {
-                if (! Str::contains($name, '.')) {
+                if (!Str::contains($name, '.')) {
                     // when subfields are present, $relatedModel->{$name} returns a model instance
                     // otherwise returns the model attribute.
                     if ($relatedModel->{$name}) {

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

Spacing +16 added lines, -17 removed lines

@@ -28,7 +28,7 @@ 
         $higherAction = $this->getSaveActionByOrder(1);
 
         //if there is an higher action and that action is not the backpack default higher one `save_and_back` we return it.
-        if (! empty($higherAction) && key($higherAction) !== 'save_and_back') {
+        if (!empty($higherAction) && key($higherAction) !== 'save_and_back') {
             return key($higherAction);
         }
 
@@ -47,7 +47,7 @@ 
      */
     public function getSaveActionByOrder($order)
     {
-        return array_filter($this->getOperationSetting('save_actions'), function ($arr) use ($order) {
+        return array_filter($this->getOperationSetting('save_actions'), function($arr) use ($order) {
             return $arr['order'] == $order;
         });
     }
@@ -81,7 +81,7 @@ 
         $orderCounter = $this->getOperationSetting('save_actions') !== null ? (count($this->getOperationSetting('save_actions')) + 1) : 1;
         //check for some mandatory fields
         $saveAction['name'] ?? abort(500, 'Please define save action name.');
-        $saveAction['redirect'] = $saveAction['redirect'] ?? function ($crud, $request, $itemId) {
+        $saveAction['redirect'] = $saveAction['redirect'] ?? function($crud, $request, $itemId) {
             return $request->has('_http_referrer') ? $request->get('_http_referrer') : $crud->route;
         };
         $saveAction['visible'] = $saveAction['visible'] ?? true;
@@ -90,7 +90,7 @@ 
 
         $actions = $this->getOperationSetting('save_actions') ?? [];
 
-        if (! in_array($saveAction['name'], $actions)) {
+        if (!in_array($saveAction['name'], $actions)) {
             $actions[$saveAction['name']] = $saveAction;
         }
 
@@ -107,7 +107,7 @@ 
     public function orderSaveAction(string $saveAction, int $wantedOrder)
     {
         $actions = $this->getOperationSetting('save_actions') ?? [];
-        if (! empty($actions)) {
+        if (!empty($actions)) {
             $replaceOrder = isset($actions[$saveAction]) ? $actions[$saveAction]['order'] : count($actions) + 1;
 
             foreach ($actions as $key => $sv) {
@@ -201,7 +201,7 @@ 
     public function orderSaveActions(array $saveActions)
     {
         foreach ($saveActions as $sv => $order) {
-            if (! is_int($order)) {
+            if (!is_int($order)) {
                 $this->orderSaveAction($order, $sv + 1);
             } else {
                 $this->orderSaveAction($sv, $order);
@@ -218,7 +218,7 @@ 
     {
         $actions = $this->getOperationSetting('save_actions') ?? [];
 
-        uasort($actions, function ($a, $b) {
+        uasort($actions, function($a, $b) {
             return $a['order'] <=> $b['order'];
         });
 
@@ -240,7 +240,7 @@ 
             }
         }
 
-        return array_filter($actions, function ($action) {
+        return array_filter($actions, function($action) {
             return $action['visible'] == true;
         }, ARRAY_FILTER_USE_BOTH);
     }
@@ -303,8 +303,7 @@ 
      */
     public function setSaveAction($forceSaveAction = null)
     {
-        $saveAction = $forceSaveAction ?:
-            \Request::input('_save_action', $this->getFallBackSaveAction());
+        $saveAction = $forceSaveAction ?: \Request::input('_save_action', $this->getFallBackSaveAction());
 
         $showBubble = $this->getOperationSetting('showSaveActionChange') ?? config('backpack.crud.operations.'.$this->getCurrentOperation().'.showSaveActionChange') ?? true;
 
@@ -371,20 +370,20 @@ 
         $defaultSaveActions = [
             [
                 'name' => 'save_and_back',
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return $crud->hasAccess('list');
                 },
-                'redirect' => function ($crud, $request, $itemId = null) {
+                'redirect' => function($crud, $request, $itemId = null) {
                     return $request->request->has('_http_referrer') ? $request->request->get('_http_referrer') : $crud->route;
                 },
                 'button_text' => trans('backpack::crud.save_action_save_and_back'),
             ],
             [
                 'name' => 'save_and_edit',
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return $crud->hasAccess('update');
                 },
-                'redirect' => function ($crud, $request, $itemId = null) {
+                'redirect' => function($crud, $request, $itemId = null) {
                     $itemId = $itemId ?: $request->request->get('id');
                     $redirectUrl = $crud->route.'/'.$itemId.'/edit';
                     if ($request->request->has('_locale')) {
@@ -396,17 +395,17 @@ 
 
                     return $redirectUrl;
                 },
-                'referrer_url' => function ($crud, $request, $itemId) {
+                'referrer_url' => function($crud, $request, $itemId) {
                     return url($crud->route.'/'.$itemId.'/edit');
                 },
                 'button_text' => trans('backpack::crud.save_action_save_and_edit'),
             ],
             [
                 'name' => 'save_and_new',
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return $crud->hasAccess('create');
                 },
-                'redirect' => function ($crud, $request, $itemId = null) {
+                'redirect' => function($crud, $request, $itemId = null) {
                     return $this->route.'/create';
                 },
                 'button_text' => trans('backpack::crud.save_action_save_and_new'),

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

Spacing +9 added lines, -9 removed lines

@@ -23,9 +23,9 @@ 
      */
     public function applySearchTerm($searchTerm)
     {
-        return $this->query->where(function ($query) use ($searchTerm) {
+        return $this->query->where(function($query) use ($searchTerm) {
             foreach ($this->columns() as $column) {
-                if (! isset($column['type'])) {
+                if (!isset($column['type'])) {
                     abort(400, 'Missing column type when trying to apply search term.');
                 }
 
@@ -85,7 +85,7 @@ 
 
                 case 'select':
                 case 'select_multiple':
-                    $query->orWhereHas($column['entity'], function ($q) use ($column, $searchTerm, $searchOperator) {
+                    $query->orWhereHas($column['entity'], function($q) use ($column, $searchTerm, $searchOperator) {
                         $q->where($this->getColumnWithTableNamePrefixed($q, $column['attribute']), $searchOperator, '%'.$searchTerm.'%');
                     });
                     break;
@@ -109,7 +109,7 @@ 
                 $column_number = (int) $order['column'];
                 $column_direction = (strtolower((string) $order['dir']) == 'asc' ? 'ASC' : 'DESC');
                 $column = $this->findColumnById($column_number);
-                if ($column['tableColumn'] && ! isset($column['orderLogic'])) {
+                if ($column['tableColumn'] && !isset($column['orderLogic'])) {
                     if (method_exists($this->model, 'translationEnabled') &&
                         $this->model->translationEnabled() &&
                         $this->model->isTranslatableAttribute($column['name']) &&
@@ -136,12 +136,12 @@ 
         $table = $this->model->getTable();
         $key = $this->model->getKeyName();
 
-        $hasOrderByPrimaryKey = collect($orderBy)->some(function ($item) use ($key, $table) {
+        $hasOrderByPrimaryKey = collect($orderBy)->some(function($item) use ($key, $table) {
             return (isset($item['column']) && $item['column'] === $key)
                 || (isset($item['sql']) && str_contains($item['sql'], "$table.$key"));
         });
 
-        if (! $hasOrderByPrimaryKey) {
+        if (!$hasOrderByPrimaryKey) {
             $this->orderByWithPrefix($key, 'DESC');
         }
     }
@@ -320,13 +320,13 @@ 
         if (isset($column['type'])) {
             // create a list of paths to column blade views
             // including the configured view_namespaces
-            $columnPaths = array_map(function ($item) use ($column) {
+            $columnPaths = array_map(function($item) use ($column) {
                 return $item.'.'.$column['type'];
             }, ViewNamespaces::getFor('columns'));
 
             // but always fall back to the stock 'text' column
             // if a view doesn't exist
-            if (! in_array('crud::columns.text', $columnPaths)) {
+            if (!in_array('crud::columns.text', $columnPaths)) {
                 $columnPaths[] = 'crud::columns.text';
             }
 
@@ -365,7 +365,7 @@ 
      */
     private function renderCellView($view, $column, $entry, $rowNumber = false)
     {
-        if (! view()->exists($view)) {
+        if (!view()->exists($view)) {
             $view = 'crud::columns.text'; // fallback to text column
         }
 

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

Spacing +3 added lines, -3 removed lines

@@ -111,7 +111,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);
@@ -275,7 +275,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();
         }
 
@@ -320,7 +320,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/Library/CrudPanel/Traits/FieldsProtectedMethods.php

Spacing +8 added lines, -8 removed lines

@@ -69,7 +69,7 @@ 
      */
     public function overwriteFieldNameFromDotNotationToArray($field)
     {
-        if (! is_array($field['name']) && strpos($field['name'], '.') !== false) {
+        if (!is_array($field['name']) && strpos($field['name'], '.') !== false) {
             $entity_array = explode('.', $field['name']);
             $name_string = '';
 
@@ -115,7 +115,7 @@ 
             return ['name' => $field];
         }
 
-        if (is_array($field) && ! isset($field['name'])) {
+        if (is_array($field) && !isset($field['name'])) {
             abort(500, 'All fields must have their name defined');
         }
 
@@ -192,7 +192,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();
         }
 
@@ -208,7 +208,7 @@ 
      */
     protected function makeSureFieldHasLabel($field)
     {
-        if (! isset($field['label'])) {
+        if (!isset($field['label'])) {
             $name = is_array($field['name']) ? $field['name'][0] : $field['name'];
             $name = str_replace('_id', '', $name);
             $field['label'] = mb_ucfirst(str_replace('_', ' ', $name));
@@ -226,7 +226,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']);
         }
 
@@ -263,7 +263,7 @@ 
      */
     protected function makeSureSubfieldsHaveNecessaryAttributes($field)
     {
-        if (! isset($field['subfields'])) {
+        if (!isset($field['subfields'])) {
             return $field;
         }
 
@@ -279,7 +279,7 @@ 
 
             $subfield['parentFieldName'] = is_array($field['name']) ? false : $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';
@@ -331,7 +331,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/Views.php

Spacing +2 added lines, -2 removed lines

@@ -299,7 +299,7 @@ 
         }
         // otherwise, loop through all the possible view namespaces
         // until you find a view that exists
-        $paths = array_map(function ($item) use ($viewPath) {
+        $paths = array_map(function($item) use ($viewPath) {
             return $item.'.'.$viewPath;
         }, ViewNamespaces::getFor('fields'));
 
@@ -310,7 +310,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 field view: '.$viewPath.'. Please check for typos.'.(backpack_pro() ? '' : ' If you are trying to use a PRO field, please first purchase and install the backpack/pro addon from backpackforlaravel.com'), 1);
         }
         abort(500, "Cannot find '{$viewPath}' field view in any of the regular locations.");

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

Spacing +10 added lines, -10 removed lines

@@ -46,10 +46,10 @@ 
     private function getValidationRulesFromFieldsAndSubfields($fields)
     {
         $rules = collect($fields)
-            ->filter(function ($value, $key) {
+            ->filter(function($value, $key) {
                 // only keep fields where 'validationRules' OR there are subfields
                 return array_key_exists('validationRules', $value) || array_key_exists('subfields', $value);
-            })->map(function ($item, $key) {
+            })->map(function($item, $key) {
                 $validationRules = [];
                 // only keep the rules, not the entire field definition
                 if (isset($item['validationRules'])) {
@@ -57,7 +57,7 @@ 
                 }
                 // add validation rules for subfields
                 if (array_key_exists('subfields', $item)) {
-                    $subfieldsWithValidation = array_filter($item['subfields'], function ($subfield) {
+                    $subfieldsWithValidation = array_filter($item['subfields'], function($subfield) {
                         return array_key_exists('validationRules', $subfield);
                     });
 
@@ -82,10 +82,10 @@ 
     {
         $messages = [];
         collect($fields)
-            ->filter(function ($value, $key) {
+            ->filter(function($value, $key) {
                 // only keep fields where 'validationMessages' OR there are subfields
                 return array_key_exists('validationMessages', $value) || array_key_exists('subfields', $value);
-            })->each(function ($item, $key) use (&$messages) {
+            })->each(function($item, $key) use (&$messages) {
                 if (isset($item['validationMessages'])) {
                     foreach ($item['validationMessages'] as $rule => $message) {
                         $messages[$key.'.'.$rule] = $message;
@@ -93,7 +93,7 @@ 
                 }
                 // add messages from subfields
                 if (array_key_exists('subfields', $item)) {
-                    $subfieldsWithValidationMessages = array_filter($item['subfields'], function ($subfield) {
+                    $subfieldsWithValidationMessages = array_filter($item['subfields'], function($subfield) {
                         return array_key_exists('validationRules', $subfield);
                     });
 
@@ -129,7 +129,7 @@ 
      */
     public function setValidation($classOrRulesArray = false, $messages = [])
     {
-        if (! $classOrRulesArray) {
+        if (!$classOrRulesArray) {
             $this->setValidationFromFields();
         } elseif (is_array($classOrRulesArray)) {
             $this->setValidationFromArray($classOrRulesArray, $messages);
@@ -236,7 +236,7 @@ 
             return app(get_class($extendedRequest), ['rules' => $rules, 'messages' => $messages]);
         }
 
-        return ! empty($rules) ? $this->checkRequestValidity($rules, $messages) : $this->getRequest();
+        return !empty($rules) ? $this->checkRequestValidity($rules, $messages) : $this->getRequest();
     }
 
     /**
@@ -306,7 +306,7 @@ 
      */
     public function isRequired($inputKey)
     {
-        if (! $this->hasOperationSetting('requiredFields')) {
+        if (!$this->hasOperationSetting('requiredFields')) {
             return false;
         }
 
@@ -327,7 +327,7 @@ 
     {
         [$rules, $messages] = $this->getValidationRulesAndMessagesFromField($field, $parent);
 
-        if (! empty($rules)) {
+        if (!empty($rules)) {
             $this->setValidation($rules, $messages);
         }
     }