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 — fix-default-bottom-margin ( 153c18...0da20e )
by Pedro
11:45
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/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/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.
tests/Unit/CrudPanel/CrudPanelValidationTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
             'password' => 'test',
46 46
         ]);
47 47
 
48
-        $request->setRouteResolver(function () use ($request) {
48
+        $request->setRouteResolver(function() use ($request) {
49 49
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController', 'create']))->bind($request);
50 50
         });
51 51
 
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
             'email' => '',
86 86
         ]);
87 87
 
88
-        $request->setRouteResolver(function () use ($request) {
88
+        $request->setRouteResolver(function() use ($request) {
89 89
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController', 'create']))->bind($request);
90 90
         });
91 91
 
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
             'password' => 'test',
126 126
         ]);
127 127
 
128
-        $request->setRouteResolver(function () use ($request) {
128
+        $request->setRouteResolver(function() use ($request) {
129 129
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController', 'create']))->bind($request);
130 130
         });
131 131
 
@@ -247,7 +247,7 @@  discard block
 block discarded – undo
247 247
             'name' => 'test',
248 248
         ]);
249 249
 
250
-        $request->setRouteResolver(function () use ($request) {
250
+        $request->setRouteResolver(function() use ($request) {
251 251
             return (new Route('POST', 'users', ['Backpack\CRUD\Tests\Config\Http\Controllers\UserCrudController', 'create']))->bind($request);
252 252
         });
253 253
 
Please login to merge, or discard this patch.
src/app/Library/Uploaders/Support/Traits/HandleRepeatableUploads.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
 {
14 14
     public bool $handleRepeatableFiles = false;
15 15
 
16
-    public null|string $repeatableContainerName = null;
16
+    public null | string $repeatableContainerName = null;
17 17
 
18 18
     /*******************************
19 19
      * Setters - fluently configure the uploader
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
     /*******************************
31 31
      * Getters
32 32
      *******************************/
33
-    public function getRepeatableContainerName(): null|string
33
+    public function getRepeatableContainerName(): null | string
34 34
     {
35 35
         return $this->repeatableContainerName;
36 36
     }
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
         foreach (app('UploadersRepository')->getRepeatableUploadersFor($this->getRepeatableContainerName()) as $uploader) {
77 77
             $uploadedValues = $uploader->uploadRepeatableFiles($values->pluck($uploader->getAttributeName())->toArray(), $this->getPreviousRepeatableValues($entry, $uploader));
78 78
 
79
-            $values = $values->map(function ($item, $key) use ($uploadedValues, $uploader) {
79
+            $values = $values->map(function($item, $key) use ($uploadedValues, $uploader) {
80 80
                 $item[$uploader->getAttributeName()] = $uploadedValues[$key] ?? null;
81 81
 
82 82
                 return $item;
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
 
97 97
         $values = $entry->{$this->getRepeatableContainerName()};
98 98
         $values = is_string($values) ? json_decode($values, true) : $values;
99
-        $values = array_map(function ($item) use ($repeatableUploaders) {
99
+        $values = array_map(function($item) use ($repeatableUploaders) {
100 100
             foreach ($repeatableUploaders as $upload) {
101 101
                 $item[$upload->getAttributeName()] = $this->getValuesWithPathStripped($item, $upload);
102 102
             }
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
 
112 112
     private function retrieveRepeatableRelationFiles(Model $entry)
113 113
     {
114
-        switch($this->getRepeatableRelationType()) {
114
+        switch ($this->getRepeatableRelationType()) {
115 115
             case 'BelongsToMany':
116 116
             case 'MorphToMany':
117 117
                 $pivotClass = app('crud')->getModel()->{$this->getUploaderSubfield()['baseEntity']}()->getPivotClass();
@@ -157,12 +157,12 @@  discard block
 block discarded – undo
157 157
 
158 158
         $repeatableValues = collect($entry->{$this->getName()});
159 159
         foreach (app('UploadersRepository')->getRepeatableUploadersFor($this->getRepeatableContainerName()) as $upload) {
160
-            if (! $upload->shouldDeleteFiles()) {
160
+            if (!$upload->shouldDeleteFiles()) {
161 161
                 continue;
162 162
             }
163 163
             $values = $repeatableValues->pluck($upload->getName())->toArray();
164 164
             foreach ($values as $value) {
165
-                if (! $value) {
165
+                if (!$value) {
166 166
                     continue;
167 167
                 }
168 168
 
@@ -185,7 +185,7 @@  discard block
 block discarded – undo
185 185
     /**
186 186
      * Given two multidimensional arrays/collections, merge them recursively.
187 187
      */
188
-    protected function mergeValuesRecursive(array|Collection $array1, array|Collection $array2): array|Collection
188
+    protected function mergeValuesRecursive(array | Collection $array1, array | Collection $array2): array | Collection
189 189
     {
190 190
         $merged = $array1;
191 191
         foreach ($array2 as $key => &$value) {
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
     {
208 208
         $items = CRUD::getRequest()->input('_order_'.$this->getRepeatableContainerName()) ?? [];
209 209
 
210
-        array_walk($items, function (&$key, $value) {
210
+        array_walk($items, function(&$key, $value) {
211 211
             $requestValue = $key[$this->getName()] ?? null;
212 212
             $key = $this->handleMultipleFiles ? (is_string($requestValue) ? explode(',', $requestValue) : $requestValue) : $requestValue;
213 213
         });
@@ -219,18 +219,18 @@  discard block
 block discarded – undo
219 219
     {
220 220
         $previousValues = json_decode($entry->getOriginal($uploader->getRepeatableContainerName()), true);
221 221
 
222
-        if (! empty($previousValues)) {
222
+        if (!empty($previousValues)) {
223 223
             $previousValues = array_column($previousValues, $uploader->getName());
224 224
         }
225 225
 
226 226
         return $previousValues ?? [];
227 227
     }
228 228
 
229
-    private function getValuesWithPathStripped(array|string|null $item, UploaderInterface $upload)
229
+    private function getValuesWithPathStripped(array | string | null $item, UploaderInterface $upload)
230 230
     {
231 231
         $uploadedValues = $item[$upload->getName()] ?? null;
232 232
         if (is_array($uploadedValues)) {
233
-            return array_map(function ($value) use ($upload) {
233
+            return array_map(function($value) use ($upload) {
234 234
                 return Str::after($value, $upload->getPath());
235 235
             }, $uploadedValues);
236 236
         }
@@ -242,19 +242,19 @@  discard block
 block discarded – undo
242 242
     {
243 243
         if (in_array($this->getRepeatableRelationType(), ['BelongsToMany', 'MorphToMany'])) {
244 244
             $pivotAttributes = $entry->getAttributes();
245
-            $connectedPivot = $entry->pivotParent->{$this->getRepeatableContainerName()}->where(function ($item) use ($pivotAttributes) {
245
+            $connectedPivot = $entry->pivotParent->{$this->getRepeatableContainerName()}->where(function($item) use ($pivotAttributes) {
246 246
                 $itemPivotAttributes = $item->pivot->only(array_keys($pivotAttributes));
247 247
 
248 248
                 return $itemPivotAttributes === $pivotAttributes;
249 249
             })->first();
250 250
 
251
-            if (! $connectedPivot) {
251
+            if (!$connectedPivot) {
252 252
                 return;
253 253
             }
254 254
 
255 255
             $files = $connectedPivot->getOriginal()['pivot_'.$this->getAttributeName()];
256 256
 
257
-            if (! $files) {
257
+            if (!$files) {
258 258
                 return;
259 259
             }
260 260
 
Please login to merge, or discard this patch.