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 — operation-hooks ( eac7c4...b4d208 )
by Pedro
35:18 queued 20:24
created
src/helpers.php 1 patch
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -5,7 +5,7 @@  discard block
 block discarded – undo
5 5
 use Illuminate\Support\Facades\Log;
6 6
 use Illuminate\Support\Str;
7 7
 
8
-if (! function_exists('backpack_url')) {
8
+if (!function_exists('backpack_url')) {
9 9
     /**
10 10
      * Appends the configured backpack prefix and returns
11 11
      * the URL using the standard Laravel helpers.
@@ -15,13 +15,13 @@  discard block
 block discarded – undo
15 15
      */
16 16
     function backpack_url($path = null, $parameters = [], $secure = null)
17 17
     {
18
-        $path = ! $path || (substr($path, 0, 1) == '/') ? $path : '/'.$path;
18
+        $path = !$path || (substr($path, 0, 1) == '/') ? $path : '/'.$path;
19 19
 
20 20
         return url(config('backpack.base.route_prefix', 'admin').$path, $parameters, $secure);
21 21
     }
22 22
 }
23 23
 
24
-if (! function_exists('backpack_authentication_column')) {
24
+if (!function_exists('backpack_authentication_column')) {
25 25
     /**
26 26
      * Return the username column name.
27 27
      * The Laravel default (and Backpack default) is 'email'.
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
     }
35 35
 }
36 36
 
37
-if (! function_exists('backpack_email_column')) {
37
+if (!function_exists('backpack_email_column')) {
38 38
     /**
39 39
      * Return the email column name.
40 40
      * The Laravel default (and Backpack default) is 'email'.
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
     }
48 48
 }
49 49
 
50
-if (! function_exists('backpack_form_input')) {
50
+if (!function_exists('backpack_form_input')) {
51 51
     /**
52 52
      * Parse the submitted input in request('form') to an usable array.
53 53
      * Joins the multiple[] fields in a single key and transform the dot notation fields into arrayed ones.
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
     }
121 121
 }
122 122
 
123
-if (! function_exists('backpack_users_have_email')) {
123
+if (!function_exists('backpack_users_have_email')) {
124 124
     /**
125 125
      * Check if the email column is present on the user table.
126 126
      *
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
     }
136 136
 }
137 137
 
138
-if (! function_exists('backpack_avatar_url')) {
138
+if (!function_exists('backpack_avatar_url')) {
139 139
     /**
140 140
      * Returns the avatar URL of a user.
141 141
      *
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
     {
147 147
         switch (config('backpack.base.avatar_type')) {
148 148
             case 'gravatar':
149
-                if (backpack_users_have_email() && ! empty($user->email)) {
149
+                if (backpack_users_have_email() && !empty($user->email)) {
150 150
                     $avatarLink = Gravatar::fallback(config('backpack.base.gravatar_fallback'))->get($user->email, ['size' => 80]);
151 151
 
152 152
                     // if we can save it locally, for safer loading, let's do it
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
     }
165 165
 }
166 166
 
167
-if (! function_exists('backpack_middleware')) {
167
+if (!function_exists('backpack_middleware')) {
168 168
     /**
169 169
      * Return the key of the middleware used across Backpack.
170 170
      * That middleware checks if the visitor is an admin.
@@ -178,7 +178,7 @@  discard block
 block discarded – undo
178 178
     }
179 179
 }
180 180
 
181
-if (! function_exists('backpack_guard_name')) {
181
+if (!function_exists('backpack_guard_name')) {
182 182
     /*
183 183
      * Returns the name of the guard defined
184 184
      * by the application config
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
     }
190 190
 }
191 191
 
192
-if (! function_exists('backpack_auth')) {
192
+if (!function_exists('backpack_auth')) {
193 193
     /*
194 194
      * Returns the user instance if it exists
195 195
      * of the currently authenticated admin
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
     }
202 202
 }
203 203
 
204
-if (! function_exists('backpack_user')) {
204
+if (!function_exists('backpack_user')) {
205 205
     /*
206 206
      * Returns back a user instance without
207 207
      * the admin guard, however allows you
@@ -213,7 +213,7 @@  discard block
 block discarded – undo
213 213
     }
214 214
 }
215 215
 
216
-if (! function_exists('mb_ucfirst')) {
216
+if (!function_exists('mb_ucfirst')) {
217 217
     /**
218 218
      * Capitalize the first letter of a string,
219 219
      * even if that string is multi-byte (non-latin alphabet).
@@ -234,7 +234,7 @@  discard block
 block discarded – undo
234 234
     }
235 235
 }
236 236
 
237
-if (! function_exists('backpack_view')) {
237
+if (!function_exists('backpack_view')) {
238 238
     /**
239 239
      * Returns a new displayable view path, based on the configured backpack view namespace.
240 240
      * If that view doesn't exist, it falls back to the fallback namespace.
@@ -259,8 +259,8 @@  discard block
 block discarded – undo
259 259
 
260 260
         $errorMessage = 'The view: ['.$view.'] was not found in any of the following view paths: ['.implode(' ], [ ', $viewPaths).']';
261 261
 
262
-        $errorDetails = (function () {
263
-            if (env('APP_ENV') === 'production' || ! env('APP_DEBUG')) {
262
+        $errorDetails = (function() {
263
+            if (env('APP_ENV') === 'production' || !env('APP_DEBUG')) {
264 264
                 return '';
265 265
             }
266 266
 
@@ -276,7 +276,7 @@  discard block
 block discarded – undo
276 276
     }
277 277
 }
278 278
 
279
-if (! function_exists('backpack_theme_config')) {
279
+if (!function_exists('backpack_theme_config')) {
280 280
     /**
281 281
      * Returns a config value from the current theme's config file.
282 282
      * It assumes the theme's config namespace is the same as the view namespace.
@@ -315,7 +315,7 @@  discard block
 block discarded – undo
315 315
     }
316 316
 }
317 317
 
318
-if (! function_exists('square_brackets_to_dots')) {
318
+if (!function_exists('square_brackets_to_dots')) {
319 319
     /**
320 320
      * Turns a string from bracket-type array to dot-notation array.
321 321
      * Ex: array[0][property] turns into array.0.property.
@@ -331,7 +331,7 @@  discard block
 block discarded – undo
331 331
     }
332 332
 }
333 333
 
334
-if (! function_exists('old_empty_or_null')) {
334
+if (!function_exists('old_empty_or_null')) {
335 335
     /**
336 336
      * This method is an alternative to Laravel's old() helper, which mistakenly
337 337
      * returns NULL it two cases:
@@ -363,7 +363,7 @@  discard block
 block discarded – undo
363 363
     }
364 364
 }
365 365
 
366
-if (! function_exists('is_multidimensional_array')) {
366
+if (!function_exists('is_multidimensional_array')) {
367 367
     /**
368 368
      * Check if the array is multidimensional.
369 369
      *
@@ -372,10 +372,10 @@  discard block
 block discarded – undo
372 372
     function is_multidimensional_array(array $array, bool $strict = false): bool
373 373
     {
374 374
         foreach ($array as $item) {
375
-            if ($strict && ! is_array($item)) {
375
+            if ($strict && !is_array($item)) {
376 376
                 return false;
377 377
             }
378
-            if (! $strict && is_array($item)) {
378
+            if (!$strict && is_array($item)) {
379 379
                 return true;
380 380
             }
381 381
         }
@@ -383,7 +383,7 @@  discard block
 block discarded – undo
383 383
     }
384 384
 }
385 385
 
386
-if (! function_exists('backpack_pro')) {
386
+if (!function_exists('backpack_pro')) {
387 387
     /**
388 388
      * Check if the backpack/pro package is installed.
389 389
      *
@@ -394,7 +394,7 @@  discard block
 block discarded – undo
394 394
         if (app()->runningUnitTests()) {
395 395
             return true;
396 396
         }
397
-        if (! \Composer\InstalledVersions::isInstalled('backpack/pro')) {
397
+        if (!\Composer\InstalledVersions::isInstalled('backpack/pro')) {
398 398
             return false;
399 399
         }
400 400
 
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.