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 — crud-uploads ( 700846...0628f1 )
by Pedro
12:28
created
src/BackpackServiceProvider.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -66,24 +66,24 @@  discard block
 block discarded – undo
66 66
         include_once __DIR__.'/macros.php';
67 67
 
68 68
         // Bind the CrudPanel object to Laravel's service container
69
-        $this->app->scoped('crud', function ($app) {
69
+        $this->app->scoped('crud', function($app) {
70 70
             return new CrudPanel();
71 71
         });
72 72
 
73
-        $this->app->scoped('DatabaseSchema', function ($app) {
73
+        $this->app->scoped('DatabaseSchema', function($app) {
74 74
             return new DatabaseSchema();
75 75
         });
76 76
 
77
-        $this->app->singleton('BackpackViewNamespaces', function ($app) {
77
+        $this->app->singleton('BackpackViewNamespaces', function($app) {
78 78
             return new ViewNamespaces();
79 79
         });
80 80
 
81 81
         // Bind the widgets collection object to Laravel's service container
82
-        $this->app->singleton('widgets', function ($app) {
82
+        $this->app->singleton('widgets', function($app) {
83 83
             return new Collection();
84 84
         });
85 85
 
86
-        $this->app->scoped('UploadersRepository', function ($app) {
86
+        $this->app->scoped('UploadersRepository', function($app) {
87 87
             return new UploadersRepository();
88 88
         });
89 89
 
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
         $middleware_key = config('backpack.base.middleware_key');
100 100
         $middleware_class = config('backpack.base.middleware_class');
101 101
 
102
-        if (! is_array($middleware_class)) {
102
+        if (!is_array($middleware_class)) {
103 103
             $router->pushMiddlewareToGroup($middleware_key, $middleware_class);
104 104
 
105 105
             return;
@@ -215,7 +215,7 @@  discard block
 block discarded – undo
215 215
         $operationConfigs = scandir(__DIR__.'/config/backpack/operations/');
216 216
         $operationConfigs = array_diff($operationConfigs, ['.', '..']);
217 217
 
218
-        if (! count($operationConfigs)) {
218
+        if (!count($operationConfigs)) {
219 219
             return;
220 220
         }
221 221
 
Please login to merge, or discard this patch.
src/macros.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -19,8 +19,8 @@  discard block
 block discarded – undo
19 19
  *      - when true: `address[street]`
20 20
  *      - when false: `[address][street]`
21 21
  */
22
-if (! Str::hasMacro('dotsToSquareBrackets')) {
23
-    Str::macro('dotsToSquareBrackets', function ($string, $ignore = [], $keyFirst = true) {
22
+if (!Str::hasMacro('dotsToSquareBrackets')) {
23
+    Str::macro('dotsToSquareBrackets', function($string, $ignore = [], $keyFirst = true) {
24 24
         $stringParts = explode('.', $string);
25 25
         $result = '';
26 26
 
@@ -34,8 +34,8 @@  discard block
 block discarded – undo
34 34
         return $result;
35 35
     });
36 36
 }
37
-if (! CrudColumn::hasMacro('withFiles')) {
38
-    CrudColumn::macro('withFiles', function ($uploadDefinition = []) {
37
+if (!CrudColumn::hasMacro('withFiles')) {
38
+    CrudColumn::macro('withFiles', function($uploadDefinition = []) {
39 39
         /** @var CrudField|CrudColumn $this */
40 40
         $uploadDefinition['uploadersGroup'] = 'withFiles';
41 41
         RegisterUploadEvents::handle($this, $uploadDefinition);
@@ -44,8 +44,8 @@  discard block
 block discarded – undo
44 44
     });
45 45
 }
46 46
 
47
-if (! CrudField::hasMacro('withFiles')) {
48
-    CrudField::macro('withFiles', function ($uploadDefinition = []) {
47
+if (!CrudField::hasMacro('withFiles')) {
48
+    CrudField::macro('withFiles', function($uploadDefinition = []) {
49 49
         /** @var CrudField|CrudColumn $this */
50 50
         $uploadDefinition['uploadersGroup'] = 'withFiles';
51 51
         RegisterUploadEvents::handle($this, $uploadDefinition);
@@ -60,8 +60,8 @@  discard block
 block discarded – undo
60 60
  *
61 61
  * It will go to the given CrudController and get the setupRoutes() method on it.
62 62
  */
63
-if (! Route::hasMacro('crud')) {
64
-    Route::macro('crud', function ($name, $controller) {
63
+if (!Route::hasMacro('crud')) {
64
+    Route::macro('crud', function($name, $controller) {
65 65
         // put together the route name prefix,
66 66
         // as passed to the Route::group() statements
67 67
         $routeName = '';
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Uploads/RegisterUploadEvents.php 2 patches
Spacing   +13 added lines, -14 removed lines patch added patch discarded remove patch
@@ -11,7 +11,7 @@  discard block
 block discarded – undo
11 11
 
12 12
 final class RegisterUploadEvents
13 13
 {
14
-    public function __construct(private readonly CrudField|CrudColumn $crudObject, private readonly array $uploadDefinition)
14
+    public function __construct(private readonly CrudField | CrudColumn $crudObject, private readonly array $uploadDefinition)
15 15
     {
16 16
     }
17 17
 
@@ -37,13 +37,13 @@  discard block
 block discarded – undo
37 37
 
38 38
         $crudObjectType = is_a($crudObject, CrudField::class) ? 'field' : (is_a($crudObject, CrudColumn::class) ? 'column' : null);
39 39
 
40
-        if (! $crudObjectType) {
40
+        if (!$crudObjectType) {
41 41
             abort(500, 'Upload handlers only work for CrudField and CrudColumn classes.');
42 42
         }
43 43
 
44 44
         $attributes['crudObjectType'] = $crudObjectType;
45 45
 
46
-        if (! isset($attributes['subfields'])) {
46
+        if (!isset($attributes['subfields'])) {
47 47
             $uploaderType = $instance->getUploader($attributes, $uploadDefinition);
48 48
             $instance->setupModelEvents($attributes['entryClass'], $uploaderType);
49 49
             self::setupUploadConfigsInCrudObject($crudObject, $uploaderType);
@@ -62,14 +62,14 @@  discard block
 block discarded – undo
62 62
      * @param  UploaderInterface|RepeatableUploaderInterface  $uploader
63 63
      * @return void
64 64
      */
65
-    private function setupModelEvents(string $model, UploaderInterface|RepeatableUploaderInterface $uploader): void
65
+    private function setupModelEvents(string $model, UploaderInterface | RepeatableUploaderInterface $uploader): void
66 66
     {
67 67
         if (app('UploadersRepository')->isUploadHandled($uploader->getName())) {
68 68
             return;
69 69
         }
70 70
 
71 71
         if ($uploader->getCrudObjectType() === 'field') {
72
-            $model::saving(function ($entry) use ($uploader) {
72
+            $model::saving(function($entry) use ($uploader) {
73 73
                 $updatedCountKey = 'uploaded_'.$uploader->getName().'_count';
74 74
 
75 75
                 CRUD::set($updatedCountKey, CRUD::get($updatedCountKey) ?? 0);
@@ -80,18 +80,18 @@  discard block
 block discarded – undo
80 80
             });
81 81
         }
82 82
 
83
-        $model::retrieved(function ($entry) use ($uploader) {
83
+        $model::retrieved(function($entry) use ($uploader) {
84 84
             $entry = $uploader->retrieveUploadedFile($entry);
85 85
         });
86 86
 
87
-        $model::deleting(function ($entry) use ($uploader) {
87
+        $model::deleting(function($entry) use ($uploader) {
88 88
             if (is_a($uploader, RepeatableUploaderInterface::class)) {
89 89
                 $uploader->deleteUploadedFile($entry);
90 90
 
91 91
                 return;
92 92
             }
93 93
             if ($uploader->deleteWhenEntryIsDeleted) {
94
-                if (! in_array(SoftDeletes::class, class_uses_recursive($entry), true)) {
94
+                if (!in_array(SoftDeletes::class, class_uses_recursive($entry), true)) {
95 95
                     $uploader->deleteUploadedFile($entry);
96 96
                 } else {
97 97
                     if ($entry->forceDeleting === true) {
@@ -125,8 +125,7 @@  discard block
 block discarded – undo
125 125
                 $subfielduploadDefinition = $subfield['withUploads'] ?? $subfield['withMedia'];
126 126
 
127 127
                 $subfielduploadDefinition = is_array($subfielduploadDefinition) ?
128
-                                                array_merge($uploadDefinition, $subfielduploadDefinition) :
129
-                                                $uploadDefinition;
128
+                                                array_merge($uploadDefinition, $subfielduploadDefinition) : $uploadDefinition;
130 129
 
131 130
                 $uploaderType = $this->getUploader($subfield, $subfielduploadDefinition);
132 131
 
@@ -135,7 +134,7 @@  discard block
 block discarded – undo
135 134
         }
136 135
 
137 136
         foreach ($repeatableDefinitions as $model => $uploaderTypes) {
138
-            $repeatableDefinition = app('UploadersRepository')->getUploadFor('repeatable', $uploadDefinition['uploadersGroup'] ?? null)::for($crudObject)->uploads(...$uploaderTypes);
137
+            $repeatableDefinition = app('UploadersRepository')->getUploadFor('repeatable', $uploadDefinition['uploadersGroup'] ?? null)::for ($crudObject)->uploads(...$uploaderTypes);
139 138
             $this->setupModelEvents($model, $repeatableDefinition);
140 139
         }
141 140
     }
@@ -157,13 +156,13 @@  discard block
 block discarded – undo
157 156
     private function getUploader(array $crudObject, array $uploadDefinition)
158 157
     {
159 158
         if (isset($uploadDefinition['uploaderType'])) {
160
-            return $uploadDefinition['uploaderType']::for($crudObject, $uploadDefinition);
159
+            return $uploadDefinition['uploaderType']::for ($crudObject, $uploadDefinition);
161 160
         }
162 161
 
163 162
         $uploaders = $uploadDefinition['uploadersGroup'] ?? null;
164 163
 
165 164
         if (app('UploadStore')->hasUploadFor($crudObject['type'], $uploaders)) {
166
-            return app('UploadStore')->getUploadFor($crudObject['type'], $uploaders)::for($crudObject, $uploadDefinition);
165
+            return app('UploadStore')->getUploadFor($crudObject['type'], $uploaders)::for ($crudObject, $uploadDefinition);
167 166
         }
168 167
 
169 168
         throw new Exception('Undefined upload type for '.$crudObject['crudObjectType'].' type: '.$crudObject['type']);
@@ -176,7 +175,7 @@  discard block
 block discarded – undo
176 175
      * @param  UploaderInterface  $uploader
177 176
      * @return void
178 177
      */
179
-    private static function setupUploadConfigsInCrudObject(CrudField|CrudColumn $crudObject, UploaderInterface $uploader)
178
+    private static function setupUploadConfigsInCrudObject(CrudField | CrudColumn $crudObject, UploaderInterface $uploader)
180 179
     {
181 180
         $crudObject->upload(true)->disk($uploader->disk)->prefix($uploader->path);
182 181
     }
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -135,7 +135,9 @@
 block discarded – undo
135 135
         }
136 136
 
137 137
         foreach ($repeatableDefinitions as $model => $uploaderTypes) {
138
-            $repeatableDefinition = app('UploadersRepository')->getUploadFor('repeatable', $uploadDefinition['uploadersGroup'] ?? null)::for($crudObject)->uploads(...$uploaderTypes);
138
+            $repeatableDefinition = app('UploadersRepository')->getUploadFor('repeatable', $uploadDefinition['uploadersGroup'] ?? null)::for($crudObject) {
139
+                ->uploads(...$uploaderTypes);
140
+            }
139 141
             $this->setupModelEvents($model, $repeatableDefinition);
140 142
         }
141 143
     }
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Uploads/Uploaders/Uploader.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
         $this->name = $crudObject['name'];
102 102
         $this->disk = $configuration['disk'] ?? $crudObject['disk'] ?? $this->disk;
103 103
         $this->temporary = $configuration['temporary'] ?? $this->temporary;
104
-        $this->temporaryUrlExpirationTime = $configuration['expiration'] ?? $this->temporaryUrlExpirationTime ;
104
+        $this->temporaryUrlExpirationTime = $configuration['expiration'] ?? $this->temporaryUrlExpirationTime;
105 105
         $this->entryClass = $crudObject['entryClass'];
106 106
         $this->path = $configuration['path'] ?? $crudObject['prefix'] ?? $this->path;
107 107
         $this->path = empty($this->path) ? $this->path : Str::of($this->path)->finish('/')->value;
@@ -192,7 +192,7 @@  discard block
 block discarded – undo
192 192
     {
193 193
         $value = $entry->{$this->name};
194 194
 
195
-        if ($this->isMultiple && ! isset($entry->getCasts()[$this->name]) && is_string($value)) {
195
+        if ($this->isMultiple && !isset($entry->getCasts()[$this->name]) && is_string($value)) {
196 196
             $entry->{$this->name} = json_decode($value, true);
197 197
         } else {
198 198
             $entry->{$this->name} = Str::after($value, $this->path);
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
         $values = $entry->{$this->name};
213 213
 
214 214
         if ($this->isMultiple) {
215
-            if (! isset($entry->getCasts()[$this->name]) && is_string($values)) {
215
+            if (!isset($entry->getCasts()[$this->name]) && is_string($values)) {
216 216
                 $values = json_decode($values, true);
217 217
             }
218 218
         } else {
@@ -231,7 +231,7 @@  discard block
 block discarded – undo
231 231
      * @param  array  $definition
232 232
      * @return self
233 233
      */
234
-    public static function for(array $crudObject, array $definition)
234
+    public static function for (array $crudObject, array $definition)
235 235
     {
236 236
         return new static($crudObject, $definition);
237 237
     }
@@ -294,7 +294,7 @@  discard block
 block discarded – undo
294 294
     {
295 295
         $items = CRUD::getRequest()->input('_order_'.$this->repeatableContainerName) ?? [];
296 296
 
297
-        array_walk($items, function (&$key, $value) {
297
+        array_walk($items, function(&$key, $value) {
298 298
             $requestValue = $key[$this->name] ?? null;
299 299
             $key = $this->isMultiple ? (is_string($requestValue) ? explode(',', $requestValue) : $requestValue) : $requestValue;
300 300
         });
@@ -321,7 +321,7 @@  discard block
 block discarded – undo
321 321
     protected function getPreviousRepeatableValues(Model $entry)
322 322
     {
323 323
         $previousValues = json_decode($entry->getOriginal($this->repeatableContainerName), true);
324
-        if (! empty($previousValues)) {
324
+        if (!empty($previousValues)) {
325 325
             $previousValues = array_column($previousValues, $this->name);
326 326
         }
327 327
 
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Uploads/Uploaders/MultipleFiles.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -9,14 +9,14 @@  discard block
 block discarded – undo
9 9
 
10 10
 class MultipleFiles extends Uploader
11 11
 {
12
-    public static function for(array $field, $configuration)
12
+    public static function for (array $field, $configuration)
13 13
     {
14 14
         return (new self($field, $configuration))->multiple();
15 15
     }
16 16
 
17 17
     public function save(Model $entry, $value = null)
18 18
     {
19
-        return $this->isRepeatable && ! $this->isRelationship ? $this->saveRepeatableMultipleFiles($entry, $value) : $this->saveMultipleFiles($entry, $value);
19
+        return $this->isRepeatable && !$this->isRelationship ? $this->saveRepeatableMultipleFiles($entry, $value) : $this->saveMultipleFiles($entry, $value);
20 20
     }
21 21
 
22 22
     private function saveMultipleFiles($entry, $value = null)
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
 
28 28
         $previousFiles = $entry->getOriginal($this->name) ?? [];
29 29
 
30
-        if (! is_array($previousFiles) && is_string($previousFiles)) {
30
+        if (!is_array($previousFiles) && is_string($previousFiles)) {
31 31
             $previousFiles = json_decode($previousFiles, true);
32 32
         }
33 33
 
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
                 if (in_array($previousFile, $filesToDelete)) {
37 37
                     Storage::disk($this->disk)->delete($previousFile);
38 38
 
39
-                    $previousFiles = Arr::where($previousFiles, function ($value, $key) use ($previousFile) {
39
+                    $previousFiles = Arr::where($previousFiles, function($value, $key) use ($previousFile) {
40 40
                         return $value != $previousFile;
41 41
                     });
42 42
                 }
Please login to merge, or discard this patch.