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 — eager-load-when-edition ( 7ddd83 )
by Pedro
10:43
created
src/app/Library/Uploaders/Support/Traits/HandleRepeatableUploads.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -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->getName())->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->getName()] = $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->getName()] = $this->getValuesWithPathStripped($item, $upload);
102 102
             }
@@ -119,12 +119,12 @@  discard block
 block discarded – undo
119 119
 
120 120
         $repeatableValues = collect($entry->{$this->getName()});
121 121
         foreach (app('UploadersRepository')->getRepeatableUploadersFor($this->getRepeatableContainerName()) as $upload) {
122
-            if (! $upload->shouldDeleteFiles()) {
122
+            if (!$upload->shouldDeleteFiles()) {
123 123
                 continue;
124 124
             }
125 125
             $values = $repeatableValues->pluck($upload->getName())->toArray();
126 126
             foreach ($values as $value) {
127
-                if (! $value) {
127
+                if (!$value) {
128 128
                     continue;
129 129
                 }
130 130
 
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
     /**
148 148
      * Given two multidimensional arrays/collections, merge them recursively.
149 149
      */
150
-    protected function mergeValuesRecursive(array|Collection $array1, array|Collection $array2): array|Collection
150
+    protected function mergeValuesRecursive(array | Collection $array1, array | Collection $array2): array | Collection
151 151
     {
152 152
         $merged = $array1;
153 153
         foreach ($array2 as $key => &$value) {
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
     {
170 170
         $items = CRUD::getRequest()->input('_order_'.$this->getRepeatableContainerName()) ?? [];
171 171
 
172
-        array_walk($items, function (&$key, $value) {
172
+        array_walk($items, function(&$key, $value) {
173 173
             $requestValue = $key[$this->getName()] ?? null;
174 174
             $key = $this->handleMultipleFiles ? (is_string($requestValue) ? explode(',', $requestValue) : $requestValue) : $requestValue;
175 175
         });
@@ -181,18 +181,18 @@  discard block
 block discarded – undo
181 181
     {
182 182
         $previousValues = json_decode($entry->getOriginal($uploader->getRepeatableContainerName()), true);
183 183
 
184
-        if (! empty($previousValues)) {
184
+        if (!empty($previousValues)) {
185 185
             $previousValues = array_column($previousValues, $uploader->getName());
186 186
         }
187 187
 
188 188
         return $previousValues ?? [];
189 189
     }
190 190
 
191
-    private function getValuesWithPathStripped(array|string|null $item, UploaderInterface $upload)
191
+    private function getValuesWithPathStripped(array | string | null $item, UploaderInterface $upload)
192 192
     {
193 193
         $uploadedValues = $item[$upload->getName()] ?? null;
194 194
         if (is_array($uploadedValues)) {
195
-            return array_map(function ($value) use ($upload) {
195
+            return array_map(function($value) use ($upload) {
196 196
                 return Str::after($value, $upload->getPath());
197 197
             }, $uploadedValues);
198 198
         }
Please login to merge, or discard this patch.
src/app/Library/Uploaders/Uploader.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
 
26 26
     private bool $deleteWhenEntryIsDeleted = true;
27 27
 
28
-    private bool|string $attachedToFakeField = false;
28
+    private bool | string $attachedToFakeField = false;
29 29
 
30 30
     /**
31 31
      * Cloud disks have the ability to generate temporary URLs to files, should we do it?
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
     /*******************************
58 58
      * Static methods
59 59
      *******************************/
60
-    public static function for(array $crudObject, array $definition): UploaderInterface
60
+    public static function for (array $crudObject, array $definition): UploaderInterface
61 61
     {
62 62
         return new static($crudObject, $definition);
63 63
     }
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
     public function deleteUploadedFiles(Model $entry): void
99 99
     {
100 100
         if ($this->deleteWhenEntryIsDeleted) {
101
-            if (! in_array(SoftDeletes::class, class_uses_recursive($entry), true)) {
101
+            if (!in_array(SoftDeletes::class, class_uses_recursive($entry), true)) {
102 102
                 $this->performFileDeletion($entry);
103 103
 
104 104
                 return;
@@ -159,7 +159,7 @@  discard block
 block discarded – undo
159 159
 
160 160
     public function getPreviousFiles(Model $entry): mixed
161 161
     {
162
-        if (! $this->attachedToFakeField) {
162
+        if (!$this->attachedToFakeField) {
163 163
             return $this->getOriginalValue($entry);
164 164
         }
165 165
 
@@ -198,7 +198,7 @@  discard block
 block discarded – undo
198 198
         $value = $entry->{$this->name};
199 199
 
200 200
         if ($this->handleMultipleFiles) {
201
-            if (! isset($entry->getCasts()[$this->name]) && is_string($value)) {
201
+            if (!isset($entry->getCasts()[$this->name]) && is_string($value)) {
202 202
                 $entry->{$this->name} = json_decode($value, true);
203 203
             }
204 204
 
@@ -226,7 +226,7 @@  discard block
 block discarded – undo
226 226
 
227 227
         if ($this->handleMultipleFiles) {
228 228
             // ensure we have an array of values when field is not casted in model.
229
-            if (! isset($entry->getCasts()[$this->name]) && is_string($values)) {
229
+            if (!isset($entry->getCasts()[$this->name]) && is_string($values)) {
230 230
                 $values = json_decode($values, true);
231 231
             }
232 232
             foreach ($values as $value) {
@@ -242,7 +242,7 @@  discard block
 block discarded – undo
242 242
 
243 243
     private function performFileDeletion(Model $entry)
244 244
     {
245
-        if ($this->isRelationship || ! $this->handleRepeatableFiles) {
245
+        if ($this->isRelationship || !$this->handleRepeatableFiles) {
246 246
             $this->deleteFiles($entry);
247 247
 
248 248
             return;
@@ -265,7 +265,7 @@  discard block
 block discarded – undo
265 265
     {
266 266
         $previousValue = $entry->getOriginal($field ?? $this->getName());
267 267
 
268
-        if (! $previousValue) {
268
+        if (!$previousValue) {
269 269
             return $previousValue;
270 270
         }
271 271
 
Please login to merge, or discard this patch.
src/app/Library/Uploaders/Support/RegisterUploadEvents.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -13,18 +13,18 @@  discard block
 block discarded – undo
13 13
     private string $crudObjectType;
14 14
 
15 15
     public function __construct(
16
-        private readonly CrudField|CrudColumn $crudObject,
16
+        private readonly CrudField | CrudColumn $crudObject,
17 17
         private readonly array $uploaderConfiguration,
18 18
         private readonly string $macro
19 19
         ) {
20 20
         $this->crudObjectType = is_a($crudObject, CrudField::class) ? 'field' : (is_a($crudObject, CrudColumn::class) ? 'column' : null);
21 21
 
22
-        if (! $this->crudObjectType) {
22
+        if (!$this->crudObjectType) {
23 23
             abort(500, 'Upload handlers only work for CrudField and CrudColumn classes.');
24 24
         }
25 25
     }
26 26
 
27
-    public static function handle(CrudField|CrudColumn $crudObject, array $uploaderConfiguration, string $macro, ?array $subfield = null, ?bool $registerModelEvents = true): void
27
+    public static function handle(CrudField | CrudColumn $crudObject, array $uploaderConfiguration, string $macro, ?array $subfield = null, ?bool $registerModelEvents = true): void
28 28
     {
29 29
         $instance = new self($crudObject, $uploaderConfiguration, $macro);
30 30
 
@@ -34,9 +34,9 @@  discard block
 block discarded – undo
34 34
     /*******************************
35 35
      * Private methods - implementation
36 36
      *******************************/
37
-    private function registerEvents(array|null $subfield = [], ?bool $registerModelEvents = true): void
37
+    private function registerEvents(array | null $subfield = [], ?bool $registerModelEvents = true): void
38 38
     {
39
-        if (! empty($subfield)) {
39
+        if (!empty($subfield)) {
40 40
             $this->registerSubfieldEvent($subfield, $registerModelEvents);
41 41
 
42 42
             return;
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
         }
76 76
 
77 77
         $subfields = collect($this->crudObject->getAttributes()['subfields']);
78
-        $subfields = $subfields->map(function ($item) use ($subfield, $uploader) {
78
+        $subfields = $subfields->map(function($item) use ($subfield, $uploader) {
79 79
             if ($item['name'] === $subfield['name']) {
80 80
                 $item['upload'] = true;
81 81
                 $item['disk'] = $uploader->getDisk();
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
         }
106 106
 
107 107
         if ($this->crudObjectType === 'field') {
108
-            $model::saving(function ($entry) use ($uploader) {
108
+            $model::saving(function($entry) use ($uploader) {
109 109
                 $updatedCountKey = 'uploaded_'.($uploader->getRepeatableContainerName() ?? $uploader->getName()).'_count';
110 110
 
111 111
                 CRUD::set($updatedCountKey, CRUD::get($updatedCountKey) ?? 0);
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
         if (app('crud')->entry) {
121 121
             app('crud')->entry = $uploader->retrieveUploadedFiles(app('crud')->entry);
122 122
         } else {
123
-            $model::retrieved(function ($entry) use ($uploader) {
123
+            $model::retrieved(function($entry) use ($uploader) {
124 124
                 if ($entry->translationEnabled()) {
125 125
                     $locale = request('_locale', \App::getLocale());
126 126
                     if (in_array($locale, array_keys($entry->getAvailableLocales()))) {
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
             });
132 132
         }
133 133
 
134
-        $model::deleting(function ($entry) use ($uploader) {
134
+        $model::deleting(function($entry) use ($uploader) {
135 135
             $uploader->deleteUploadedFiles($entry);
136 136
         });
137 137
 
@@ -153,13 +153,13 @@  discard block
 block discarded – undo
153 153
         $customUploader = isset($uploaderConfiguration['uploader']) && class_exists($uploaderConfiguration['uploader']);
154 154
 
155 155
         if ($customUploader) {
156
-            return $uploaderConfiguration['uploader']::for($crudObject, $uploaderConfiguration);
156
+            return $uploaderConfiguration['uploader']::for ($crudObject, $uploaderConfiguration);
157 157
         }
158 158
 
159 159
         $uploader = app('UploadersRepository')->hasUploadFor($crudObject['type'], $this->macro);
160 160
 
161 161
         if ($uploader) {
162
-            return app('UploadersRepository')->getUploadFor($crudObject['type'], $this->macro)::for($crudObject, $uploaderConfiguration);
162
+            return app('UploadersRepository')->getUploadFor($crudObject['type'], $this->macro)::for ($crudObject, $uploaderConfiguration);
163 163
         }
164 164
 
165 165
         throw new Exception('Undefined upload type for '.$this->crudObjectType.' type: '.$crudObject['type']);
Please login to merge, or discard this patch.
src/app/Console/Commands/Install.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
 
84 84
         // Install Backpack Generators
85 85
         $this->progressBlock('Installing Generators');
86
-        if (! file_exists('vendor/backpack/generators/composer.json')) {
86
+        if (!file_exists('vendor/backpack/generators/composer.json')) {
87 87
             // only do this if Generators aren't already required
88 88
             $process = new Process(['composer', 'require', '--dev', 'backpack/generators']);
89 89
             $process->setTimeout(300);
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
         $this->closeProgressBlock();
98 98
 
99 99
         // Optional commands
100
-        if (! $this->option('no-interaction')) {
100
+        if (!$this->option('no-interaction')) {
101 101
             // Themes
102 102
             $this->installTheme();
103 103
 
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
 
107 107
             // Addons
108 108
             $this->installAddons();
109
-        } elseif (! $this->isAnyThemeInstalled()) {
109
+        } elseif (!$this->isAnyThemeInstalled()) {
110 110
             // Install default theme
111 111
             $this->progressBlock('Installing default theme');
112 112
             $this->executeArtisanProcess('backpack:require:theme-tabler');
@@ -187,7 +187,7 @@  discard block
 block discarded – undo
187 187
 
188 188
         $this->deleteLines(3);
189 189
 
190
-        if (! $total) {
190
+        if (!$total) {
191 191
             $this->deleteLines();
192 192
             $this->note('Skipping creating an admin user.');
193 193
             $this->newLine();
@@ -196,14 +196,14 @@  discard block
 block discarded – undo
196 196
 
197 197
     private function isEveryAddonInstalled()
198 198
     {
199
-        return collect($this->addons)->every(function ($addon) {
199
+        return collect($this->addons)->every(function($addon) {
200 200
             return file_exists($addon->path);
201 201
         });
202 202
     }
203 203
 
204 204
     private function updateAddonsStatus()
205 205
     {
206
-        $this->addons = $this->addons->each(function (&$addon) {
206
+        $this->addons = $this->addons->each(function(&$addon) {
207 207
             $isInstalled = file_exists($addon->path);
208 208
             $addon->status = $isInstalled ? 'installed' : 'not installed';
209 209
             $addon->statusColor = $isInstalled ? 'green' : 'yellow';
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
     {
215 215
         // map the addons
216 216
         $this->addons = collect($this->addons)
217
-            ->map(function ($class) {
217
+            ->map(function($class) {
218 218
                 return (object) $class::$addon;
219 219
             });
220 220
 
@@ -233,15 +233,15 @@  discard block
 block discarded – undo
233 233
 
234 234
         // Calculate the printed line count
235 235
         $printedLines = $this->addons
236
-            ->map(function ($e) {
236
+            ->map(function($e) {
237 237
                 return count($e->description);
238 238
             })
239
-            ->reduce(function ($sum, $item) {
239
+            ->reduce(function($sum, $item) {
240 240
                 return $sum + $item + 2;
241 241
             }, 0);
242 242
 
243 243
         $total = 0;
244
-        while (! $this->isEveryAddonInstalled()) {
244
+        while (!$this->isEveryAddonInstalled()) {
245 245
             $input = (int) $this->listChoice('Would you like to install a premium Backpack add-on? <fg=gray>(enter option number: 1, 2 or 3)</>', $this->addons->toArray());
246 246
 
247 247
             if ($input < 1 || $input > $this->addons->count()) {
@@ -276,14 +276,14 @@  discard block
 block discarded – undo
276 276
 
277 277
     private function isEveryThemeInstalled()
278 278
     {
279
-        return $this->themes()->every(function ($theme) {
279
+        return $this->themes()->every(function($theme) {
280 280
             return $theme->status == 'installed';
281 281
         });
282 282
     }
283 283
 
284 284
     private function isAnyThemeInstalled()
285 285
     {
286
-        return $this->themes()->filter(function ($theme) {
286
+        return $this->themes()->filter(function($theme) {
287 287
             return $theme->status == 'installed';
288 288
         })->count() > 0;
289 289
     }
@@ -301,10 +301,10 @@  discard block
 block discarded – undo
301 301
 
302 302
         // Calculate the printed line count
303 303
         $printedLines = $this->themes()
304
-            ->map(function ($e) {
304
+            ->map(function($e) {
305 305
                 return count($e->description);
306 306
             })
307
-            ->reduce(function ($sum, $item) {
307
+            ->reduce(function($sum, $item) {
308 308
                 return $sum + $item + 2;
309 309
             }, 0);
310 310
 
@@ -340,9 +340,9 @@  discard block
 block discarded – undo
340 340
     public function themes()
341 341
     {
342 342
         return collect($this->themes)
343
-            ->map(function ($class) {
343
+            ->map(function($class) {
344 344
                 return (object) $class::$addon;
345
-            })->each(function (&$theme) {
345
+            })->each(function(&$theme) {
346 346
                 $isInstalled = file_exists($theme->path);
347 347
                 $theme->status = $isInstalled ? 'installed' : 'not installed';
348 348
                 $theme->statusColor = $isInstalled ? 'green' : 'yellow';
Please login to merge, or discard this patch.
src/BackpackServiceProvider.php 1 patch
Spacing   +13 added lines, -15 removed lines patch added patch discarded remove patch
@@ -81,24 +81,24 @@  discard block
 block discarded – undo
81 81
         $this->registerBackpackErrorViews();
82 82
 
83 83
         // Bind the CrudPanel object to Laravel's service container
84
-        $this->app->scoped('crud', function ($app) {
84
+        $this->app->scoped('crud', function($app) {
85 85
             return new CrudPanel();
86 86
         });
87 87
 
88
-        $this->app->scoped('DatabaseSchema', function ($app) {
88
+        $this->app->scoped('DatabaseSchema', function($app) {
89 89
             return new DatabaseSchema();
90 90
         });
91 91
 
92
-        $this->app->singleton('BackpackViewNamespaces', function ($app) {
92
+        $this->app->singleton('BackpackViewNamespaces', function($app) {
93 93
             return new ViewNamespaces();
94 94
         });
95 95
 
96 96
         // Bind the widgets collection object to Laravel's service container
97
-        $this->app->singleton('widgets', function ($app) {
97
+        $this->app->singleton('widgets', function($app) {
98 98
             return new Collection();
99 99
         });
100 100
 
101
-        $this->app->scoped('UploadersRepository', function ($app) {
101
+        $this->app->scoped('UploadersRepository', function($app) {
102 102
             return new UploadersRepository();
103 103
         });
104 104
 
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
         $middleware_key = config('backpack.base.middleware_key');
115 115
         $middleware_class = config('backpack.base.middleware_class');
116 116
 
117
-        if (! is_array($middleware_class)) {
117
+        if (!is_array($middleware_class)) {
118 118
             $router->pushMiddlewareToGroup($middleware_key, $middleware_class);
119 119
 
120 120
             return;
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
         $configs = scandir(__DIR__."/config/backpack/$dir/");
228 228
         $configs = array_diff($configs, ['.', '..']);
229 229
 
230
-        if (! count($configs)) {
230
+        if (!count($configs)) {
231 231
             return;
232 232
         }
233 233
 
@@ -276,8 +276,7 @@  discard block
 block discarded – undo
276 276
         // add the backpack_users password broker to the configuration
277 277
         $laravelAuthPasswordBrokers = app()->config['auth.passwords'];
278 278
         $laravelFirstPasswordBroker = is_array($laravelAuthPasswordBrokers) && current($laravelAuthPasswordBrokers) ?
279
-                                        current($laravelAuthPasswordBrokers)['table'] :
280
-                                        '';
279
+                                        current($laravelAuthPasswordBrokers)['table'] : '';
281 280
 
282 281
         $backpackPasswordBrokerTable = config('backpack.base.password_resets_table') ??
283 282
                                         config('auth.passwords.users.table') ??
@@ -305,7 +304,7 @@  discard block
 block discarded – undo
305 304
 
306 305
     public function loadViewComponents()
307 306
     {
308
-        $this->app->afterResolving(BladeCompiler::class, function () {
307
+        $this->app->afterResolving(BladeCompiler::class, function() {
309 308
             Blade::componentNamespace('Backpack\\CRUD\\app\\View\\Components', 'backpack');
310 309
         });
311 310
     }
@@ -331,13 +330,13 @@  discard block
 block discarded – undo
331 330
     private function registerBackpackErrorViews()
332 331
     {
333 332
         // register the backpack error when the exception handler is resolved from the container
334
-        $this->callAfterResolving(ExceptionHandler::class, function ($handler) {
335
-            if (! Str::startsWith(request()->path(), config('backpack.base.route_prefix'))) {
333
+        $this->callAfterResolving(ExceptionHandler::class, function($handler) {
334
+            if (!Str::startsWith(request()->path(), config('backpack.base.route_prefix'))) {
336 335
                 return;
337 336
             }
338 337
 
339 338
             // parse the namespaces set in config
340
-            [$themeNamespace, $themeFallbackNamespace] = (function () {
339
+            [$themeNamespace, $themeFallbackNamespace] = (function() {
341 340
                 $themeNamespace = config('backpack.ui.view_namespace');
342 341
                 $themeFallbackNamespace = config('backpack.ui.view_namespace_fallback');
343 342
 
@@ -354,8 +353,7 @@  discard block
 block discarded – undo
354 353
             // - fallback theme paths
355 354
             // - ui path
356 355
             $themeErrorPaths = $viewFinderHints[$themeNamespace] ?? [];
357
-            $themeErrorPaths = $themeNamespace === $themeFallbackNamespace ? $themeErrorPaths :
358
-                array_merge($viewFinderHints[$themeFallbackNamespace] ?? [], $themeErrorPaths);
356
+            $themeErrorPaths = $themeNamespace === $themeFallbackNamespace ? $themeErrorPaths : array_merge($viewFinderHints[$themeFallbackNamespace] ?? [], $themeErrorPaths);
359 357
             $uiErrorPaths = [base_path('vendor/backpack/crud/src/resources/views/ui')];
360 358
             $themeErrorPaths = array_merge($themeErrorPaths, $uiErrorPaths);
361 359
 
Please login to merge, or discard this patch.