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-multiple-upload ( 3aceb0 )
by Pedro
29:41 queued 15:00
created
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.
src/app/Library/Uploaders/SingleFile.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
             return $this->getPath().$fileName;
24 24
         }
25 25
 
26
-        if (! $value && CrudPanelFacade::getRequest()->has($this->getNameForRequest()) && $previousFile) {
26
+        if (!$value && CrudPanelFacade::getRequest()->has($this->getNameForRequest()) && $previousFile) {
27 27
             Storage::disk($this->getDisk())->delete($previousFile);
28 28
 
29 29
             return null;
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
         }
48 48
 
49 49
         foreach ($previousRepeatableValues as $row => $file) {
50
-            if ($file && ! isset($orderedFiles[$row])) {
50
+            if ($file && !isset($orderedFiles[$row])) {
51 51
                 $orderedFiles[$row] = null;
52 52
                 Storage::disk($this->getDisk())->delete($file);
53 53
             }
Please login to merge, or discard this patch.
src/app/Library/Uploaders/MultipleFiles.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@  discard block
 block discarded – undo
10 10
 
11 11
 class MultipleFiles extends Uploader
12 12
 {
13
-    public static function for(array $field, $configuration): UploaderInterface
13
+    public static function for (array $field, $configuration): UploaderInterface
14 14
     {
15 15
         return (new self($field, $configuration))->multiple();
16 16
     }
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
         $value = $value ?? collect(CRUD::getRequest()->file($this->getNameForRequest()))->flatten()->toArray();
26 26
         $previousFiles = $this->getPreviousFiles($entry) ?? [];
27 27
 
28
-        if (! is_array($previousFiles) && is_string($previousFiles)) {
28
+        if (!is_array($previousFiles) && is_string($previousFiles)) {
29 29
             $previousFiles = json_decode($previousFiles, true);
30 30
         }
31 31
 
@@ -34,14 +34,14 @@  discard block
 block discarded – undo
34 34
                 if (in_array($previousFile, $filesToDelete)) {
35 35
                     Storage::disk($this->getDisk())->delete($previousFile);
36 36
 
37
-                    $previousFiles = Arr::where($previousFiles, function ($value, $key) use ($previousFile) {
37
+                    $previousFiles = Arr::where($previousFiles, function($value, $key) use ($previousFile) {
38 38
                         return $value != $previousFile;
39 39
                     });
40 40
                 }
41 41
             }
42 42
         }
43 43
 
44
-        if (! is_array($value)) {
44
+        if (!is_array($value)) {
45 45
             $value = [];
46 46
         }
47 47
 
Please login to merge, or discard this patch.