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
Pull Request — main (#5243)
by Cristian
27:04 queued 12:27
created
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;
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
 
165 165
     public function getPreviousFiles(Model $entry): mixed
166 166
     {
167
-        if (! $this->attachedToFakeField) {
167
+        if (!$this->attachedToFakeField) {
168 168
             return $this->getOriginalValue($entry);
169 169
         }
170 170
 
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
203 203
         $value = $entry->{$this->name};
204 204
 
205 205
         if ($this->handleMultipleFiles) {
206
-            if (! isset($entry->getCasts()[$this->name]) && is_string($value)) {
206
+            if (!isset($entry->getCasts()[$this->name]) && is_string($value)) {
207 207
                 $entry->{$this->name} = json_decode($value, true);
208 208
             }
209 209
 
@@ -231,7 +231,7 @@  discard block
 block discarded – undo
231 231
 
232 232
         if ($this->handleMultipleFiles) {
233 233
             // ensure we have an array of values when field is not casted in model.
234
-            if (! isset($entry->getCasts()[$this->name]) && is_string($values)) {
234
+            if (!isset($entry->getCasts()[$this->name]) && is_string($values)) {
235 235
                 $values = json_decode($values, true);
236 236
             }
237 237
             foreach ($values as $value) {
@@ -247,7 +247,7 @@  discard block
 block discarded – undo
247 247
 
248 248
     private function performFileDeletion(Model $entry)
249 249
     {
250
-        if (! $this->handleRepeatableFiles) {
250
+        if (!$this->handleRepeatableFiles) {
251 251
             $this->deleteFiles($entry);
252 252
 
253 253
             return;
@@ -270,7 +270,7 @@  discard block
 block discarded – undo
270 270
     {
271 271
         $previousValue = $entry->getOriginal($field ?? $this->getName());
272 272
 
273
-        if (! $previousValue) {
273
+        if (!$previousValue) {
274 274
             return $previousValue;
275 275
         }
276 276
 
Please login to merge, or discard this patch.
src/app/Library/Uploaders/Support/RegisterUploadEvents.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -14,18 +14,18 @@  discard block
 block discarded – undo
14 14
     private string $crudObjectType;
15 15
 
16 16
     public function __construct(
17
-        private readonly CrudField|CrudColumn $crudObject,
17
+        private readonly CrudField | CrudColumn $crudObject,
18 18
         private readonly array $uploaderConfiguration,
19 19
         private readonly string $macro
20 20
         ) {
21 21
         $this->crudObjectType = is_a($crudObject, CrudField::class) ? 'field' : (is_a($crudObject, CrudColumn::class) ? 'column' : null);
22 22
 
23
-        if (! $this->crudObjectType) {
23
+        if (!$this->crudObjectType) {
24 24
             abort(500, 'Upload handlers only work for CrudField and CrudColumn classes.');
25 25
         }
26 26
     }
27 27
 
28
-    public static function handle(CrudField|CrudColumn $crudObject, array $uploaderConfiguration, string $macro, ?array $subfield = null, ?bool $registerModelEvents = true): void
28
+    public static function handle(CrudField | CrudColumn $crudObject, array $uploaderConfiguration, string $macro, ?array $subfield = null, ?bool $registerModelEvents = true): void
29 29
     {
30 30
         $instance = new self($crudObject, $uploaderConfiguration, $macro);
31 31
 
@@ -35,9 +35,9 @@  discard block
 block discarded – undo
35 35
     /*******************************
36 36
      * Private methods - implementation
37 37
      *******************************/
38
-    private function registerEvents(array|null $subfield = [], ?bool $registerModelEvents = true): void
38
+    private function registerEvents(array | null $subfield = [], ?bool $registerModelEvents = true): void
39 39
     {
40
-        if (! empty($subfield)) {
40
+        if (!empty($subfield)) {
41 41
             $this->registerSubfieldEvent($subfield, $registerModelEvents);
42 42
 
43 43
             return;
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
         }
78 78
 
79 79
         $subfields = collect($this->crudObject->getAttributes()['subfields']);
80
-        $subfields = $subfields->map(function ($item) use ($subfield, $uploader) {
80
+        $subfields = $subfields->map(function($item) use ($subfield, $uploader) {
81 81
             if ($item['name'] === $subfield['name']) {
82 82
                 $item['upload'] = true;
83 83
                 $item['disk'] = $uploader->getDisk();
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
         }
108 108
 
109 109
         if ($this->crudObjectType === 'field') {
110
-            $model::saving(function ($entry) use ($uploader) {
110
+            $model::saving(function($entry) use ($uploader) {
111 111
                 $updatedCountKey = 'uploaded_'.($uploader->getRepeatableContainerName() ?? $uploader->getName()).'_count';
112 112
 
113 113
                 CRUD::set($updatedCountKey, CRUD::get($updatedCountKey) ?? 0);
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
             // is not called in pivot models when eager loading the relations.
127 127
             $retrieveModel = $this->getRetrieveModel($model, $uploader);
128 128
 
129
-            $retrieveModel::retrieved(function ($entry) use ($uploader) {
129
+            $retrieveModel::retrieved(function($entry) use ($uploader) {
130 130
                 if ($entry->translationEnabled()) {
131 131
                     $locale = request('_locale', \App::getLocale());
132 132
                     if (in_array($locale, array_keys($entry->getAvailableLocales()))) {
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
             });
138 138
         }
139 139
 
140
-        $model::deleting(function ($entry) use ($uploader) {
140
+        $model::deleting(function($entry) use ($uploader) {
141 141
             $uploader->deleteUploadedFiles($entry);
142 142
         });
143 143
 
@@ -159,13 +159,13 @@  discard block
 block discarded – undo
159 159
         $customUploader = isset($uploaderConfiguration['uploader']) && class_exists($uploaderConfiguration['uploader']);
160 160
 
161 161
         if ($customUploader) {
162
-            return $uploaderConfiguration['uploader']::for($crudObject, $uploaderConfiguration);
162
+            return $uploaderConfiguration['uploader']::for ($crudObject, $uploaderConfiguration);
163 163
         }
164 164
 
165 165
         $uploader = app('UploadersRepository')->hasUploadFor($crudObject['type'], $this->macro);
166 166
 
167 167
         if ($uploader) {
168
-            return app('UploadersRepository')->getUploadFor($crudObject['type'], $this->macro)::for($crudObject, $uploaderConfiguration);
168
+            return app('UploadersRepository')->getUploadFor($crudObject['type'], $this->macro)::for ($crudObject, $uploaderConfiguration);
169 169
         }
170 170
 
171 171
         throw new Exception('Undefined upload type for '.$this->crudObjectType.' type: '.$crudObject['type']);
@@ -181,11 +181,11 @@  discard block
 block discarded – undo
181 181
 
182 182
     private function getSubfieldModel(array $subfield, UploaderInterface $uploader)
183 183
     {
184
-        if (! $uploader->isRelationship()) {
184
+        if (!$uploader->isRelationship()) {
185 185
             return $subfield['baseModel'] ?? get_class(app('crud')->getModel());
186 186
         }
187 187
 
188
-        switch($subfield['relation_type']) {
188
+        switch ($subfield['relation_type']) {
189 189
             case 'BelongsToMany':
190 190
                 return app('crud')->getModel()->{$subfield['baseEntity']}()->getPivotClass();
191 191
         }
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
 
194 194
     private function getRetrieveModel(string $model, UploaderInterface $uploader)
195 195
     {
196
-        if (! $uploader->isRelationship()) {
196
+        if (!$uploader->isRelationship()) {
197 197
             return $model;
198 198
         }
199 199
 
Please login to merge, or discard this patch.
src/app/Library/Uploaders/Support/Traits/HandleRepeatableUploads.php 1 patch
Spacing   +11 added lines, -11 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
             }
@@ -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
                 $pivotClass = app('crud')->getModel()->{$this->getUploaderSubfield()['baseEntity']}()->getPivotClass();
117 117
                 $pivotFieldName = 'pivot_'.$this->getName();
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
     private function deleteRepeatableFiles(Model $entry): void
148 148
     {
149 149
         if ($this->isRelationship) {
150
-            switch($this->getRepeatableRelationType()) {
150
+            switch ($this->getRepeatableRelationType()) {
151 151
                 case 'BelongsToMany':
152 152
                     // handle belongs to many deletion
153 153
                     return;
@@ -160,12 +160,12 @@  discard block
 block discarded – undo
160 160
 
161 161
         $repeatableValues = collect($entry->{$this->getName()});
162 162
         foreach (app('UploadersRepository')->getRepeatableUploadersFor($this->getRepeatableContainerName()) as $upload) {
163
-            if (! $upload->shouldDeleteFiles()) {
163
+            if (!$upload->shouldDeleteFiles()) {
164 164
                 continue;
165 165
             }
166 166
             $values = $repeatableValues->pluck($upload->getName())->toArray();
167 167
             foreach ($values as $value) {
168
-                if (! $value) {
168
+                if (!$value) {
169 169
                     continue;
170 170
                 }
171 171
 
@@ -188,7 +188,7 @@  discard block
 block discarded – undo
188 188
     /**
189 189
      * Given two multidimensional arrays/collections, merge them recursively.
190 190
      */
191
-    protected function mergeValuesRecursive(array|Collection $array1, array|Collection $array2): array|Collection
191
+    protected function mergeValuesRecursive(array | Collection $array1, array | Collection $array2): array | Collection
192 192
     {
193 193
         $merged = $array1;
194 194
         foreach ($array2 as $key => &$value) {
@@ -210,7 +210,7 @@  discard block
 block discarded – undo
210 210
     {
211 211
         $items = CRUD::getRequest()->input('_order_'.$this->getRepeatableContainerName()) ?? [];
212 212
 
213
-        array_walk($items, function (&$key, $value) {
213
+        array_walk($items, function(&$key, $value) {
214 214
             $requestValue = $key[$this->getName()] ?? null;
215 215
             $key = $this->handleMultipleFiles ? (is_string($requestValue) ? explode(',', $requestValue) : $requestValue) : $requestValue;
216 216
         });
@@ -222,18 +222,18 @@  discard block
 block discarded – undo
222 222
     {
223 223
         $previousValues = json_decode($entry->getOriginal($uploader->getRepeatableContainerName()), true);
224 224
 
225
-        if (! empty($previousValues)) {
225
+        if (!empty($previousValues)) {
226 226
             $previousValues = array_column($previousValues, $uploader->getName());
227 227
         }
228 228
 
229 229
         return $previousValues ?? [];
230 230
     }
231 231
 
232
-    private function getValuesWithPathStripped(array|string|null $item, UploaderInterface $upload)
232
+    private function getValuesWithPathStripped(array | string | null $item, UploaderInterface $upload)
233 233
     {
234 234
         $uploadedValues = $item[$upload->getName()] ?? null;
235 235
         if (is_array($uploadedValues)) {
236
-            return array_map(function ($value) use ($upload) {
236
+            return array_map(function($value) use ($upload) {
237 237
                 return Str::after($value, $upload->getPath());
238 238
             }, $uploadedValues);
239 239
         }
Please login to merge, or discard this patch.