Laravel-Backpack / CRUD

src/app/Http/Controllers/Auth/ResetPasswordController.php

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
 
         $this->middleware("guest:$guard");
 
-        if (! backpack_users_have_email()) {
+        if (!backpack_users_have_email()) {
             abort(501, trans('backpack::base.no_email_column'));
         }
 
@@ -90,7 +90,7 @@ 
         // will update the password on an actual user model and persist it to the
         // database. Otherwise we will parse the error and return the response.
         $response = $this->broker()->reset(
-            $this->credentials($request), function ($user, $password) {
+            $this->credentials($request), function($user, $password) {
                 $this->resetPassword($user, $password);
             }
         );

src/app/Http/Middleware/AuthenticateSession.php

Spacing +6 added lines, -6 removed lines

@@ -38,19 +38,19 @@ 
      */
     public function handle($request, Closure $next)
     {
-        if (! $request->hasSession() || ! $this->user) {
+        if (!$request->hasSession() || !$this->user) {
             return $next($request);
         }
 
         if ($this->guard()->viaRemember()) {
             $passwordHash = explode('|', $request->cookies->get($this->guard()->getRecallerName()))[2] ?? null;
 
-            if (! $passwordHash || $passwordHash != $this->user->getAuthPassword()) {
+            if (!$passwordHash || $passwordHash != $this->user->getAuthPassword()) {
                 $this->logout($request);
             }
         }
 
-        if (! $request->session()->has('password_hash_'.backpack_guard_name())) {
+        if (!$request->session()->has('password_hash_'.backpack_guard_name())) {
             $this->storePasswordHashInSession($request);
         }
 
@@ -58,8 +58,8 @@ 
             $this->logout($request);
         }
 
-        return tap($next($request), function () use ($request) {
-            if (! is_null($this->guard()->user())) {
+        return tap($next($request), function() use ($request) {
+            if (!is_null($this->guard()->user())) {
                 $this->storePasswordHashInSession($request);
             }
         });
@@ -73,7 +73,7 @@ 
      */
     protected function storePasswordHashInSession($request)
     {
-        if (! $this->user) {
+        if (!$this->user) {
             return;
         }
 

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

Spacing +11 added lines, -11 removed lines

@@ -65,7 +65,7 @@ 
         $model = $this->getRelationModel($entity, -1);
         $lastSegmentAfterDot = Str::of($field['entity'])->afterLast('.');
 
-        if (! method_exists($model, $lastSegmentAfterDot)) {
+        if (!method_exists($model, $lastSegmentAfterDot)) {
             return (string) Str::of($field['entity'])->beforeLast('.');
         }
 
@@ -86,7 +86,7 @@ 
 
         return collect($this->getCleanStateFields())
             ->whereIn('relation_type', $relation_types)
-            ->filter(function ($item) use ($nested) {
+            ->filter(function($item) use ($nested) {
                 if ($nested) {
                     return true;
                 }
@@ -130,7 +130,7 @@ 
      */
     private function getRelationFieldsWithoutRelationType($relations, $fields = [])
     {
-        if (! is_array($relations)) {
+        if (!is_array($relations)) {
             $relations = [$relations];
         }
 
@@ -139,8 +139,8 @@ 
         }
 
         foreach ($relations as $relation) {
-            $fields = array_filter($fields, function ($field) use ($relation) {
-                if (! isset($field['relation_type'])) {
+            $fields = array_filter($fields, function($field) use ($relation) {
+                if (!isset($field['relation_type'])) {
                     return false;
                 }
 
@@ -174,7 +174,7 @@ 
                     $fields = array_merge($field['subfields'], $fields);
                 }
             }
-            $fields = array_filter($fields, function ($field) {
+            $fields = array_filter($fields, function($field) {
                 return isset($field['relation_type']) && $field['relation_type'] === 'BelongsTo';
             });
         }
@@ -243,8 +243,8 @@ 
     {
         $all_relation_fields = $this->getRelationFields();
 
-        return Arr::where($all_relation_fields, function ($value, $key) {
-            return isset($value['pivot']) && ! $value['pivot'];
+        return Arr::where($all_relation_fields, function($value, $key) {
+            return isset($value['pivot']) && !$value['pivot'];
         });
     }
 
@@ -257,7 +257,7 @@ 
     {
         $all_relation_fields = $this->getRelationFields();
 
-        return Arr::where($all_relation_fields, function ($value, $key) {
+        return Arr::where($all_relation_fields, function($value, $key) {
             return isset($value['pivot']) && $value['pivot'];
         });
     }
@@ -338,7 +338,7 @@ 
         }
 
         // relationships are always public methods.
-        if (! $methodReflection->isPublic()) {
+        if (!$methodReflection->isPublic()) {
             return false;
         }
 
@@ -368,7 +368,7 @@ 
      */
     private function isAttributeInRelationString($field)
     {
-        if (! str_contains($field['entity'], '.')) {
+        if (!str_contains($field['entity'], '.')) {
             return false;
         }
 

src/ViewNamespaces.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
     public static function addFor(string $domain, $viewNamespaces)
     {
         foreach ((array) $viewNamespaces as $viewNamespace) {
-            if (! in_array($viewNamespace, self::getForDomain($domain))) {
+            if (!in_array($viewNamespace, self::getForDomain($domain))) {
                 self::$viewNamespaces[$domain][] = $viewNamespace;
             }
         }
@@ -83,7 +83,7 @@ 
      */
     public static function getViewPathsFor(string $domain, string $viewName)
     {
-        return array_map(function ($item) use ($viewName) {
+        return array_map(function($item) use ($viewName) {
             return $item.'.'.$viewName;
         }, self::getFor($domain));
     }

tests/Unit/CrudPanel/CrudPanelTitlesAndHeadingsTest.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
     {
         $this->crudPanel->setModel(User::class);
         $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']);
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request);
         });
         $this->crudPanel->setRequest($request);
@@ -51,7 +51,7 @@ 
     {
         $this->crudPanel->setModel(User::class);
         $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']);
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request);
         });
         $this->crudPanel->setRequest($request);
@@ -66,7 +66,7 @@ 
     {
         $this->crudPanel->setModel(User::class);
         $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']);
-        $request->setRouteResolver(function () use ($request) {
+        $request->setRouteResolver(function() use ($request) {
             return (new Route('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request);
         });
         $this->crudPanel->setRequest($request);

tests/Unit/CrudPanel/CrudPanelMacroTest.php

Spacing +2 added lines, -2 removed lines

@@ -9,7 +9,7 @@ 
 {
     public function testItCanRegisterMacro()
     {
-        $this->crudPanel::macro('validMacro', function () {
+        $this->crudPanel::macro('validMacro', function() {
             return true;
         });
 
@@ -19,7 +19,7 @@ 
     public function testThrowsErrorIfMacroExists()
     {
         try {
-            $this->crudPanel::macro('setModel', function () {
+            $this->crudPanel::macro('setModel', function() {
                 return true;
             });
         } catch (\Throwable $e) {

tests/Unit/CrudPanel/CrudPanelCreateTest.php

Spacing +2 added lines, -2 removed lines

@@ -605,7 +605,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',
@@ -741,7 +741,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',

tests/Unit/CrudPanel/CrudPanelOperationsTest.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 
     public function testItCanConfigureOperations()
     {
-        $this->crudPanel->operation(['create', 'update'], function () {
+        $this->crudPanel->operation(['create', 'update'], function() {
             $this->crudPanel->addField(['name' => 'test', 'type' => 'text']);
         });
         $this->crudPanel->applyConfigurationFromSettings('create');

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