Laravel-Backpack / CRUD

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

Spacing +15 added lines, -16 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;
         });
     }
@@ -88,7 +88,7 @@ 
 
         $actions = $this->getOperationSetting('save_actions') ?? [];
 
-        if (! in_array($saveAction['name'], $actions)) {
+        if (!in_array($saveAction['name'], $actions)) {
             $actions[$saveAction['name']] = $saveAction;
         }
 
@@ -105,7 +105,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) {
@@ -199,7 +199,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);
@@ -216,7 +216,7 @@ 
     {
         $actions = $this->getOperationSetting('save_actions') ?? [];
 
-        uasort($actions, function ($a, $b) {
+        uasort($actions, function($a, $b) {
             return $a['order'] <=> $b['order'];
         });
 
@@ -238,7 +238,7 @@ 
             }
         }
 
-        return array_filter($actions, function ($action) {
+        return array_filter($actions, function($action) {
             return $action['visible'] == true;
         }, ARRAY_FILTER_USE_BOTH);
     }
@@ -300,8 +300,7 @@ 
      */
     public function setSaveAction($forceSaveAction = null)
     {
-        $saveAction = $forceSaveAction ?:
-            $this->getRequest()->input('_save_action', $this->getFallBackSaveAction());
+        $saveAction = $forceSaveAction ?: $this->getRequest()->input('_save_action', $this->getFallBackSaveAction());
 
         $showBubble = $this->getOperationSetting('showSaveActionChange') ?? config('backpack.crud.operations.'.$this->getCurrentOperation().'.showSaveActionChange') ?? true;
 
@@ -369,20 +368,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->get('id');
                     $redirectUrl = $crud->route.'/'.$itemId.'/edit';
                     if ($request->has('_locale')) {
@@ -394,17 +393,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'),

tests/Unit/CrudPanel/CrudPanelCreateTest.php

Spacing +10 added lines, -10 removed lines

@@ -818,7 +818,7 @@ 
 
     public function testCreateBelongsToFake()
     {
-        $belongsToField = [   // select_grouped
+        $belongsToField = [// select_grouped
             'label' => 'Select_grouped',
             'type' => 'select_grouped', //https://github.com/Laravel-Backpack/CRUD/issues/502
             'name' => 'bang_relation_field',
@@ -954,7 +954,7 @@ 
                 'nickname' => 'i_have_has_one',
                 'profile_picture' => 'ohh my picture 1.jpg',
                 'article' => 1,
-                'addresses' => [ // HasOne is tested in other test function
+                'addresses' => [// HasOne is tested in other test function
                     [
                         'city' => 2,
                         'street' => 'test',
@@ -1707,23 +1707,23 @@ 
         $this->crudPanel->addField([
             'name' => 'name',
             'events' => [
-                'created' => function ($entry) {
+                'created' => function($entry) {
                 },
-                'creating' => function ($entry) {
+                'creating' => function($entry) {
                     $entry->email = 'backpack@laravel.com';
                     $entry->password = Hash::make('password');
                 },
-                'saving' => function ($entry) {
+                'saving' => function($entry) {
                 },
-                'saved' => function ($entry) {
+                'saved' => function($entry) {
                 },
-                'updating' => function ($entry) {
+                'updating' => function($entry) {
                 },
-                'updated' => function ($entry) {
+                'updated' => function($entry) {
                 },
-                'deleting' => function ($entry) {
+                'deleting' => function($entry) {
                 },
-                'deleted' => function ($entry) {
+                'deleted' => function($entry) {
                 },
             ],
         ]);

tests/Unit/CrudPanel/CrudPanelSearchTest.php

Spacing +3 added lines, -3 removed lines

@@ -90,7 +90,7 @@ 
         $this->crudPanel->addColumn([
             'name' => 'test',
             'type' => 'my_custom_type',
-            'searchLogic' => function ($query, $column, $searchTerm) {
+            'searchLogic' => function($query, $column, $searchTerm) {
                 $query->where($column['name'], 'like', "%{$searchTerm}%");
             },
             'tableColumn' => true,
@@ -225,7 +225,7 @@ 
             'name' => 'name',
             'type' => 'text',
             'tableColumn' => true,
-            'orderLogic' => function ($query, $column, $searchTerm) {
+            'orderLogic' => function($query, $column, $searchTerm) {
                 $query->orderBy('name', 'asc');
             },
         ]);
@@ -244,7 +244,7 @@ 
             'name' => 'name',
             'type' => 'text',
             'tableColumn' => true,
-            'orderLogic' => function ($query, $column, $searchTerm) {
+            'orderLogic' => function($query, $column, $searchTerm) {
                 $query->orderBy('id', 'asc');
             },
         ]);

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

Spacing +9 added lines, -10 removed lines

@@ -50,7 +50,7 @@ 
             return ['name' => Str::replace(' ', '', $column)];
         }
 
-        if (is_array($column) && ! isset($column['name'])) {
+        if (is_array($column) && !isset($column['name'])) {
             $column['name'] = 'anonymous_column_'.Str::random(5);
         }
 
@@ -68,7 +68,7 @@ 
      */
     protected function makeSureColumnHasLabel($column)
     {
-        if (! isset($column['label'])) {
+        if (!isset($column['label'])) {
             $column['label'] = mb_ucfirst($this->makeLabel($column['name']));
         }
 
@@ -153,7 +153,7 @@ 
      */
     protected function makeSureColumnHasKey($column)
     {
-        if (! isset($column['key'])) {
+        if (!isset($column['key'])) {
             $column['key'] = str_replace('.', '__', $column['name']);
         }
 
@@ -170,7 +170,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']);
         }
 
@@ -292,12 +292,11 @@ 
         $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);
 
-            if($element['priority'] === count($columnsArray)) {
+            if ($element['priority'] === count($columnsArray)) {
                 // the priority was most likely auto-set as it corresponds to the column array count
                 // update the priority to the target column position
                 $element['priority'] = $targetColumnPosition;
@@ -322,7 +321,7 @@ 
     {
         static $cache = [];
 
-        if (! $this->driverIsSql()) {
+        if (!$this->driverIsSql()) {
             return true;
         }
 
@@ -338,7 +337,7 @@ 
     /**
      * Prepare the column attributes and add it to operation settings.
      */
-    private function prepareAttributesAndAddColumn(array|string $column): CrudColumn
+    private function prepareAttributesAndAddColumn(array | string $column): CrudColumn
     {
         $column = $this->makeSureColumnHasNeededAttributes($column);
         $this->addColumnToOperationSettings($column);