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 — add-tests ( 194aa8 )
by Pedro
15:05
created
src/app/Library/CrudPanel/Traits/FieldsProtectedMethods.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
             return ['name' => Str::replace(' ', '', $field)];
116 116
         }
117 117
 
118
-        if (is_array($field) && ! isset($field['name'])) {
118
+        if (is_array($field) && !isset($field['name'])) {
119 119
             abort(500, 'All fields must have their name defined');
120 120
         }
121 121
 
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
         }
190 190
         // if there's a model defined, but no attribute
191 191
         // guess an attribute using the identifiableAttribute functionality in CrudTrait
192
-        if (isset($field['model']) && ! isset($field['attribute']) && method_exists($field['model'], 'identifiableAttribute')) {
192
+        if (isset($field['model']) && !isset($field['attribute']) && method_exists($field['model'], 'identifiableAttribute')) {
193 193
             $field['attribute'] = (new $field['model']())->identifiableAttribute();
194 194
         }
195 195
 
@@ -205,7 +205,7 @@  discard block
 block discarded – undo
205 205
      */
206 206
     protected function makeSureFieldHasLabel($field)
207 207
     {
208
-        if (! isset($field['label'])) {
208
+        if (!isset($field['label'])) {
209 209
             $name = str_replace(',', ' ', $field['name']);
210 210
             $name = str_replace('_id', '', $name);
211 211
             $field['label'] = mb_ucfirst(str_replace('_', ' ', $name));
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
      */
224 224
     protected function makeSureFieldHasType($field)
225 225
     {
226
-        if (! isset($field['type'])) {
226
+        if (!isset($field['type'])) {
227 227
             $field['type'] = isset($field['relation_type']) ? $this->inferFieldTypeFromRelationType($field['relation_type']) : $this->inferFieldTypeFromDbColumnType($field['name']);
228 228
         }
229 229
 
@@ -256,16 +256,16 @@  discard block
 block discarded – undo
256 256
      */
257 257
     protected function makeSureSubfieldsHaveNecessaryAttributes($field)
258 258
     {
259
-        if (! isset($field['subfields']) || ! is_array($field['subfields'])) {
259
+        if (!isset($field['subfields']) || !is_array($field['subfields'])) {
260 260
             return $field;
261 261
         }
262 262
 
263
-        if (! is_multidimensional_array($field['subfields'], true)) {
263
+        if (!is_multidimensional_array($field['subfields'], true)) {
264 264
             abort(500, 'Subfields of «'.$field['name'].'» are malformed. Make sure you provide an array of subfields.');
265 265
         }
266 266
 
267 267
         foreach ($field['subfields'] as $key => $subfield) {
268
-            if (empty($subfield) || ! isset($subfield['name'])) {
268
+            if (empty($subfield) || !isset($subfield['name'])) {
269 269
                 abort(500, 'A subfield of «'.$field['name'].'» is malformed. Subfield attribute name can\'t be empty.');
270 270
             }
271 271
 
@@ -278,7 +278,7 @@  discard block
 block discarded – undo
278 278
 
279 279
             $subfield['parentFieldName'] = $field['name'];
280 280
 
281
-            if (! isset($field['model'])) {
281
+            if (!isset($field['model'])) {
282 282
                 // we're inside a simple 'repeatable' with no model/relationship, so
283 283
                 // we assume all subfields are supposed to be text fields
284 284
                 $subfield['type'] = $subfield['type'] ?? 'text';
@@ -305,11 +305,11 @@  discard block
 block discarded – undo
305 305
                 case 'BelongsToMany':
306 306
                     $pivotSelectorField = static::getPivotFieldStructure($field);
307 307
 
308
-                    $pivot = Arr::where($field['subfields'], function ($item) use ($pivotSelectorField) {
308
+                    $pivot = Arr::where($field['subfields'], function($item) use ($pivotSelectorField) {
309 309
                         return $item['name'] === $pivotSelectorField['name'];
310 310
                     });
311 311
 
312
-                    if (! empty($pivot)) {
312
+                    if (!empty($pivot)) {
313 313
                         break;
314 314
                     }
315 315
 
@@ -322,11 +322,11 @@  discard block
 block discarded – undo
322 322
                     $entity = isset($field['baseEntity']) ? $field['baseEntity'].'.'.$field['entity'] : $field['entity'];
323 323
                     $relationInstance = $this->getRelationInstance(['entity' => $entity]);
324 324
 
325
-                    $localKeyField = Arr::where($field['subfields'], function ($item) use ($relationInstance) {
325
+                    $localKeyField = Arr::where($field['subfields'], function($item) use ($relationInstance) {
326 326
                         return $item['name'] === $relationInstance->getRelated()->getKeyName();
327 327
                     });
328 328
 
329
-                    if (! empty($localKeyField)) {
329
+                    if (!empty($localKeyField)) {
330 330
                         break;
331 331
                     }
332 332
 
@@ -351,7 +351,7 @@  discard block
 block discarded – undo
351 351
     {
352 352
         // if a tab was mentioned, we should enable it
353 353
         if (isset($field['tab'])) {
354
-            if (! $this->tabsEnabled()) {
354
+            if (!$this->tabsEnabled()) {
355 355
                 $this->enableTabs();
356 356
             }
357 357
         }
Please login to merge, or discard this patch.
src/app/Http/Middleware/EnsureEmailVerification.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -24,13 +24,13 @@
 block discarded – undo
24 24
 
25 25
         // the Laravel middleware needs the user resolver to be set with the backpack guard
26 26
         $userResolver = $request->getUserResolver();
27
-        $request->setUserResolver(function () use ($userResolver) {
27
+        $request->setUserResolver(function() use ($userResolver) {
28 28
             return $userResolver(backpack_guard_name());
29 29
         });
30 30
 
31 31
         try {
32 32
             $verifiedMiddleware = new (app('router')->getMiddleware()['verified'])();
33
-        } catch(Throwable) {
33
+        } catch (Throwable) {
34 34
             throw new Exception('Missing "verified" alias middleware in App/Http/Kernel.php. More info: https://backpackforlaravel.com/docs/6.x/base-how-to#enable-email-verification-in-backpack-routes');
35 35
         }
36 36
 
Please login to merge, or discard this patch.
src/app/Library/Auth/AuthenticatesUsers.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -205,7 +205,7 @@
 block discarded – undo
205 205
         return Auth::guard();
206 206
     }
207 207
 
208
-    private function logoutIfEmailNotVerified(Request $request): Response|\Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector
208
+    private function logoutIfEmailNotVerified(Request $request): Response | \Illuminate\Http\RedirectResponse | \Illuminate\Routing\Redirector
209 209
     {
210 210
         $user = $this->guard()->user();
211 211
 
Please login to merge, or discard this patch.
src/app/Http/Controllers/Auth/VerifyEmailController.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -11,7 +11,7 @@  discard block
 block discarded – undo
11 11
 
12 12
 class VerifyEmailController extends Controller
13 13
 {
14
-    public null|string $redirectTo = null;
14
+    public null | string $redirectTo = null;
15 15
 
16 16
     /**
17 17
      * Create a new controller instance.
@@ -20,21 +20,21 @@  discard block
 block discarded – undo
20 20
      */
21 21
     public function __construct()
22 22
     {
23
-        if (! app('router')->getMiddleware()['signed'] ?? null) {
23
+        if (!app('router')->getMiddleware()['signed'] ?? null) {
24 24
             throw new Exception('Missing "signed" alias middleware in App/Http/Kernel.php. More info: https://backpackforlaravel.com/docs/6.x/base-how-to#enable-email-verification-in-backpack-routes');
25 25
         }
26 26
 
27 27
         $this->middleware('signed')->only('verifyEmail');
28 28
         $this->middleware('throttle:'.config('backpack.base.email_verification_throttle_access'))->only('resendVerificationEmail');
29 29
 
30
-        if (! backpack_users_have_email()) {
30
+        if (!backpack_users_have_email()) {
31 31
             abort(500, trans('backpack::base.no_email_column'));
32 32
         }
33 33
         // where to redirect after the email is verified
34 34
         $this->redirectTo = $this->redirectTo ?? backpack_url('dashboard');
35 35
     }
36 36
 
37
-    public function emailVerificationRequired(Request $request): \Illuminate\Contracts\View\View|\Illuminate\Http\RedirectResponse
37
+    public function emailVerificationRequired(Request $request): \Illuminate\Contracts\View\View | \Illuminate\Http\RedirectResponse
38 38
     {
39 39
         $this->getUserOrRedirect($request);
40 40
 
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
         return $request->user(backpack_guard_name()) ?? (new UserFromCookie())();
78 78
     }
79 79
 
80
-    private function getUserOrRedirect(Request $request): \Illuminate\Contracts\Auth\MustVerifyEmail|\Illuminate\Http\RedirectResponse
80
+    private function getUserOrRedirect(Request $request): \Illuminate\Contracts\Auth\MustVerifyEmail | \Illuminate\Http\RedirectResponse
81 81
     {
82 82
         if ($user = $this->getUser($request)) {
83 83
             return $user;
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
         }
@@ -241,8 +241,8 @@  discard block
 block discarded – undo
241 241
     {
242 242
         $all_relation_fields = $this->getRelationFields();
243 243
 
244
-        return Arr::where($all_relation_fields, function ($value, $key) {
245
-            return isset($value['pivot']) && ! $value['pivot'];
244
+        return Arr::where($all_relation_fields, function($value, $key) {
245
+            return isset($value['pivot']) && !$value['pivot'];
246 246
         });
247 247
     }
248 248
 
@@ -255,7 +255,7 @@  discard block
 block discarded – undo
255 255
     {
256 256
         $all_relation_fields = $this->getRelationFields();
257 257
 
258
-        return Arr::where($all_relation_fields, function ($value, $key) {
258
+        return Arr::where($all_relation_fields, function($value, $key) {
259 259
             return isset($value['pivot']) && $value['pivot'];
260 260
         });
261 261
     }
@@ -336,7 +336,7 @@  discard block
 block discarded – undo
336 336
         }
337 337
 
338 338
         // relationships are always public methods.
339
-        if (! $methodReflection->isPublic()) {
339
+        if (!$methodReflection->isPublic()) {
340 340
             return false;
341 341
         }
342 342
 
@@ -363,7 +363,7 @@  discard block
 block discarded – undo
363 363
      */
364 364
     public function isAttributeInRelationString(array $field): bool
365 365
     {
366
-        if (! str_contains($field['entity'], '.')) {
366
+        if (!str_contains($field['entity'], '.')) {
367 367
             return false;
368 368
         }
369 369
 
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/Support/MacroableWithAttributes.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@
 block discarded – undo
44 44
                                         ->filter(fn ($item) => isset($item[$macro]));
45 45
 
46 46
                 $subfieldsWithMacros->each(
47
-                    function ($item) use ($subfieldsWithMacros, $macro) {
47
+                    function($item) use ($subfieldsWithMacros, $macro) {
48 48
                         if ($subfieldsWithMacros->last() === $item) {
49 49
                             $this->{$macro}($item[$macro], $item);
50 50
                         } else {
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/Access.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@  discard block
 block discarded – undo
10 10
     /**
11 11
      * Set an operation as having access using the Settings API.
12 12
      */
13
-    public function allowAccess(array|string $operation): bool
13
+    public function allowAccess(array | string $operation): bool
14 14
     {
15 15
         foreach ((array) $operation as $op) {
16 16
             $this->set($op.'.access', true);
@@ -22,13 +22,13 @@  discard block
 block discarded – undo
22 22
     /**
23 23
      * Disable the access to a certain operation, or the current one.
24 24
      */
25
-    public function denyAccess(array|string $operation): bool
25
+    public function denyAccess(array | string $operation): bool
26 26
     {
27 27
         foreach ((array) $operation as $op) {
28 28
             $this->set($op.'.access', false);
29 29
         }
30 30
 
31
-        return ! $this->hasAccessToAny($operation);
31
+        return !$this->hasAccessToAny($operation);
32 32
     }
33 33
 
34 34
     /**
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
     /**
53 53
      * Check if any operations are allowed for a Crud Panel. Return false if not.
54 54
      */
55
-    public function hasAccessToAny(array|string $operation_array, ?Model $entry = null): bool
55
+    public function hasAccessToAny(array | string $operation_array, ?Model $entry = null): bool
56 56
     {
57 57
         foreach ((array) $operation_array as $key => $operation) {
58 58
             if ($this->hasAccess($operation, $entry) == true) {
@@ -66,10 +66,10 @@  discard block
 block discarded – undo
66 66
     /**
67 67
      * Check if all operations are allowed for a Crud Panel. Return false if not.
68 68
      */
69
-    public function hasAccessToAll(array|string $operation_array, ?Model $entry = null): bool
69
+    public function hasAccessToAll(array | string $operation_array, ?Model $entry = null): bool
70 70
     {
71 71
         foreach ((array) $operation_array as $key => $operation) {
72
-            if (! $this->hasAccess($operation, $entry)) {
72
+            if (!$this->hasAccess($operation, $entry)) {
73 73
                 return false;
74 74
             }
75 75
         }
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
      */
85 85
     public function hasAccessOrFail(string $operation, ?Model $entry = null): bool
86 86
     {
87
-        if (! $this->hasAccess($operation, $entry)) {
87
+        if (!$this->hasAccess($operation, $entry)) {
88 88
             throw new AccessDeniedException(trans('backpack::crud.unauthorized_access', ['access' => $operation]));
89 89
         }
90 90
 
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
      * Get an operation's access condition, if set. A condition
96 96
      * can be anything, but usually a boolean or a callable.
97 97
      */
98
-    public function getAccessCondition(string $operation): bool|callable|null
98
+    public function getAccessCondition(string $operation): bool | callable | null
99 99
     {
100 100
         return $this->get($operation.'.access');
101 101
     }
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
     /**
104 104
      * Set the condition under which an operation is allowed for a Crud Panel.
105 105
      */
106
-    public function setAccessCondition(array|string $operation, bool|callable|null $condition): void
106
+    public function setAccessCondition(array | string $operation, bool | callable | null $condition): void
107 107
     {
108 108
         foreach ((array) $operation as $op) {
109 109
             $this->set($op.'.access', $condition);
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
      * Allow access only to operations in the array.
132 132
      * By denying access to all other operations.
133 133
      */
134
-    public function allowAccessOnlyTo(array|string $operation): void
134
+    public function allowAccessOnlyTo(array | string $operation): void
135 135
     {
136 136
         $this->denyAllAccess();
137 137
         $this->allowAccess($operation);
Please login to merge, or discard this patch.
src/routes/backpack/base.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@
 block discarded – undo
18 18
         'middleware' => config('backpack.base.web_middleware', 'web'),
19 19
         'prefix' => config('backpack.base.route_prefix'),
20 20
     ],
21
-    function () {
21
+    function() {
22 22
         // if not otherwise configured, setup the auth routes
23 23
         if (config('backpack.base.setup_auth_routes')) {
24 24
             // Authentication Routes...
Please login to merge, or discard this patch.
src/routes/backpack/testing.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@
 block discarded – undo
15 15
     (array) config('backpack.base.middleware_key', 'admin'),
16 16
     'prefix' => config('backpack.base.route_prefix', 'admin'),
17 17
 ],
18
-    function () {
18
+    function() {
19 19
         Route::crud('users', 'Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController');
20 20
     }
21 21
 );
Please login to merge, or discard this patch.