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 (#5478)
by Cristian
27:37 queued 12:39
created
tests/Feature/UploadersValidationTest.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 
63 63
         $this->assertDatabaseHas('uploaders', [
64 64
             'upload' => 'avatar1.jpg',
65
-            'upload_multiple' => json_encode(['avatar2.jpg',  'avatar3.jpg']),
65
+            'upload_multiple' => json_encode(['avatar2.jpg', 'avatar3.jpg']),
66 66
         ]);
67 67
 
68 68
         $this->assertTrue(Storage::disk('uploaders')->exists('avatar1.jpg'));
@@ -97,8 +97,8 @@  discard block
 block discarded – undo
97 97
 
98 98
         $response = $this->put($this->testBaseUrl.'/1', [
99 99
             'upload' => $this->getUploadedFile('avatar4.jpg'),
100
-            'upload_multiple' => $this->getUploadedFiles(['avatar5.jpg',  'avatar6.jpg']),
101
-            'clear_upload_multiple' => ['avatar2.jpg',  'avatar3.jpg'],
100
+            'upload_multiple' => $this->getUploadedFiles(['avatar5.jpg', 'avatar6.jpg']),
101
+            'clear_upload_multiple' => ['avatar2.jpg', 'avatar3.jpg'],
102 102
             'id' => 1,
103 103
         ]);
104 104
 
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
 
111 111
         $this->assertDatabaseHas('uploaders', [
112 112
             'upload' => 'avatar4.jpg',
113
-            'upload_multiple' => json_encode(['avatar5.jpg',  'avatar6.jpg']),
113
+            'upload_multiple' => json_encode(['avatar5.jpg', 'avatar6.jpg']),
114 114
         ]);
115 115
 
116 116
         $files = Storage::disk('uploaders')->allFiles();
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
         self::initUploaderWithFiles();
143 143
 
144 144
         $response = $this->put($this->testBaseUrl.'/1', [
145
-            'upload_multiple' => ['avatar2.jpg',  'avatar3.jpg'],
145
+            'upload_multiple' => ['avatar2.jpg', 'avatar3.jpg'],
146 146
             'id' => 1,
147 147
         ]);
148 148
 
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
154 154
 
155 155
         $this->assertDatabaseHas('uploaders', [
156 156
             'upload' => 'avatar1.jpg',
157
-            'upload_multiple' => json_encode(['avatar2.jpg',  'avatar3.jpg']),
157
+            'upload_multiple' => json_encode(['avatar2.jpg', 'avatar3.jpg']),
158 158
         ]);
159 159
 
160 160
         $files = Storage::disk('uploaders')->allFiles();
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
         self::initUploaderWithFiles();
172 172
 
173 173
         $response = $this->put($this->testBaseUrl.'/1', [
174
-            'upload_multiple' => $this->getUploadedFiles(['avatar4.jpg',  'avatar5.jpg']),
174
+            'upload_multiple' => $this->getUploadedFiles(['avatar4.jpg', 'avatar5.jpg']),
175 175
             'clear_upload_multiple' => ['avatar2.jpg'],
176 176
             'id' => 1,
177 177
         ]);
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
 
185 185
         $this->assertDatabaseHas('uploaders', [
186 186
             'upload' => 'avatar1.jpg',
187
-            'upload_multiple' => json_encode(['avatar3.jpg', 'avatar4.jpg',  'avatar5.jpg']),
187
+            'upload_multiple' => json_encode(['avatar3.jpg', 'avatar4.jpg', 'avatar5.jpg']),
188 188
         ]);
189 189
 
190 190
         $files = Storage::disk('uploaders')->allFiles();
@@ -341,7 +341,7 @@  discard block
 block discarded – undo
341 341
         $response = $this->put($this->testBaseUrl.'/1', [
342 342
             'upload' => null,
343 343
             'upload_multiple' => null,
344
-            'clear_upload_multiple' => ['avatar2.jpg',  'avatar3.jpg'],
344
+            'clear_upload_multiple' => ['avatar2.jpg', 'avatar3.jpg'],
345 345
             'id' => 1,
346 346
         ]);
347 347
 
@@ -364,7 +364,7 @@  discard block
 block discarded – undo
364 364
 
365 365
         $response = $this->put($this->testBaseUrl.'/1', [
366 366
             'upload_multiple' => $this->getUploadedFiles(['avatar2.jpg']),
367
-            'clear_upload_multiple' => ['avatar2.jpg',  'avatar3.jpg'],
367
+            'clear_upload_multiple' => ['avatar2.jpg', 'avatar3.jpg'],
368 368
             'id' => 1,
369 369
         ]);
370 370
 
@@ -386,7 +386,7 @@  discard block
 block discarded – undo
386 386
 
387 387
         Uploader::create([
388 388
             'upload' => 'avatar1.jpg',
389
-            'upload_multiple' => json_encode(['avatar2.jpg',  'avatar3.jpg']),
389
+            'upload_multiple' => json_encode(['avatar2.jpg', 'avatar3.jpg']),
390 390
         ]);
391 391
     }
