Laravel-Backpack / CRUD

src/app/Library/CrudPanel/CrudFilter.php

Braces +3 added lines, -1 removed lines

@@ -389,10 +389,12 @@
         return $this->fallbackLogic($value);
     }
 
-    public function else($value)
+    public function else {
+        ($value)
     {
         return $this->fallbackLogic($value);
     }
+    }
 
     // ---------------
     // PRIVATE METHODS

Spacing +3 added lines, -3 removed lines

@@ -84,7 +84,7 @@ 
      */
     public function wasNotApplied()
     {
-        return ! $this->applied;
+        return !$this->applied;
     }
 
     /**
@@ -105,7 +105,7 @@ 
 
         $input = $input ?? new ParameterBag($this->crud()->getRequest()->all());
 
-        if (! $input->has($this->name)) {
+        if (!$input->has($this->name)) {
             // if fallback logic was supplied and is a closure
             if (is_callable($this->fallbackLogic)) {
                 return ($this->fallbackLogic)();
@@ -285,7 +285,7 @@ 
      */
     public function values($value)
     {
-        $this->values = (! is_string($value) && is_callable($value)) ? $value() : $value;
+        $this->values = (!is_string($value) && is_callable($value)) ? $value() : $value;
 
         return $this->save();
     }

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

Spacing +13 added lines, -13 removed lines

