Scrutinizer GitHub App not installed

We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.

Install GitHub App

Passed
Push — crud-uploads ( f5e93a...469ece )
by Pedro
11:36
created
src/app/Http/Controllers/Operations/ListOperation.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@
 block discarded – undo
41 41
     {
42 42
         $this->crud->allowAccess('list');
43 43
 
44
-        $this->crud->operation('list', function () {
44
+        $this->crud->operation('list', function() {
45 45
             $this->crud->loadDefaultOperationSettingsFromConfig();
46 46
         });
47 47
     }
Please login to merge, or discard this patch.
src/app/Http/Controllers/Auth/ResetPasswordController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 
47 47
         $this->middleware("guest:$guard");
48 48
 
49
-        if (! backpack_users_have_email()) {
49
+        if (!backpack_users_have_email()) {
50 50
             abort(501, trans('backpack::base.no_email_column'));
51 51
         }
52 52
 
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
         // will update the password on an actual user model and persist it to the
91 91
         // database. Otherwise we will parse the error and return the response.
92 92
         $response = $this->broker()->reset(
93
-            $this->credentials($request), function ($user, $password) {
93
+            $this->credentials($request), function($user, $password) {
94 94
                 $this->resetPassword($user, $password);
95 95
             }
96 96
         );
Please login to merge, or discard this patch.
src/app/Http/Middleware/AuthenticateSession.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -38,19 +38,19 @@  discard block
 block discarded – undo
38 38
      */
39 39
     public function handle($request, Closure $next)
40 40
     {
41
-        if (! $request->hasSession() || ! $this->user) {
41
+        if (!$request->hasSession() || !$this->user) {
42 42
             return $next($request);
43 43
         }
44 44
 
45 45
         if ($this->guard()->viaRemember()) {
46 46
             $passwordHash = explode('|', $request->cookies->get($this->guard()->getRecallerName()))[2] ?? null;
47 47
 
48
-            if (! $passwordHash || $passwordHash != $this->user->getAuthPassword()) {
48
+            if (!$passwordHash || $passwordHash != $this->user->getAuthPassword()) {
49 49
                 $this->logout($request);
50 50
             }
51 51
         }
52 52
 
53
-        if (! $request->session()->has('password_hash_'.backpack_guard_name())) {
53
+        if (!$request->session()->has('password_hash_'.backpack_guard_name())) {
54 54
             $this->storePasswordHashInSession($request);
55 55
         }
56 56
 
@@ -58,8 +58,8 @@  discard block
 block discarded – undo
58 58
             $this->logout($request);
59 59
         }
60 60
 
61
-        return tap($next($request), function () use ($request) {
62
-            if (! is_null($this->guard()->user())) {
61
+        return tap($next($request), function() use ($request) {
62
+            if (!is_null($this->guard()->user())) {
63 63
                 $this->storePasswordHashInSession($request);
64 64
             }
65 65
         });
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
      */
74 74
     protected function storePasswordHashInSession($request)
75 75
     {
76
-        if (! $this->user) {
76
+        if (!$this->user) {
77 77
             return;
78 78
         }
79 79
 
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/Relationships.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
         $model = $this->getRelationModel($entity, -1);
66 66
         $lastSegmentAfterDot = Str::of($field['entity'])->afterLast('.');
67 67
 
68
-        if (! method_exists($model, $lastSegmentAfterDot)) {
68
+        if (!method_exists($model, $lastSegmentAfterDot)) {
69 69
             return (string) Str::of($field['entity'])->beforeLast('.');
70 70
         }
71 71
 
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
 
87 87
         return collect($this->getCleanStateFields())
88 88
             ->whereIn('relation_type', $relation_types)
89
-            ->filter(function ($item) use ($nested) {
89
+            ->filter(function($item) use ($nested) {
90 90
                 if ($nested) {
91 91
                     return true;
92 92
                 }
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
      */
131 131
     private function getRelationFieldsWithoutRelationType($relations, $fields = [])
132 132
     {
133
-        if (! is_array($relations)) {
133
+        if (!is_array($relations)) {
134 134
             $relations = [$relations];
135 135
         }
136 136
 
@@ -139,8 +139,8 @@  discard block
 block discarded – undo
139 139
         }
140 140
 
141 141
         foreach ($relations as $relation) {
142
-            $fields = array_filter($fields, function ($field) use ($relation) {
143
-                if (! isset($field['relation_type'])) {
142
+            $fields = array_filter($fields, function($field) use ($relation) {
143
+                if (!isset($field['relation_type'])) {
144 144
                     return false;
145 145
                 }
146 146
 
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
                     $fields = array_merge($field['subfields'], $fields);
175 175
                 }
176 176
             }
177
-            $fields = array_filter($fields, function ($field) {
177
+            $fields = array_filter($fields, function($field) {
178 178
                 return isset($field['relation_type']) && $field['relation_type'] === 'BelongsTo';
179 179
             });
180 180
         }
@@ -243,8 +243,8 @@  discard block
 block discarded – undo
243 243
     {
244 244
         $all_relation_fields = $this->getRelationFields();
245 245
 
246
-        return Arr::where($all_relation_fields, function ($value, $key) {
247
-            return isset($value['pivot']) && ! $value['pivot'];
246
+        return Arr::where($all_relation_fields, function($value, $key) {
247
+            return isset($value['pivot']) && !$value['pivot'];
248 248
         });
249 249
     }
250 250
 
@@ -257,7 +257,7 @@  discard block
 block discarded – undo
257 257
     {
258 258
         $all_relation_fields = $this->getRelationFields();
259 259
 
260
-        return Arr::where($all_relation_fields, function ($value, $key) {
260
+        return Arr::where($all_relation_fields, function($value, $key) {
261 261
             return isset($value['pivot']) && $value['pivot'];
262 262
         });
263 263
     }
@@ -338,7 +338,7 @@  discard block
 block discarded – undo
338 338
         }
339 339
 
340 340
         // relationships are always public methods.
341
-        if (! $methodReflection->isPublic()) {
341
+        if (!$methodReflection->isPublic()) {
342 342
             return false;
343 343
         }
344 344
 
@@ -368,7 +368,7 @@  discard block
 block discarded – undo
368 368
      */
369 369
     private function isAttributeInRelationString($field)
370 370
     {
371
-        if (! str_contains($field['entity'], '.')) {
371
+        if (!str_contains($field['entity'], '.')) {
372 372
             return false;
373 373
         }
374 374
 
Please login to merge, or discard this patch.
src/ViewNamespaces.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
     public static function addFor(string $domain, $viewNamespaces)
30 30
     {
31 31
         foreach ((array) $viewNamespaces as $viewNamespace) {
32
-            if (! in_array($viewNamespace, self::getForDomain($domain))) {
32
+            if (!in_array($viewNamespace, self::getForDomain($domain))) {
33 33
                 self::$viewNamespaces[$domain][] = $viewNamespace;
34 34
             }
35 35
         }
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
      */
84 84
     public static function getViewPathsFor(string $domain, string $viewName)
85 85
     {
86
-        return array_map(function ($item) use ($viewName) {
86
+        return array_map(function($item) use ($viewName) {
87 87
             return $item.'.'.$viewName;
88 88
         }, self::getFor($domain));
89 89
     }
Please login to merge, or discard this patch.
tests/Unit/CrudPanel/CrudPanelTitlesAndHeadingsTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
     {
37 37
         $this->crudPanel->setModel(User::class);
38 38
         $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']);
39
-        $request->setRouteResolver(function () use ($request) {
39
+        $request->setRouteResolver(function() use ($request) {
40 40
             return (new Route('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request);
41 41
         });
42 42
         $this->crudPanel->setRequest($request);
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
     {
52 52
         $this->crudPanel->setModel(User::class);
53 53
         $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']);
54
-        $request->setRouteResolver(function () use ($request) {
54
+        $request->setRouteResolver(function() use ($request) {
55 55
             return (new Route('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request);
56 56
         });
57 57
         $this->crudPanel->setRequest($request);
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
     {
67 67
         $this->crudPanel->setModel(User::class);
68 68
         $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']);
69
-        $request->setRouteResolver(function () use ($request) {
69
+        $request->setRouteResolver(function() use ($request) {
70 70
             return (new Route('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request);
71 71
         });
72 72
         $this->crudPanel->setRequest($request);
Please login to merge, or discard this patch.
tests/Unit/CrudPanel/CrudPanelMacroTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -9,7 +9,7 @@  discard block
 block discarded – undo
9 9
 {
10 10
     public function testItCanRegisterMacro()
11 11
     {
12
-        $this->crudPanel::macro('validMacro', function () {
12
+        $this->crudPanel::macro('validMacro', function() {
13 13
             return true;
14 14
         });
15 15
 
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
     public function testThrowsErrorIfMacroExists()
20 20
     {
21 21
         try {
22
-            $this->crudPanel::macro('setModel', function () {
22
+            $this->crudPanel::macro('setModel', function() {
23 23
                 return true;
24 24
             });
25 25
         } catch (\Throwable $e) {
Please login to merge, or discard this patch.
tests/Unit/CrudPanel/CrudPanelCreateTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -605,7 +605,7 @@  discard block
 block discarded – undo
605 605
 
606 606
     public function testCreateBelongsToFake()
607 607
     {
608
-        $belongsToField = [   // select_grouped
608
+        $belongsToField = [// select_grouped
609 609
             'label'                      => 'Select_grouped',
610 610
             'type'                       => 'select_grouped', //https://github.com/Laravel-Backpack/CRUD/issues/502
611 611
             'name'                       => 'bang_relation_field',
@@ -741,7 +741,7 @@  discard block
 block discarded – undo
741 741
                 'nickname'        => 'i_have_has_one',
742 742
                 'profile_picture' => 'ohh my picture 1.jpg',
743 743
                 'article'         => 1,
744
-                'addresses'       => [ // HasOne is tested in other test function
744
+                'addresses'       => [// HasOne is tested in other test function
745 745
                     [
746 746
                         'city'   => 2,
747 747
                         'street' => 'test',
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/Validation.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -46,10 +46,10 @@  discard block
 block discarded – undo
46 46
     private function getValidationRulesFromFieldsAndSubfields($fields)
47 47
     {
48 48
         $rules = collect($fields)
49
-            ->filter(function ($value, $key) {
49
+            ->filter(function($value, $key) {
50 50
                 // only keep fields where 'validationRules' OR there are subfields
51 51
                 return array_key_exists('validationRules', $value) || array_key_exists('subfields', $value);
52
-            })->map(function ($item, $key) {
52
+            })->map(function($item, $key) {
53 53
                 $validationRules = [];
54 54
                 // only keep the rules, not the entire field definition
55 55
                 if (isset($item['validationRules'])) {
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
                 }
58 58
                 // add validation rules for subfields
59 59
                 if (array_key_exists('subfields', $item)) {
60
-                    $subfieldsWithValidation = array_filter($item['subfields'], function ($subfield) {
60
+                    $subfieldsWithValidation = array_filter($item['subfields'], function($subfield) {
61 61
                         return array_key_exists('validationRules', $subfield);
62 62
                     });
63 63
 
@@ -82,10 +82,10 @@  discard block
 block discarded – undo
82 82
     {
83 83
         $messages = [];
84 84
         collect($fields)
85
-            ->filter(function ($value, $key) {
85
+            ->filter(function($value, $key) {
86 86
                 // only keep fields where 'validationMessages' OR there are subfields
87 87
                 return array_key_exists('validationMessages', $value) || array_key_exists('subfields', $value);
88
-            })->each(function ($item, $key) use (&$messages) {
88
+            })->each(function($item, $key) use (&$messages) {
89 89
                 if (isset($item['validationMessages'])) {
90 90
                     foreach ($item['validationMessages'] as $rule => $message) {
91 91
                         $messages[$key.'.'.$rule] = $message;
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
                 }
94 94
                 // add messages from subfields
95 95
                 if (array_key_exists('subfields', $item)) {
96
-                    $subfieldsWithValidationMessages = array_filter($item['subfields'], function ($subfield) {
96
+                    $subfieldsWithValidationMessages = array_filter($item['subfields'], function($subfield) {
97 97
                         return array_key_exists('validationRules', $subfield);
98 98
                     });
99 99
 
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
      */
130 130
     public function setValidation($classOrRulesArray = false, $messages = [])
131 131
     {
132
-        if (! $classOrRulesArray) {
132
+        if (!$classOrRulesArray) {
133 133
             $this->setValidationFromFields();
134 134
         } elseif (is_array($classOrRulesArray)) {
135 135
             $this->setValidationFromArray($classOrRulesArray, $messages);
@@ -210,7 +210,7 @@  discard block
 block discarded – undo
210 210
             return $this->checkRequestValidity($extendedRules, $extendedMessages, $formRequest);
211 211
         }
212 212
 
213
-        return ! empty($rules) ? $this->checkRequestValidity($rules, $messages) : $this->getRequest();
213
+        return !empty($rules) ? $this->checkRequestValidity($rules, $messages) : $this->getRequest();
214 214
     }
215 215
 
216 216
     /**
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
     {
226 226
         $extendedRules = [];
227 227
         $requestRules = $this->getRequestRulesAsArray($request);
228
-        $rules = array_map(function ($ruleDefinition) {
228
+        $rules = array_map(function($ruleDefinition) {
229 229
             return is_array($ruleDefinition) ? $ruleDefinition : explode('|', $ruleDefinition);
230 230
         }, $rules);
231 231
 
@@ -322,7 +322,7 @@  discard block
 block discarded – undo
322 322
      */
323 323
     public function isRequired($inputKey)
324 324
     {
325
-        if (! $this->hasOperationSetting('requiredFields')) {
325
+        if (!$this->hasOperationSetting('requiredFields')) {
326 326
             return false;
327 327
         }
328 328
 
@@ -343,7 +343,7 @@  discard block
 block discarded – undo
343 343
     {
344 344
         [$rules, $messages] = $this->getValidationRulesAndMessagesFromField($field, $parent);
345 345
 
346
-        if (! empty($rules)) {
346
+        if (!empty($rules)) {
347 347
             $this->setValidation($rules, $messages);
348 348
         }
349 349
     }
Please login to merge, or discard this patch.