392 392
 
@@ -398,7 +398,7 @@  discard block
 block discarded – undo
398 398
 
399 399
         Uploader::create([
400 400
             'upload' => 'avatar1.jpg',
401
-            'upload_multiple' => json_encode(['avatar2.jpg',  'avatar3.jpg']),
401
+            'upload_multiple' => json_encode(['avatar2.jpg', 'avatar3.jpg']),
402 402
         ]);
403 403
     }
404 404
 
Please login to merge, or discard this patch.
tests/config/Uploads/HasUploadedFiles.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
 
14 14
     protected function getUploadedFiles(array $fileNames, string $mime = 'image/jpg')
15 15
     {
16
-        return array_map(function ($fileName) use ($mime) {
16
+        return array_map(function($fileName) use ($mime) {
17 17
             return new UploadedFile(__DIR__.'/assets/'.$fileName, $fileName, $mime, null, true);
18 18
         }, $fileNames);
19 19
     }
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;
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
         }
83 83
 
84 84
         $subfields = collect($this->crudObject->getAttributes()['subfields']);
85
-        $subfields = $subfields->map(function ($item) use ($subfield, $uploader) {
85
+        $subfields = $subfields->map(function($item) use ($subfield, $uploader) {
86 86
             if ($item['name'] === $subfield['name']) {
87 87
                 $item['upload'] = true;
88 88
                 $item['disk'] = $uploader->getDisk();
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
         }
113 113
 
114 114
         if ($this->crudObjectType === 'field') {
115
-            $model::saving(function ($entry) use ($uploader) {
115
+            $model::saving(function($entry) use ($uploader) {
116 116
                 $entry = $uploader->storeUploadedFiles($entry);
117 117
             });
118 118
         }
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
             // is not called in pivot models when loading the relations.
126 126
             $retrieveModel = $this->getModelForRetrieveEvent($model, $uploader);
127 127
 
128
-            $retrieveModel::retrieved(function ($entry) use ($uploader) {
128
+            $retrieveModel::retrieved(function($entry) use ($uploader) {
129 129
                 if ($entry->translationEnabled()) {
130 130
                     $locale = request('_locale', app()->getLocale());
131 131
                     if (in_array($locale, array_keys($entry->getAvailableLocales()))) {
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
             });
137 137
         }
138 138
 
139
-        $model::deleting(function ($entry) use ($uploader) {
139
+        $model::deleting(function($entry) use ($uploader) {
140 140
             $uploader->deleteUploadedFiles($entry);
141 141
         });
142 142
 
@@ -157,18 +157,18 @@  discard block
 block discarded – undo
157 157
     {
158 158
         $hasCustomUploader = isset($uploaderConfiguration['uploader']);
159 159
 
160
-        if ($hasCustomUploader && ! is_a($uploaderConfiguration['uploader'], UploaderInterface::class, true)) {
160
+        if ($hasCustomUploader && !is_a($uploaderConfiguration['uploader'], UploaderInterface::class, true)) {
161 161
             throw new Exception('Invalid uploader class provided for '.$this->crudObjectType.' type: '.$crudObject['type']);
162 162
         }
163 163
 
164 164
         if ($hasCustomUploader) {
165
-            return $uploaderConfiguration['uploader']::for($crudObject, $uploaderConfiguration);
165
+            return $uploaderConfiguration['uploader']::for ($crudObject, $uploaderConfiguration);
166 166
         }
167 167
 
168 168
         $uploader = app('UploadersRepository')->hasUploadFor($crudObject['type'], $this->macro);
169 169
 
170 170
         if ($uploader) {
171
-            return app('UploadersRepository')->getUploadFor($crudObject['type'], $this->macro)::for($crudObject, $uploaderConfiguration);
171
+            return app('UploadersRepository')->getUploadFor($crudObject['type'], $this->macro)::for ($crudObject, $uploaderConfiguration);
172 172
         }
173 173
 
174 174
         throw new Exception('Undefined upload type for '.$this->crudObjectType.' type: '.$crudObject['type']);
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
 
190 190
     private function getSubfieldModel(array $subfield, UploaderInterface $uploader)
191 191
     {
192
-        if (! $uploader->isRelationship()) {
192
+        if (!$uploader->isRelationship()) {
193 193
             return $subfield['baseModel'] ?? get_class(app('crud')->getModel());
194 194
         }
195 195
 
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
 
203 203
     private function getModelForRetrieveEvent(string $model, UploaderInterface $uploader)
204 204
     {
205
-        if (! $uploader->isRelationship()) {
205
+        if (!$uploader->isRelationship()) {
206 206
             return $model;
207 207
         }
208 208
 
Please login to merge, or discard this patch.
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.
config/database/migrations/2024_02_15_125654_create_uploaders_table.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -11,7 +11,7 @@
 block discarded – undo
11 11
      */
12 12
     public function up(): void
13 13
     {
14
-        Schema::create('uploaders', function (Blueprint $table) {
14
+        Schema::create('uploaders', function(Blueprint $table) {
15 15
             $table->increments('id');
16 16
             $table->string('upload')->nullable();
17 17
             $table->string('image')->nullable();
Please login to merge, or discard this patch.
src/app/Library/Database/DatabaseSchema.php 1 patch
Spacing   +6 added lines, -7 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
     {
27 27
         $connection = $connection ?: config('database.default');
28 28
 
29
-        self::$schema[$connection] = LazyCollection::make(self::getCreateSchema($connection)->getTables())->mapWithKeys(function ($table, $key) use ($connection) {
29
+        self::$schema[$connection] = LazyCollection::make(self::getCreateSchema($connection)->getTables())->mapWithKeys(function($table, $key) use ($connection) {
30 30
             $tableName = is_array($table) ? $table['name'] : $table->getName();
31 31
 
32 32
             if ($existingTable = self::$schema[$connection][$tableName] ?? false) {
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
      */
66 66
     private static function generateDatabaseSchema(string $connection, string $table)
67 67
     {
68
-        if (! isset(self::$schema[$connection][$table])) {
68
+        if (!isset(self::$schema[$connection][$table])) {
69 69
             self::$schema[$connection][$table] = self::mapTable($connection, $table);
70 70
         }
71 71
     }
@@ -74,13 +74,12 @@  discard block
 block discarded – undo
74 74
     {
75 75
         try {
76 76
             $table = method_exists(self::getCreateSchema($connection), 'getTable') ?
77
-                        self::getCreateSchema($connection)->getTable($tableName) :
78
-                        self::getCreateSchema($connection)->getColumns($tableName);
77
+                        self::getCreateSchema($connection)->getTable($tableName) : self::getCreateSchema($connection)->getColumns($tableName);
79 78
         } catch (\Exception $e) {
80 79
             return new Table($tableName, []);
81 80
         }
82 81
 
83
-        if (! is_array($table)) {
82
+        if (!is_array($table)) {
84 83
             return $table;
85 84
         }
86 85
 
@@ -91,7 +90,7 @@  discard block
 block discarded – undo
91 90
         $schemaManager = self::getSchemaManager($connection);
92 91
         $indexes = $schemaManager->getIndexes($tableName);
93 92
 
94
-        $indexes = array_map(function ($index) {
93
+        $indexes = array_map(function($index) {
95 94
             return $index['columns'];
96 95
         }, $indexes);
97 96
 
@@ -109,7 +108,7 @@  discard block
 block discarded – undo
109 108
 
110 109
         $indexes = self::$schema[$connection][$table]->getIndexes();
111 110
 
112
-        $indexes = \Illuminate\Support\Arr::flatten(array_map(function ($index) {
111
+        $indexes = \Illuminate\Support\Arr::flatten(array_map(function($index) {
113 112
             return is_string($index) ? $index : $index->getColumns();
114 113
         }, $indexes));
115 114
 
Please login to merge, or discard this patch.
src/app/Library/Uploaders/Support/UploadersRepository.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
      */
35 35
     public function markAsHandled(string $objectName): void
36 36
     {
37
-        if (! in_array($objectName, $this->handledUploaders)) {
37
+        if (!in_array($objectName, $this->handledUploaders)) {
38 38
             $this->handledUploaders[] = $objectName;
39 39
         }
40 40
     }
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
      */
61 61
     public function getUploadFor(string $objectType, string $group): string
62 62
     {
63
-        if (! $this->hasUploadFor($objectType, $group)) {
63
+        if (!$this->hasUploadFor($objectType, $group)) {
64 64
             throw new \Exception('There is no uploader defined for the given field type.');
65 65
         }
66 66
 
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
     {
83 83
         // ensure all uploaders implement the UploaderInterface
84 84
         foreach ($uploaders as $uploader) {
85
-            if (! is_a($uploader, UploaderInterface::class, true)) {
85
+            if (!is_a($uploader, UploaderInterface::class, true)) {
86 86
                 throw new \Exception('The uploader class must implement the UploaderInterface.');
87 87
             }
88 88
         }
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
      */
103 103
     public function registerRepeatableUploader(string $uploadName, UploaderInterface $uploader): void
104 104
     {
105
-        if (! array_key_exists($uploadName, $this->repeatableUploaders) || ! in_array($uploader, $this->repeatableUploaders[$uploadName])) {
105
+        if (!array_key_exists($uploadName, $this->repeatableUploaders) || !in_array($uploader, $this->repeatableUploaders[$uploadName])) {
106 106
             $this->repeatableUploaders[$uploadName][] = $uploader;
107 107
         }
108 108
     }
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
      */
137 137
     public function getRegisteredUploadNames(string $uploadName): array
138 138
     {
139
-        return array_map(function ($uploader) {
139
+        return array_map(function($uploader) {
140 140
             return $uploader->getName();
141 141
         }, $this->getRepeatableUploadersFor($uploadName));
142 142
     }
@@ -167,11 +167,11 @@  discard block
 block discarded – undo
167 167
 
168 168
             $uploaders = $this->getRepeatableUploadersFor($repeatableContainerName);
169 169
 
170
-            $uploader = Arr::first($uploaders, function ($uploader) use ($requestInputName) {
170
+            $uploader = Arr::first($uploaders, function($uploader) use ($requestInputName) {
171 171
                 return $uploader->getName() === $requestInputName;
172 172
             });
173 173
 
174
-            if (! $uploader) {
174
+            if (!$uploader) {
175 175
                 abort(500, 'Could not find the field in the repeatable uploaders.');
176 176
             }
177 177
 
@@ -182,16 +182,16 @@  discard block
 block discarded – undo
182 182
             abort(500, 'Could not find the field in the CRUD fields.');
183 183
         }
184 184
 
185
-        if (! $uploaderMacro = $this->getUploadCrudObjectMacroType($crudObject)) {
185
+        if (!$uploaderMacro = $this->getUploadCrudObjectMacroType($crudObject)) {
186 186
             abort(500, 'There is no uploader defined for the given field type.');
187 187
         }
188 188
 
189
-        if (! $this->isValidUploadField($crudObject, $uploaderMacro)) {
189
+        if (!$this->isValidUploadField($crudObject, $uploaderMacro)) {
190 190
             abort(500, 'Invalid field for upload.');
191 191
         }
192 192
 
193 193
         $uploaderConfiguration = $crudObject[$uploaderMacro] ?? [];
194
-        $uploaderConfiguration = ! is_array($uploaderConfiguration) ? [] : $uploaderConfiguration;
194
+        $uploaderConfiguration = !is_array($uploaderConfiguration) ? [] : $uploaderConfiguration;
195 195
         $uploaderClass = $this->getUploadFor($crudObject['type'], $uploaderMacro);
196 196
 
197 197
         return new $uploaderClass(['name' => $requestInputName], $uploaderConfiguration);
@@ -200,7 +200,7 @@  discard block
 block discarded – undo
200 200
     /**
201 201
      * Get the upload field macro type for the given object.
202 202
      */
203
-    private function getUploadCrudObjectMacroType(array $crudObject): string|null
203
+    private function getUploadCrudObjectMacroType(array $crudObject): string | null
204 204
     {
205 205
         $uploadersGroups = $this->getUploadersGroupsNames();
206 206
 
@@ -217,11 +217,11 @@  discard block
 block discarded – undo
217 217
     {
218 218
         if (Str::contains($crudObject['name'], '#')) {
219 219
             $container = Str::before($crudObject['name'], '#');
220
-            $field = array_filter(CRUD::fields()[$container]['subfields'] ?? [], function ($item) use ($crudObject, $uploaderMacro) {
220
+            $field = array_filter(CRUD::fields()[$container]['subfields'] ?? [], function($item) use ($crudObject, $uploaderMacro) {
221 221
                 return $item['name'] === $crudObject['name'] && in_array($item['type'], $this->getAjaxUploadTypes($uploaderMacro));
222 222
             });
223 223
 
224
-            return ! empty($field);
224
+            return !empty($field);
225 225
         }
226 226
 
227 227
         return in_array($crudObject['type'], $this->getAjaxUploadTypes($uploaderMacro));
Please login to merge, or discard this patch.
src/app/Library/Uploaders/SingleFile.php 1 patch
Spacing   +3 added lines, -3 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,10 +55,10 @@  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
-                if (! in_array($file, $orderedFiles)) {
61
+                if (!in_array($file, $orderedFiles)) {
62 62
                     Storage::disk($this->getDisk())->delete($file);
63 63
                 }
64 64
             }
Please login to merge, or discard this patch.