Laravel-Backpack / CRUD

tests/Unit/CrudPanel/CrudPanelAccessTest.php

Spacing +5 added lines, -5 removed lines

@@ -105,7 +105,7 @@ 
 
     public function testItCanUseAClosureToResolveAccess()
     {
-        $this->crudPanel->setAccessCondition('list', function () {
+        $this->crudPanel->setAccessCondition('list', function() {
             return true;
         });
 
@@ -116,7 +116,7 @@ 
 
     public function testItCanUseAClosureToResolveAccessForMultipleOperations()
     {
-        $this->crudPanel->setAccessCondition(['list', 'create'], function () {
+        $this->crudPanel->setAccessCondition(['list', 'create'], function() {
             return true;
         });
 
@@ -127,7 +127,7 @@ 
 
     public function testItCanCheckIfAnOperationHasAccessConditions()
     {
-        $this->crudPanel->setAccessCondition(['list', 'create'], function () {
+        $this->crudPanel->setAccessCondition(['list', 'create'], function() {
             return true;
         });
 
@@ -137,7 +137,7 @@ 
 
     public function testItCanCheckAccessToAll()
     {
-        $this->crudPanel->allowAccess(['list', 'create'], function () {
+        $this->crudPanel->allowAccess(['list', 'create'], function() {
             return true;
         });
 
@@ -147,7 +147,7 @@ 
 
     public function testItCanAllowAccessToSomeSpecificOperationWhileDenyingOthers()
     {
-        $this->crudPanel->allowAccess(['list', 'create'], function () {
+        $this->crudPanel->allowAccess(['list', 'create'], function() {
             return true;
         });
 

src/app/Console/Commands/AddCustomRouteContent.php

Spacing +3 added lines, -3 removed lines

@@ -43,7 +43,7 @@ 
     {
         $routeFilePath = base_path($this->option('route-file'));
 
-        if (! file_exists($routeFilePath)) {
+        if (!file_exists($routeFilePath)) {
             if ($routeFilePath !== base_path($this->backpackCustomRouteFile)) {
                 $this->info('The route file <fg=blue>'.$routeFilePath.'</> does not exist. Please create it first.');
 
@@ -100,7 +100,7 @@ 
 
     private function cleanContentArray(array $content)
     {
-        return array_filter(array_map(function ($line) {
+        return array_filter(array_map(function($line) {
             $lineText = trim($line);
             if ($lineText === '' ||
                 $lineText === '\n' ||
@@ -133,7 +133,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/app/Library/CrudPanel/Traits/Fields.php

Spacing +13 added lines, -13 removed lines

@@ -136,7 +136,7 @@ 
      */
     public function afterField($targetFieldName)
     {
-        $this->transformFields(function ($fields) use ($targetFieldName) {
+        $this->transformFields(function($fields) use ($targetFieldName) {
             return $this->moveField($fields, $targetFieldName, false);
         });
     }
@@ -148,7 +148,7 @@ 
      */
     public function beforeField($targetFieldName)
     {
-        $this->transformFields(function ($fields) use ($targetFieldName) {
+        $this->transformFields(function($fields) use ($targetFieldName) {
             return $this->moveField($fields, $targetFieldName, true);
         });
     }
@@ -160,7 +160,7 @@ 
      */
     public function makeFirstField()
     {
-        if (! $this->fields()) {
+        if (!$this->fields()) {
             return false;
         }
 
@@ -175,7 +175,7 @@ 
      */
     public function removeField($name)
     {
-        $this->transformFields(function ($fields) use ($name) {
+        $this->transformFields(function($fields) use ($name) {
             Arr::forget($fields, $name);
 
             return $fields;
@@ -189,7 +189,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);
             }
@@ -202,7 +202,7 @@ 
     public function removeAllFields()
     {
         $current_fields = $this->getCleanStateFields();
-        if (! empty($current_fields)) {
+        if (!empty($current_fields)) {
             foreach ($current_fields as $field) {
                 $this->removeField($field['name']);
             }
@@ -290,7 +290,7 @@ 
      */
     public function orderFields($order)
     {
-        $this->transformFields(function ($fields) use ($order) {
+        $this->transformFields(function($fields) use ($order) {
             return $this->applyOrderToFields($fields, $order);
         });
     }
@@ -314,7 +314,7 @@ 
     public function hasUploadFields()
     {
         $fields = $this->getCleanStateFields();
-        $upload_fields = Arr::where($fields, function ($value, $key) {
+        $upload_fields = Arr::where($fields, function($value, $key) {
             // check if any subfields have uploads
             if (isset($value['subfields'])) {
                 foreach ($value['subfields'] as $subfield) {
@@ -386,7 +386,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);
 
@@ -427,7 +427,7 @@ 
      */
     public function fieldTypeNotLoaded($field)
     {
-        return ! in_array($this->getFieldTypeWithNamespace($field), $this->getLoadedFieldTypes());
+        return !in_array($this->getFieldTypeWithNamespace($field), $this->getLoadedFieldTypes());
     }
 
     /**
@@ -439,7 +439,7 @@ 
     {
         $fieldNamesArray = array_column($this->getCleanStateFields(), 'name');
 
-        return array_reduce($fieldNamesArray, function ($names, $item) {
+        return array_reduce($fieldNamesArray, function($names, $item) {
             if (strpos($item, ',') === false) {
                 $names[] = $item;
 
@@ -490,7 +490,7 @@ 
      */
     public function hasFieldWhere($attribute, $value)
     {
-        $match = Arr::first($this->getCleanStateFields(), function ($field, $fieldKey) use ($attribute, $value) {
+        $match = Arr::first($this->getCleanStateFields(), function($field, $fieldKey) use ($attribute, $value) {
             return isset($field[$attribute]) && $field[$attribute] == $value;
         });
 
@@ -506,7 +506,7 @@ 
      */
     public function firstFieldWhere($attribute, $value)
     {
-        return Arr::first($this->getCleanStateFields(), function ($field, $fieldKey) use ($attribute, $value) {
+        return Arr::first($this->getCleanStateFields(), function($field, $fieldKey) use ($attribute, $value) {
             return isset($field[$attribute]) && $field[$attribute] == $value;
         });
     }

src/app/Models/Traits/HasIdentifiableAttribute.php

Spacing +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
         // we get the first column from database
         // that is NOT indexed (usually primary, foreign keys)
         foreach ($columnNames as $columnName) {
-            if (! in_array($columnName, $indexes)) {
+            if (!in_array($columnName, $indexes)) {
                 //check for convention "field<_id>" in case developer didn't add foreign key constraints.
                 if (strpos($columnName, '_id') !== false) {
                     continue;
@@ -67,7 +67,7 @@ 
 
         // in case everything fails we just return the first column in database
         $firstColumnInTable = Arr::first($columnNames);
-        if (! empty($firstColumnInTable)) {
+        if (!empty($firstColumnInTable)) {
             return $firstColumnInTable;
         }
 

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'),