@@ -62,7 +62,7 @@ 
      */
     public function afterField($targetFieldName)
     {
-        $this->transformFields(function ($fields) use ($targetFieldName) {
+        $this->transformFields(function($fields) use ($targetFieldName) {
             return $this->moveField($fields, $targetFieldName, false);
         });
     }
@@ -74,7 +74,7 @@ 
      */
     public function beforeField($targetFieldName)
     {
-        $this->transformFields(function ($fields) use ($targetFieldName) {
+        $this->transformFields(function($fields) use ($targetFieldName) {
             return $this->moveField($fields, $targetFieldName, true);
         });
     }
@@ -86,7 +86,7 @@ 
      */
     public function makeFirstField()
     {
-        if (! $this->fields()) {
+        if (!$this->fields()) {
             return false;
         }
 
@@ -101,7 +101,7 @@ 
      */
     public function removeField($name)
     {
-        $this->transformFields(function ($fields) use ($name) {
+        $this->transformFields(function($fields) use ($name) {
             Arr::forget($fields, $name);
 
             return $fields;
@@ -115,7 +115,7 @@ 
      */
     public function removeFields($array_of_names)
     {
-        if (! empty($array_of_names)) {
+        if (!empty($array_of_names)) {
             foreach ($array_of_names as $name) {
                 $this->removeField($name);
             }
@@ -128,7 +128,7 @@ 
     public function removeAllFields()
     {
         $current_fields = $this->getCurrentFields();
-        if (! empty($current_fields)) {
+        if (!empty($current_fields)) {
             foreach ($current_fields as $field) {
                 $this->removeField($field['name']);
             }
@@ -220,7 +220,7 @@ 
                 $jsonCastables = ['array', 'object', 'json'];
                 $fieldCasting = $casted_attributes[$field['name']];
 
-                if (in_array($fieldCasting, $jsonCastables) && isset($data[$field['name']]) && ! empty($data[$field['name']]) && ! is_array($data[$field['name']])) {
+                if (in_array($fieldCasting, $jsonCastables) && isset($data[$field['name']]) && !empty($data[$field['name']]) && !is_array($data[$field['name']])) {
                     try {
                         $data[$field['name']] = json_decode($data[$field['name']]);
                     } catch (\Exception $e) {
@@ -249,7 +249,7 @@ 
      */
     public function orderFields($order)
     {
-        $this->transformFields(function ($fields) use ($order) {
+        $this->transformFields(function($fields) use ($order) {
             return $this->applyOrderToFields($fields, $order);
         });
     }
@@ -276,7 +276,7 @@ 
     public function hasUploadFields()
     {
         $fields = $this->getFields();
-        $upload_fields = Arr::where($fields, function ($value, $key) {
+        $upload_fields = Arr::where($fields, function($value, $key) {
             return isset($value['upload']) && $value['upload'] == true;
         });
 
@@ -339,7 +339,7 @@ 
         $alreadyLoaded = $this->getLoadedFieldTypes();
         $type = $this->getFieldTypeWithNamespace($field);
 
-        if (! in_array($type, $this->getLoadedFieldTypes(), true)) {
+        if (!in_array($type, $this->getLoadedFieldTypes(), true)) {
             $alreadyLoaded[] = $type;
             $this->setLoadedFieldTypes($alreadyLoaded);
 
@@ -380,7 +380,7 @@ 
      */
     public function fieldTypeNotLoaded($field)
     {
-        return ! in_array($this->getFieldTypeWithNamespace($field), $this->getLoadedFieldTypes());
+        return !in_array($this->getFieldTypeWithNamespace($field), $this->getLoadedFieldTypes());
     }
 
     /**
@@ -421,7 +421,7 @@ 
      */
     public function hasFieldWhere($attribute, $value)
     {
-        $match = Arr::first($this->fields(), function ($field, $fieldKey) use ($attribute, $value) {
+        $match = Arr::first($this->fields(), function($field, $fieldKey) use ($attribute, $value) {
             return isset($field[$attribute]) && $field[$attribute] == $value;
         });
 
@@ -437,7 +437,7 @@ 
      */
     public function firstFieldWhere($attribute, $value)
     {
-        return Arr::first($this->fields(), function ($field, $fieldKey) use ($attribute, $value) {
+        return Arr::first($this->fields(), function($field, $fieldKey) use ($attribute, $value) {
             return isset($field[$attribute]) && $field[$attribute] == $value;
         });
     }

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

Spacing +13 added lines, -13 removed lines

@@ -25,7 +25,7 @@ 
         $otherButtons = collect([]);
 
         // we get the buttons that belong to the specified stack
-        $stackButtons = $this->buttons()->reject(function ($item) use ($stack, $otherButtons) {
+        $stackButtons = $this->buttons()->reject(function($item) use ($stack, $otherButtons) {
             if ($item->stack != $stack) {
                 // if the button does not belong to this stack we just add it for merging later
                 $otherButtons->push($item);
@@ -37,8 +37,8 @@ 
         });
 
         // we parse the ordered buttons
-        collect($order)->each(function ($btnKey) use ($newButtons, $stackButtons) {
-            if (! $button = $stackButtons->where('name', $btnKey)->first()) {
+        collect($order)->each(function($btnKey) use ($newButtons, $stackButtons) {
+            if (!$button = $stackButtons->where('name', $btnKey)->first()) {
                 abort(500, 'Button name [«'.$btnKey.'»] not found.');
             }
             $newButtons->push($button);
@@ -48,7 +48,7 @@ 
         // we add the remaining buttons to the end of the ordered ones
         if (count($newButtons) < count($stackButtons)) {
             foreach ($stackButtons as $button) {
-                if (! $newButtons->where('name', $button->name)->first()) {
+                if (!$newButtons->where('name', $button->name)->first()) {
                     $newButtons->push($button);
                 }
             }
@@ -114,7 +114,7 @@ 
          */
         $button = $this->buttons()->firstWhere('name', $name);
 
-        if (! $button) {
+        if (!$button) {
             abort(500, 'CRUD Button "'.$name.'" not found. Please check the button exists before you modify it.');
         }
 
@@ -135,7 +135,7 @@ 
      */
     public function removeButton($name, $stack = null)
     {
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($name, $stack) {
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($name, $stack) {
             return $stack == null ? $button->name == $name : ($button->stack == $stack) && ($button->name == $name);
         }));
     }
@@ -146,7 +146,7 @@ 
      */
     public function removeButtons($names, $stack = null)
     {
-        if (! empty($names)) {
+        if (!empty($names)) {
             foreach ($names as $name) {
                 $this->removeButton($name, $stack);
             }
@@ -160,14 +160,14 @@ 
 
     public function removeAllButtonsFromStack($stack)
     {
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($stack) {
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($stack) {
             return $button->stack == $stack;
         }));
     }
 
     public function removeButtonFromStack($name, $stack)
     {
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($name, $stack) {
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($name, $stack) {
             return $button->name == $name && $button->stack == $stack;
         }));
     }
@@ -185,15 +185,15 @@ 
         $destinationButton = $this->firstButtonWhere('name', $destination);
         $destinationKey = $this->getButtonKey($destination);
         $newDestinationKey = ($where == 'before' ? $destinationKey : $destinationKey + 1);
-        $newButtons = $this->buttons()->filter(function ($value, $key) use ($target) {
+        $newButtons = $this->buttons()->filter(function($value, $key) use ($target) {
             return $value->name != $target;
         });
 
-        if (! $targetButton) {
+        if (!$targetButton) {
             return;
         }
 
-        if (! $destinationButton) {
+        if (!$destinationButton) {
             return;
         }
 
@@ -201,7 +201,7 @@ 
         $lastSlice = $newButtons->slice($newDestinationKey, null);
 
         $newButtons = $firstSlice->push($targetButton);
-        $lastSlice->each(function ($item, $key) use ($newButtons) {
+        $lastSlice->each(function($item, $key) use ($newButtons) {
             $newButtons->push($item);
         });
 

src/app/Library/CrudPanel/CrudButton.php

Spacing +1 added lines, -1 removed lines

@@ -388,7 +388,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/Traits/Update.php

Spacing +3 added lines, -3 removed lines

@@ -52,7 +52,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) {
@@ -65,7 +65,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(),
@@ -91,7 +91,7 @@ 
 
             $relation_array = explode('.', $relational_entity);
 
-            $relatedModel = $relatedModel = array_reduce(array_splice($relation_array, 0, -1), function ($obj, $method) {
+            $relatedModel = $relatedModel = array_reduce(array_splice($relation_array, 0, -1), function($obj, $method) {
                 return $obj->{$method} ? $obj->{$method} : $obj;
             }, $model);
 

src/app/Console/Commands/AddSidebarContent.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
      */
     private function getLastLineNumberThatContains($needle, $haystack)
     {
-        $matchingLines = array_filter($haystack, function ($k) use ($needle) {
+        $matchingLines = array_filter($haystack, function($k) use ($needle) {
             return strpos($k, $needle) !== false;
         });
 

src/resources/views/crud/fields/page_or_link.blade.php

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
         </div>
         <div class="col-sm-9">
             <!-- external link input -->
-              <div class="page_or_link_value page_or_link_external_link <?php if (! isset($entry) || $entry->$entry_type != 'external_link') {
+              <div class="page_or_link_value page_or_link_external_link <?php if (!isset($entry) || $entry->$entry_type != 'external_link') {
     echo 'd-none';
 } ?>">
                 <input
@@ -65,7 +65,7 @@ 
                     >
               </div>
               <!-- internal link input -->
-              <div class="page_or_link_value page_or_link_internal_link <?php if (! isset($entry) || $entry->$entry_type != 'internal_link') {
+              <div class="page_or_link_value page_or_link_internal_link <?php if (!isset($entry) || $entry->$entry_type != 'internal_link') {
     echo 'd-none';
 } ?>">
                 <input

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

Spacing +10 added lines, -10 removed lines

@@ -72,7 +72,7 @@ 
         $this->enableFilters();
 
         // check if another filter with the same name exists
-        if (! isset($options['name'])) {
+        if (!isset($options['name'])) {
             abort(500, 'All your filters need names.');
         }
 
@@ -120,7 +120,7 @@ 
     {
         $unappliedFilters = $this->filters()->where('applied', false);
         if ($unappliedFilters->count()) {
-            $unappliedFilters->each(function ($filter) {
+            $unappliedFilters->each(function($filter) {
                 $filter->apply();
             });
         }
@@ -170,7 +170,7 @@ 
     {
         $filter = $this->filters()->firstWhere('name', $name);
 
-        if (! $filter) {
+        if (!$filter) {
             abort(500, 'CRUD Filter "'.$name.'" not found. Please check the filter exists before you modify it.');
         }
 
@@ -185,7 +185,7 @@ 
 
     public function replaceFilter($name, $newFilter)
     {
-        $newFilters = $this->filters()->map(function ($filter, $key) use ($name, $newFilter) {
+        $newFilters = $this->filters()->map(function($filter, $key) use ($name, $newFilter) {
             if ($filter->name != $name) {
                 return $filter;
             }
@@ -198,7 +198,7 @@ 
 
     public function removeFilter($name)
     {
-        $strippedCollection = $this->filters()->reject(function ($filter) use ($name) {
+        $strippedCollection = $this->filters()->reject(function($filter) use ($name) {
             return $filter->name == $name;
         });
 
@@ -241,7 +241,7 @@ 
      */
     public function makeFirstFilter()
     {
-        if (! $this->filters()) {
+        if (!$this->filters()) {
             return false;
         }
 
@@ -271,15 +271,15 @@ 
         $destinationFilter = $this->firstFilterWhere('name', $destination);
         $destinationKey = $this->getFilterKey($destination);
         $newDestinationKey = ($where == 'before' ? $destinationKey : $destinationKey + 1);
-        $newFilters = $this->filters()->filter(function ($value, $key) use ($target) {
+        $newFilters = $this->filters()->filter(function($value, $key) use ($target) {
             return $value->name != $target;
         });
 
-        if (! $targetFilter) {
+        if (!$targetFilter) {
             return;
         }
 
-        if (! $destinationFilter) {
+        if (!$destinationFilter) {
             return;
         }
 
@@ -287,7 +287,7 @@ 
         $lastSlice = $newFilters->slice($newDestinationKey, null);
 
         $newFilters = $firstSlice->push($targetFilter);
-        $lastSlice->each(function ($item, $key) use ($newFilters) {
+        $lastSlice->each(function($item, $key) use ($newFilters) {
             $newFilters->push($item);
         });
 

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);
     }
@@ -304,8 +304,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;
 
@@ -373,20 +372,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->has('http_referrer') ? $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->input('id');
                     $redirectUrl = $crud->route.'/'.$itemId.'/edit';
                     if ($request->has('locale')) {
@@ -398,17 +397,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'),