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

Test Failed
Pull Request — main (#5478)
by Pedro
27:58
created
src/resources/views/crud/reorder.blade.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
     <?php
28 28
     function tree_element($entry, $key, $all_entries, $crud)
29 29
     {
30
-        if (! isset($entry->tree_element_shown)) {
30
+        if (!isset($entry->tree_element_shown)) {
31 31
             // mark the element as shown
32 32
             $all_entries[$key]->tree_element_shown = true;
33 33
             $entry->tree_element_shown = true;
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
                 <ol class="sortable mt-0 mb-0">
71 71
                     <?php
72 72
                     $all_entries = collect($entries->all())->sortBy('lft')->keyBy($crud->getModel()->getKeyName());
73
-    $root_entries = $all_entries->filter(function ($item) {
73
+    $root_entries = $all_entries->filter(function($item) {
74 74
         return $item->parent_id == 0;
75 75
     });
76 76
     foreach ($root_entries as $key => $entry) {
Please login to merge, or discard this patch.
src/app/Console/Commands/AddMenuContent.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
         $this->progressBlock("Adding menu entry to <fg=blue>$path</>");
50 50
 
51 51
         // Validate file exists
52
-        if (! $disk->exists($path)) {
52
+        if (!$disk->exists($path)) {
53 53
             $this->errorProgressBlock();
54 54
             $this->note('The menu_items file does not exist. Make sure Backpack is properly installed.', 'red');
55 55
 
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
             return;
67 67
         }
68 68
 
69
-        if (! $disk->put($path, $contents.PHP_EOL.$code)) {
69
+        if (!$disk->put($path, $contents.PHP_EOL.$code)) {
70 70
             $this->errorProgressBlock();
71 71
             $this->note('Could not write to menu_items file.', 'red');
72 72
 
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
      */
86 86
     private function getLastLineNumberThatContains($needle, $haystack)
87 87
     {
88
-        $matchingLines = array_filter($haystack, function ($k) use ($needle) {
88
+        $matchingLines = array_filter($haystack, function($k) use ($needle) {
89 89
             return strpos($k, $needle) !== false;
90 90
         });
91 91
 
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/CrudField.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -226,7 +226,7 @@  discard block
 block discarded – undo
226 226
      */
227 227
     public function subfields($subfields)
228 228
     {
229
-        $callAttributeMacro = ! isset($this->attributes['subfields']);
229
+        $callAttributeMacro = !isset($this->attributes['subfields']);
230 230
         $this->attributes['subfields'] = $subfields;
231 231
         $this->attributes = $this->crud()->makeSureFieldHasNecessaryAttributes($this->attributes);
232 232
         if ($callAttributeMacro) {
@@ -276,7 +276,7 @@  discard block
 block discarded – undo
276 276
         // append the field name to the rule name of validationMessages array.
277 277
         // eg: ['required => 'This field is required']
278 278
         // will be transformed into: ['field_name.required' => 'This field is required]
279
-        $this->crud()->setValidationFromArray([], array_merge(...array_map(function ($rule, $message) {
279
+        $this->crud()->setValidationFromArray([], array_merge(...array_map(function($rule, $message) {
280 280
             return [$this->attributes['name'].'.'.$rule => $message];
281 281
         }, array_keys($messages), $messages)));
282 282
 
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/CrudButton.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -299,7 +299,7 @@  discard block
 block discarded – undo
299 299
     private function getViewPathsWithFallbacks()
300 300
     {
301 301
         $type = $this->name;
302
-        $paths = array_map(function ($item) use ($type) {
302
+        $paths = array_map(function($item) use ($type) {
303 303
             return $item.'.'.$type;
304 304
         }, ViewNamespaces::getFor('buttons'));
305 305
 
@@ -467,7 +467,7 @@  discard block
 block discarded – undo
467 467
 
468 468
         $itemExists = $this->collection()->contains('name', $this->name);
469 469
 
470
-        if (! $itemExists) {
470
+        if (!$itemExists) {
471 471
             $this->crud()->addCrudButton($this);
472 472
             if ($this->position == 'beginning') {
473 473
                 $this->before($this->collection()->first()->name);
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/Validation.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
      */
67 67
     public function setValidation($classOrRulesArray = false, $messages = [], $attributes = [])
68 68
     {
69
-        if (! $classOrRulesArray) {
69
+        if (!$classOrRulesArray) {
70 70
             $this->setValidationFromFields();
71 71
         } elseif (is_array($classOrRulesArray)) {
72 72
             $this->setValidationFromArray($classOrRulesArray, $messages, $attributes);
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
             return $this->checkRequestValidity($extendedRules, $extendedMessages, $extendedAttributes, $formRequest);
148 148
         }
149 149
 
150
-        return ! empty($rules) ? $this->checkRequestValidity($rules, $messages, $attributes) : $this->getRequest();
150
+        return !empty($rules) ? $this->checkRequestValidity($rules, $messages, $attributes) : $this->getRequest();
151 151
     }
152 152
 
153 153
     /**
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
                 if (is_string($validationRules)) {
196 196
                     $validationRules = explode('|', $validationRules);
197 197
                 }
198
-                if (! is_array($validationRules)) {
198
+                if (!is_array($validationRules)) {
199 199
                     $validationRules = [$validationRules];
200 200
                 }
201 201
                 foreach ($validationRules as $rule) {
@@ -238,7 +238,7 @@  discard block
 block discarded – undo
238 238
      */
239 239
     public function isRequired($inputKey)
240 240
     {
241
-        if (! $this->hasOperationSetting('requiredFields')) {
241
+        if (!$this->hasOperationSetting('requiredFields')) {
242 242
             return false;
243 243
         }
244 244
 
@@ -259,7 +259,7 @@  discard block
 block discarded – undo
259 259
     {
260 260
         [$rules, $messages, $attributes] = $this->getValidationDataFromField($field, $parent);
261 261
 
262
-        if (! empty($rules)) {
262
+        if (!empty($rules)) {
263 263
             $this->setValidation($rules, $messages, $attributes);
264 264
         }
265 265
     }
@@ -274,10 +274,10 @@  discard block
 block discarded – undo
274 274
     {
275 275
         $messages = [];
276 276
         collect($fields)
277
-            ->filter(function ($value, $key) {
277
+            ->filter(function($value, $key) {
278 278
                 // only keep fields where 'validationMessages' OR there are subfields
279 279
                 return array_key_exists('validationMessages', $value) || array_key_exists('subfields', $value);
280
-            })->each(function ($item, $key) use (&$messages) {
280
+            })->each(function($item, $key) use (&$messages) {
281 281
                 if (isset($item['validationMessages'])) {
282 282
                     foreach ($item['validationMessages'] as $rule => $message) {
283 283
                         $messages[$key.'.'.$rule] = $message;
@@ -285,7 +285,7 @@  discard block
 block discarded – undo
285 285
                 }
286 286
                 // add messages from subfields
287 287
                 if (array_key_exists('subfields', $item)) {
288
-                    $subfieldsWithValidationMessages = array_filter($item['subfields'], function ($subfield) {
288
+                    $subfieldsWithValidationMessages = array_filter($item['subfields'], function($subfield) {
289 289
                         return array_key_exists('validationRules', $subfield);
290 290
                     });
291 291
 
@@ -310,16 +310,16 @@  discard block
 block discarded – undo
310 310
     {
311 311
         $attributes = [];
312 312
         collect($fields)
313
-            ->filter(function ($value, $key) {
313
+            ->filter(function($value, $key) {
314 314
                 // only keep fields where 'validationAttribute' exists OR there are subfields
315 315
                 return array_key_exists('validationAttribute', $value) || array_key_exists('subfields', $value);
316
-            })->each(function ($item, $key) use (&$attributes) {
316
+            })->each(function($item, $key) use (&$attributes) {
317 317
                 if (isset($item['validationAttribute'])) {
318 318
                     $attributes[$key] = $item['validationAttribute'];
319 319
                 }
320 320
                 // add attributes from subfields
321 321
                 if (array_key_exists('subfields', $item)) {
322
-                    $subfieldsWithValidationAttribute = array_filter($item['subfields'], function ($subfield) {
322
+                    $subfieldsWithValidationAttribute = array_filter($item['subfields'], function($subfield) {
323 323
                         return array_key_exists('validationAttribute', $subfield);
324 324
                     });
325 325
 
@@ -341,10 +341,10 @@  discard block
 block discarded – undo
341 341
     private function getValidationRulesFromFieldsAndSubfields($fields)
342 342
     {
343 343
         $rules = collect($fields)
344
-            ->filter(function ($value, $key) {
344
+            ->filter(function($value, $key) {
345 345
                 // only keep fields where 'validationRules' OR there are subfields
346 346
                 return array_key_exists('validationRules', $value) || array_key_exists('subfields', $value);
347
-            })->map(function ($item, $key) {
347
+            })->map(function($item, $key) {
348 348
                 $validationRules = [];
349 349
                 // only keep the rules, not the entire field definition
350 350
                 if (isset($item['validationRules'])) {
@@ -352,7 +352,7 @@  discard block
 block discarded – undo
352 352
                 }
353 353
                 // add validation rules for subfields
354 354
                 if (array_key_exists('subfields', $item)) {
355
-                    $subfieldsWithValidation = array_filter($item['subfields'], function ($subfield) {
355
+                    $subfieldsWithValidation = array_filter($item['subfields'], function($subfield) {
356 356
                         return array_key_exists('validationRules', $subfield);
357 357
                     });
358 358
 
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/Buttons.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
         $otherButtons = collect([]);
25 25
 
26 26
         // we get the buttons that belong to the specified stack
27
-        $stackButtons = $this->buttons()->reject(function ($item) use ($stack, $otherButtons) {
27
+        $stackButtons = $this->buttons()->reject(function($item) use ($stack, $otherButtons) {
28 28
             if ($item->stack != $stack) {
29 29
                 // if the button does not belong to this stack we just add it for merging later
30 30
                 $otherButtons->push($item);
@@ -36,8 +36,8 @@  discard block
 block discarded – undo
36 36
         });
37 37
 
38 38
         // we parse the ordered buttons
39
-        collect($order)->each(function ($btnKey) use ($newButtons, $stackButtons) {
40
-            if (! $button = $stackButtons->where('name', $btnKey)->first()) {
39
+        collect($order)->each(function($btnKey) use ($newButtons, $stackButtons) {
40
+            if (!$button = $stackButtons->where('name', $btnKey)->first()) {
41 41
                 abort(500, 'Button name [«'.$btnKey.'»] not found.');
42 42
             }
43 43
             $newButtons->push($button);
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
         // we add the remaining buttons to the end of the ordered ones
48 48
         if (count($newButtons) < count($stackButtons)) {
49 49
             foreach ($stackButtons as $button) {
50
-                if (! $newButtons->where('name', $button->name)->first()) {
50
+                if (!$newButtons->where('name', $button->name)->first()) {
51 51
                     $newButtons->push($button);
52 52
                 }
53 53
             }
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
          */
117 117
         $button = $this->buttons()->firstWhere('name', $name);
118 118
 
119
-        if (! $button) {
119
+        if (!$button) {
120 120
             abort(500, 'CRUD Button "'.$name.'" not found. Please ensure the button exists before you modify it.');
121 121
         }
122 122
 
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
      */
138 138
     public function removeButton($name, $stack = null)
139 139
     {
140
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($name, $stack) {
140
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($name, $stack) {
141 141
             return $stack == null ? $button->name == $name : ($button->stack == $stack) && ($button->name == $name);
142 142
         }));
143 143
     }
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
      */
149 149
     public function removeButtons($names, $stack = null)
150 150
     {
151
-        if (! empty($names)) {
151
+        if (!empty($names)) {
152 152
             foreach ($names as $name) {
153 153
                 $this->removeButton($name, $stack);
154 154
             }
@@ -162,14 +162,14 @@  discard block
 block discarded – undo
162 162
 
163 163
     public function removeAllButtonsFromStack($stack)
164 164
     {
165
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($stack) {
165
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($stack) {
166 166
             return $button->stack == $stack;
167 167
         }));
168 168
     }
169 169
 
170 170
     public function removeButtonFromStack($name, $stack)
171 171
     {
172
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($name, $stack) {
172
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($name, $stack) {
173 173
             return $button->name == $name && $button->stack == $stack;
174 174
         }));
175 175
     }
@@ -190,15 +190,15 @@  discard block
 block discarded – undo
190 190
         $destinationKey = $this->getButtonKey($destination);
191 191
         $newDestinationKey = ($where == 'before' ? $destinationKey : $destinationKey + 1);
192 192
 
193
-        $newButtons = $this->buttons()->filter(function ($value, $key) use ($target) {
193
+        $newButtons = $this->buttons()->filter(function($value, $key) use ($target) {
194 194
             return $value->name != $target;
195 195
         });
196 196
 
197
-        if (! $targetButton) {
197
+        if (!$targetButton) {
198 198
             return;
199 199
         }
200 200
 
201
-        if (! $destinationButton) {
201
+        if (!$destinationButton) {
202 202
             return;
203 203
         }
204 204
 
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
 
208 208
         $newButtons = $firstSlice->push($targetButton);
209 209
 
210
-        $lastSlice->each(function ($item, $key) use ($newButtons) {
210
+        $lastSlice->each(function($item, $key) use ($newButtons) {
211 211
             $newButtons->push($item);
212 212
         });
213 213
 
@@ -266,7 +266,7 @@  discard block
 block discarded – undo
266 266
     /**
267 267
      * Add a new button to the current CRUD operation.
268 268
      */
269
-    public function button(string|array $nameOrAttributes): CrudButton
269
+    public function button(string | array $nameOrAttributes): CrudButton
270 270
     {
271 271
         return new CrudButton($nameOrAttributes);
272 272
     }
Please login to merge, or discard this patch.
src/app/Library/Validation/Rules/Support/HasFiles.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -9,7 +9,7 @@
 block discarded – undo
9 9
 {
10 10
     public array $fileRules;
11 11
 
12
-    public function file(string|array|ValidationRule|Rule $rules): self
12
+    public function file(string | array | ValidationRule | Rule $rules): self
13 13
     {
14 14
         $this->fileRules = self::getRulesAsArray($rules);
15 15
 
Please login to merge, or discard this patch.
src/app/View/Components/MenuDropdown.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
     /**
24 24
      * Get the view / contents that represent the component.
25 25
      */
26
-    public function render(): View|Closure|string
26
+    public function render(): View | Closure | string
27 27
     {
28 28
         return backpack_view('components.menu-dropdown');
29 29
     }
Please login to merge, or discard this patch.
src/app/View/Components/MenuItem.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
     /**
22 22
      * Get the view / contents that represent the component.
23 23
      */
24
-    public function render(): View|Closure|string
24
+    public function render(): View | Closure | string
25 25
     {
26 26
         return backpack_view('components.menu-item');
27 27
     }
Please login to merge, or discard this patch.