Laravel-Backpack / CRUD

tests/config/database/migrations/2024_07_30_000000_create_reorder_table.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      */
     public function up()
     {
-        Schema::create('reorders', function (Blueprint $table) {
+        Schema::create('reorders', function(Blueprint $table) {
             $table->increments('id');
             $table->string('name');
             $table->bigInteger('parent_id')->nullable();

tests/BaseTestClass.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
             (array) config('backpack.base.middleware_key', 'admin'),
             'prefix' => config('backpack.base.route_prefix', 'admin'),
         ],
-            function () {
+            function() {
                 Route::get('articles/{id}/show/{detail}', ['as' => 'article.show.detail', 'action' => 'Backpack\CRUD\Tests\config\Http\Controllers\ArticleCrudController@detail']);
                 Route::crud('users', 'Backpack\CRUD\Tests\config\Http\Controllers\UserCrudController');
                 Route::crud('articles', 'Backpack\CRUD\Tests\config\Http\Controllers\ArticleCrudController');
@@ -45,7 +45,7 @@ 
     protected function setupUserCreateRequest()
     {
         $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']);
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new RouteInstance('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request);
         });
         $this->crudPanel->setRequest($request);

tests/Unit/CrudPanel/CrudPanelQueryTest.php

Spacing +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
 
     public function testItCanAddAClauseToTheQueryUsingAClosure()
     {
-        $closure = function ($query) {
+        $closure = function($query) {
             $query->where('id', 1);
         };
 
@@ -51,7 +51,7 @@ 
 
     public function testItCanAddABaseClauseToTheQueryUsingAClosure()
     {
-        $closure = function ($query) {
+        $closure = function($query) {
             $query->where('id', 1);
         };
 
@@ -115,7 +115,7 @@ 
         $this->crudPanel->addColumn([
             'name' => 'name',
             'type' => 'text',
-            'orderLogic' => function ($query, $column, $direction) {
+            'orderLogic' => function($query, $column, $direction) {
                 $query->orderBy('name', $direction);
             },
         ]);

tests/Unit/CrudPanel/CrudPanelSaveActionsTest.php

Spacing +7 added lines, -7 removed lines

@@ -25,13 +25,13 @@ 
         $this->singleSaveAction = [
             'name' => 'save_action_one',
             'button_text' => 'custom',
-            'redirect' => function ($crud, $request, $itemId) {
+            'redirect' => function($crud, $request, $itemId) {
                 return 'https://backpackforlaravel.com';
             },
-            'referrer_url' => function ($crud, $request, $itemId) {
+            'referrer_url' => function($crud, $request, $itemId) {
                 return 'https://backpackforlaravel.com';
             },
-            'visible' => function ($crud) {
+            'visible' => function($crud) {
                 return true;
             },
         ];
@@ -39,19 +39,19 @@ 
         $this->multipleSaveActions = [
             [
                 'name' => 'save_action_one',
-                'redirect' => function ($crud, $request, $itemId) {
+                'redirect' => function($crud, $request, $itemId) {
                     return $crud->route;
                 },
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return true;
                 },
             ],
             [
                 'name' => 'save_action_two',
-                'redirect' => function ($crud, $request, $itemId) {
+                'redirect' => function($crud, $request, $itemId) {
                     return $crud->route;
                 },
-                'visible' => function ($crud) {
+                'visible' => function($crud) {
                     return true;
                 },
             ],

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/Search.php

Spacing +9 added lines, -9 removed lines

@@ -22,9 +22,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.');
                 }
 
@@ -84,7 +84,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;
@@ -108,7 +108,7 @@ 
                 $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 @@ 
         $key = $this->model->getKeyName();
         $groupBy = $this->query->toBase()->groups;
 
-        $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 && empty($groupBy)) {
+        if (!$hasOrderByPrimaryKey && empty($groupBy)) {
             $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
         }