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-uploaders-tests ( a74a83...698d73 )
by Pedro
14:37
created
src/app/Library/Uploaders/Support/Traits/HandleRepeatableUploads.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 {
18 18
     public bool $handleRepeatableFiles = false;
19 19
 
20
-    public null|string $repeatableContainerName = null;
20
+    public null | string $repeatableContainerName = null;
21 21
 
22 22
     /*******************************
23 23
      * Setters - fluently configure the uploader
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
     /*******************************
35 35
      * Getters
36 36
      *******************************/
37
-    public function getRepeatableContainerName(): null|string
37
+    public function getRepeatableContainerName(): null | string
38 38
     {
39 39
         return $this->repeatableContainerName;
40 40
     }
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
         foreach (app('UploadersRepository')->getRepeatableUploadersFor($this->getRepeatableContainerName()) as $uploader) {
163 163
             $uploadedValues = $uploader->uploadRepeatableFiles($values->pluck($uploader->getAttributeName())->toArray(), $this->getPreviousRepeatableValues($entry, $uploader));
164 164
 
165
-            $values = $values->map(function ($item, $key) use ($uploadedValues, $uploader) {
165
+            $values = $values->map(function($item, $key) use ($uploadedValues, $uploader) {
166 166
                 $item[$uploader->getAttributeName()] = $uploadedValues[$key] ?? null;
167 167
 
168 168
                 return $item;
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
 
194 194
         $values = $entry->{$this->getRepeatableContainerName()};
195 195
         $values = is_string($values) ? json_decode($values, true) : $values;
196
-        $values = array_map(function ($item) use ($repeatableUploaders) {
196
+        $values = array_map(function($item) use ($repeatableUploaders) {
197 197
             foreach ($repeatableUploaders as $upload) {
198 198
                 $item[$upload->getAttributeName()] = $this->getValuesWithPathStripped($item, $upload);
199 199
             }
@@ -208,7 +208,7 @@  discard block
 block discarded – undo
208 208
 
209 209
     private function retrieveRepeatableRelationFiles(Model $entry)
210 210
     {
211
-        switch($this->getRepeatableRelationType()) {
211
+        switch ($this->getRepeatableRelationType()) {
212 212
             case 'BelongsToMany':
213 213
             case 'MorphToMany':
214 214
                 $pivotClass = app('crud')->getModel()->{$this->getUploaderSubfield()['baseEntity']}()->getPivotClass();
@@ -261,12 +261,12 @@  discard block
 block discarded – undo
261 261
         $repeatableValues ??= collect($entry->{$this->getRepeatableContainerName()});
262 262
 
263 263
         foreach (app('UploadersRepository')->getRepeatableUploadersFor($this->getRepeatableContainerName()) as $upload) {
264
-            if (! $upload->shouldDeleteFiles()) {
264
+            if (!$upload->shouldDeleteFiles()) {
265 265
                 continue;
266 266
             }
267 267
             $values = $repeatableValues->pluck($upload->getName())->toArray();
268 268
             foreach ($values as $value) {
269
-                if (! $value) {
269
+                if (!$value) {
270 270
                     continue;
271 271
                 }
272 272
 
@@ -289,7 +289,7 @@  discard block
 block discarded – undo
289 289
     /**
290 290
      * Given two multidimensional arrays/collections, merge them recursively.
291 291
      */
292
-    protected function mergeValuesRecursive(array|Collection $array1, array|Collection $array2): array|Collection
292
+    protected function mergeValuesRecursive(array | Collection $array1, array | Collection $array2): array | Collection
293 293
     {
294 294
         $merged = $array1;
295 295
         foreach ($array2 as $key => &$value) {
@@ -311,7 +311,7 @@  discard block
 block discarded – undo
311 311
     {
312 312
         $items = CRUD::getRequest()->input('_order_'.$this->getRepeatableContainerName()) ?? [];
313 313
 
314
-        array_walk($items, function (&$key, $value) {
314
+        array_walk($items, function(&$key, $value) {
315 315
             $requestValue = $key[$this->getName()] ?? null;
316 316
             $key = $this->handleMultipleFiles ? (is_string($requestValue) ? explode(',', $requestValue) : $requestValue) : $requestValue;
317 317
         });
@@ -323,22 +323,22 @@  discard block
 block discarded – undo
323 323
     {
324 324
         $previousValues = $entry->getOriginal($uploader->getRepeatableContainerName());
325 325
 
326
-        if (! is_array($previousValues)) {
326
+        if (!is_array($previousValues)) {
327 327
             $previousValues = json_decode($previousValues, true);
328 328
         }
329 329
 
330
-        if (! empty($previousValues)) {
330
+        if (!empty($previousValues)) {
331 331
             $previousValues = array_column($previousValues, $uploader->getName());
332 332
         }
333 333
 
334 334
         return $previousValues ?? [];
335 335
     }
336 336
 
337
-    private function getValuesWithPathStripped(array|string|null $item, UploaderInterface $uploader)
337
+    private function getValuesWithPathStripped(array | string | null $item, UploaderInterface $uploader)
338 338
     {
339 339
         $uploadedValues = $item[$uploader->getName()] ?? null;
340 340
         if (is_array($uploadedValues)) {
341
-            return array_map(function ($value) use ($uploader) {
341
+            return array_map(function($value) use ($uploader) {
342 342
                 return $uploader->getValueWithoutPath($value);
343 343
             }, $uploadedValues);
344 344
         }
@@ -357,19 +357,19 @@  discard block
 block discarded – undo
357 357
     {
358 358
         if (in_array($this->getRepeatableRelationType(), ['BelongsToMany', 'MorphToMany'])) {
359 359
             $pivotAttributes = $entry->getAttributes();
360
-            $connectedPivot = $entry->pivotParent->{$this->getRepeatableContainerName()}->where(function ($item) use ($pivotAttributes) {
360
+            $connectedPivot = $entry->pivotParent->{$this->getRepeatableContainerName()}->where(function($item) use ($pivotAttributes) {
361 361
                 $itemPivotAttributes = $item->pivot->only(array_keys($pivotAttributes));
362 362
 
363 363
                 return $itemPivotAttributes === $pivotAttributes;
364 364
             })->first();
365 365
 
366
-            if (! $connectedPivot) {
366
+            if (!$connectedPivot) {
367 367
                 return;
368 368
             }
369 369
 
370 370
             $files = $connectedPivot->getOriginal()['pivot_'.$this->getAttributeName()];
371 371
 
372
-            if (! $files) {
372
+            if (!$files) {
373 373
                 return;
374 374
             }
375 375
 
Please login to merge, or discard this patch.
src/app/Library/Uploaders/SingleFile.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
             return $this->getPath().$fileName;
30 30
         }
31 31
 
32
-        if (! $value && CrudPanelFacade::getRequest()->has($this->getNameForRequest()) && $previousFile) {
32
+        if (!$value && CrudPanelFacade::getRequest()->has($this->getNameForRequest()) && $previousFile) {
33 33
             Storage::disk($this->getDisk())->delete($previousFile);
34 34
 
35 35
             return null;
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 
56 56
         foreach ($previousRepeatableValues as $row => $file) {
57 57
             if ($file) {
58
-                if(! isset($orderedFiles[$row])) {
58
+                if (!isset($orderedFiles[$row])) {
59 59
                     $orderedFiles[$row] = null;
60 60
                 }
61 61
                 
Please login to merge, or discard this patch.