Passed
Push — master ( 1e0412...ff8110 )
by Quentin
17:21 queued 10:45
created
src/Repositories/Behaviors/HandleBlocks.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -71,11 +71,11 @@  discard block
 block discarded – undo
71 71
 
72 72
         $blockRepository->bulkDelete($object->blocks()->pluck('id')->toArray());
73 73
 
74
-        $this->getBlocks($object, $fields)->each(function ($block) use ($object, $blockRepository) {
74
+        $this->getBlocks($object, $fields)->each(function($block) use ($object, $blockRepository) {
75 75
 
76 76
             $blockCreated = $blockRepository->create($block);
77 77
 
78
-            $block['blocks']->each(function ($childBlock) use ($blockCreated, $blockRepository) {
78
+            $block['blocks']->each(function($childBlock) use ($blockCreated, $blockRepository) {
79 79
                 $childBlock['parent_id'] = $blockCreated->id;
80 80
                 $blockRepository->create($childBlock);
81 81
             });
@@ -174,9 +174,9 @@  discard block
 block discarded – undo
174 174
                     ];
175 175
                 }
176 176
 
177
-                $fields['blocksFields'][] = Collection::make($block['content'])->filter(function ($value, $key) {
177
+                $fields['blocksFields'][] = Collection::make($block['content'])->filter(function($value, $key) {
178 178
                     return $key !== "browsers";
179
-                })->map(function ($value, $key) use ($block) {
179
+                })->map(function($value, $key) use ($block) {
180 180
                     return [
181 181
                         'name' => "blocks[$block->id][$key]",
182 182
                         'value' => $value,
@@ -189,15 +189,15 @@  discard block
 block discarded – undo
189 189
 
190 190
                 if ($medias) {
191 191
                     if (config('twill.media_library.translated_form_fields', false)) {
192
-                        $fields['blocksMedias'][] = Collection::make($medias)->mapWithKeys(function ($mediasByLocale, $locale) use ($block) {
193
-                            return Collection::make($mediasByLocale)->mapWithKeys(function ($value, $key) use ($block, $locale) {
192
+                        $fields['blocksMedias'][] = Collection::make($medias)->mapWithKeys(function($mediasByLocale, $locale) use ($block) {
193
+                            return Collection::make($mediasByLocale)->mapWithKeys(function($value, $key) use ($block, $locale) {
194 194
                                 return [
195 195
                                     "blocks[$block->id][$key][$locale]" => $value,
196 196
                                 ];
197 197
                             });
198 198
                         })->filter()->toArray();
199 199
                     } else {
200
-                        $fields['blocksMedias'][] = Collection::make($medias)->mapWithKeys(function ($value, $key) use ($block) {
200
+                        $fields['blocksMedias'][] = Collection::make($medias)->mapWithKeys(function($value, $key) use ($block) {
201 201
                             return [
202 202
                                 "blocks[$block->id][$key]" => $value,
203 203
                             ];
@@ -208,8 +208,8 @@  discard block
 block discarded – undo
208 208
                 $files = $blockFormFields['files'];
209 209
 
210 210
                 if ($files) {
211
-                    Collection::make($files)->each(function ($rolesWithFiles, $locale) use (&$fields, $block) {
212
-                        $fields['blocksFiles'][] = Collection::make($rolesWithFiles)->mapWithKeys(function ($files, $role) use ($locale, $block) {
211
+                    Collection::make($files)->each(function($rolesWithFiles, $locale) use (&$fields, $block) {
212
+                        $fields['blocksFiles'][] = Collection::make($rolesWithFiles)->mapWithKeys(function($files, $role) use ($locale, $block) {
213 213
                             return [
214 214
                                 "blocks[$block->id][$role][$locale]" => $files,
215 215
                             ];
@@ -248,9 +248,9 @@  discard block
 block discarded – undo
248 248
      */
249 249
     protected function getBlockBrowsers($block)
250 250
     {
251
-        return Collection::make($block['content']['browsers'])->mapWithKeys(function ($ids, $relation) use ($block) {
251
+        return Collection::make($block['content']['browsers'])->mapWithKeys(function($ids, $relation) use ($block) {
252 252
             if (Schema::hasTable(config('twill.related_table', 'related')) && $block->getRelated($relation)->isNotEmpty()) {
253
-                $items = $this->getFormFieldsForRelatedBrowser($block, $relation);;
253
+                $items = $this->getFormFieldsForRelatedBrowser($block, $relation); ;
254 254
             } else {
255 255
                 $relationRepository = $this->getModelRepository($relation);
256 256
                 $relatedItems = $relationRepository->get([], ['id' => $ids], [], -1);
@@ -260,9 +260,9 @@  discard block
 block discarded – undo
260 260
                     $sortedRelatedItems[$item->id] = $item;
261 261
                 }
262 262
 
263
-                $items = Collection::make(array_values($sortedRelatedItems))->filter(function ($value) {
263
+                $items = Collection::make(array_values($sortedRelatedItems))->filter(function($value) {
264 264
                     return is_object($value);
265
-                })->map(function ($relatedElement) use ($relation) {
265
+                })->map(function($relatedElement) use ($relation) {
266 266
                     return [
267 267
                         'id' => $relatedElement->id,
268 268
                         'name' => $relatedElement->titleInBrowser ?? $relatedElement->title,
Please login to merge, or discard this patch.
src/Commands/ModuleMake.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -102,7 +102,7 @@
 block discarded – undo
102 102
             'hasFiles',
103 103
             'hasPosition',
104 104
             'hasRevisions',
105
-        ])->filter(function ($enabled) {
105
+        ])->filter(function($enabled) {
106 106
             return $enabled;
107 107
         });
108 108
 
Please login to merge, or discard this patch.
src/Repositories/Behaviors/HandleBrowsers.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
      */
104 104
     public function getFormFieldsForBrowser($object, $relation, $routePrefix = null, $titleKey = 'title', $moduleName = null)
105 105
     {
106
-        return $object->$relation->map(function ($relatedElement) use ($titleKey, $routePrefix, $relation, $moduleName) {
106
+        return $object->$relation->map(function($relatedElement) use ($titleKey, $routePrefix, $relation, $moduleName) {
107 107
             return [
108 108
                 'id' => $relatedElement->id,
109 109
                 'name' => $relatedElement->titleInBrowser ?? $relatedElement->$titleKey,
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
      */
123 123
     public function getFormFieldsForRelatedBrowser($object, $relation)
124 124
     {
125
-        return $object->getRelated($relation)->map(function ($relatedElement) {
125
+        return $object->getRelated($relation)->map(function($relatedElement) {
126 126
             return ($relatedElement != null) ? [
127 127
                 'id' => $relatedElement->id,
128 128
                 'name' => $relatedElement->titleInBrowser ?? $relatedElement->title,
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
             ]) + (classHasTrait($relatedElement, HasMedias::class) ? [
133 133
                 'thumbnail' => $relatedElement->defaultCmsImage(['w' => 100, 'h' => 100]),
134 134
             ] : []) : [];
135
-        })->reject(function ($item) {
135
+        })->reject(function($item) {
136 136
             return empty($item);
137 137
         })->values()->toArray();
138 138
     }
Please login to merge, or discard this patch.
src/Repositories/BlockRepository.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -45,8 +45,8 @@  discard block
 block discarded – undo
45 45
         if (Schema::hasTable(config('twill.related_table', 'related'))) {
46 46
             $relatedItems = Collection::make();
47 47
 
48
-            Collection::make($fields['browsers'])->each(function ($items, $browserName) use ($object, &$relatedItems) {
49
-                Collection::make($items)->each(function ($item) use ($browserName, &$relatedItems) {
48
+            Collection::make($fields['browsers'])->each(function($items, $browserName) use ($object, &$relatedItems) {
49
+                Collection::make($items)->each(function($item) use ($browserName, &$relatedItems) {
50 50
                     try {
51 51
                         $repository = $this->getModelRepository($item['endpointType'] ?? $browserName);
52 52
                         $relatedItems->push((object) [
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
     {
75 75
         if (Schema::hasTable(config('twill.related_table', 'related'))) {
76 76
             if (isset($fields['browsers'])) {
77
-                Collection::make($fields['browsers'])->each(function ($items, $browserName) use ($object) {
77
+                Collection::make($fields['browsers'])->each(function($items, $browserName) use ($object) {
78 78
                     $object->saveRelated($items, $browserName);
79 79
                 });
80 80
             }
@@ -102,14 +102,14 @@  discard block
 block discarded – undo
102 102
     {
103 103
         $blocksFromConfig = $this->config->get('twill.block_editor.' . ($repeater ? 'repeaters' : 'blocks'));
104 104
 
105
-        $block['type'] = Collection::make($blocksFromConfig)->search(function ($blockConfig) use ($block) {
105
+        $block['type'] = Collection::make($blocksFromConfig)->search(function($blockConfig) use ($block) {
106 106
             return $blockConfig['component'] === $block['type'];
107 107
         });
108 108
 
109 109
         $block['content'] = empty($block['content']) ? new \stdClass : (object) $block['content'];
110 110
 
111 111
         if ($block['browsers']) {
112
-            $browsers = Collection::make($block['browsers'])->map(function ($items) {
112
+            $browsers = Collection::make($block['browsers'])->map(function($items) {
113 113
                 return Collection::make($items)->pluck('id');
114 114
             })->toArray();
115 115
 
Please login to merge, or discard this patch.
src/Repositories/ModuleRepository.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
 
184 184
         foreach ($fields as $field) {
185 185
             if (in_array($field, $translatedAttributes)) {
186
-                $query->orWhereHas('translations', function ($q) use ($field, $search) {
186
+                $query->orWhereHas('translations', function($q) use ($field, $search) {
187 187
                     $q->where($field, $this->getLikeOperator(), "%{$search}%");
188 188
                 });
189 189
             } else {
@@ -210,7 +210,7 @@  discard block
 block discarded – undo
210 210
      */
211 211
     public function create($fields)
212 212
     {
213
-        return DB::transaction(function () use ($fields) {
213
+        return DB::transaction(function() use ($fields) {
214 214
             $original_fields = $fields;
215 215
 
216 216
             $fields = $this->prepareFieldsBeforeCreate($fields);
@@ -265,7 +265,7 @@  discard block
 block discarded – undo
265 265
      */
266 266
     public function update($id, $fields)
267 267
     {
268
-        DB::transaction(function () use ($id, $fields) {
268
+        DB::transaction(function() use ($id, $fields) {
269 269
             $object = $this->model->findOrFail($id);
270 270
 
271 271
             $this->beforeSave($object, $fields);
@@ -288,7 +288,7 @@  discard block
 block discarded – undo
288 288
      */
289 289
     public function updateBasic($id, $values, $scopes = [])
290 290
     {
291
-        return DB::transaction(function () use ($id, $values, $scopes) {
291
+        return DB::transaction(function() use ($id, $values, $scopes) {
292 292
             // apply scopes if no id provided
293 293
             if (is_null($id)) {
294 294
                 $query = $this->model->query();
@@ -299,7 +299,7 @@  discard block
 block discarded – undo
299 299
 
300 300
                 $query->update($values);
301 301
 
302
-                $query->get()->each(function ($object) use ($values) {
302
+                $query->get()->each(function($object) use ($values) {
303 303
                     $this->afterUpdateBasic($object, $values);
304 304
                 });
305 305
 
@@ -311,7 +311,7 @@  discard block
 block discarded – undo
311 311
                 $query = $this->model->whereIn('id', $id);
312 312
                 $query->update($values);
313 313
 
314
-                $query->get()->each(function ($object) use ($values) {
314
+                $query->get()->each(function($object) use ($values) {
315 315
                     $this->afterUpdateBasic($object, $values);
316 316
                 });
317 317
 
@@ -334,7 +334,7 @@  discard block
 block discarded – undo
334 334
      */
335 335
     public function setNewOrder($ids)
336 336
     {
337
-        DB::transaction(function () use ($ids) {
337
+        DB::transaction(function() use ($ids) {
338 338
             $this->model->setNewOrder($ids);
339 339
         }, 3);
340 340
     }
@@ -345,7 +345,7 @@  discard block
 block discarded – undo
345 345
      */
346 346
     public function delete($id)
347 347
     {
348
-        return DB::transaction(function () use ($id) {
348
+        return DB::transaction(function() use ($id) {
349 349
             if (($object = $this->model->find($id)) === null) {
350 350
                 return false;
351 351
             }
@@ -365,9 +365,9 @@  discard block
 block discarded – undo
365 365
      */
366 366
     public function bulkDelete($ids)
367 367
     {
368
-        return DB::transaction(function () use ($ids) {
368
+        return DB::transaction(function() use ($ids) {
369 369
             try {
370
-                Collection::make($ids)->each(function ($id) {
370
+                Collection::make($ids)->each(function($id) {
371 371
                     $this->delete($id);
372 372
                 });
373 373
             } catch (\Exception $e) {
@@ -385,7 +385,7 @@  discard block
 block discarded – undo
385 385
      */
386 386
     public function restore($id)
387 387
     {
388
-        return DB::transaction(function () use ($id) {
388
+        return DB::transaction(function() use ($id) {
389 389
             if (($object = $this->model->withTrashed()->find($id)) != null) {
390 390
                 $object->restore();
391 391
                 $this->afterRestore($object);
@@ -402,14 +402,14 @@  discard block
 block discarded – undo
402 402
      */
403 403
     public function bulkRestore($ids)
404 404
     {
405
-        return DB::transaction(function () use ($ids) {
405
+        return DB::transaction(function() use ($ids) {
406 406
             try {
407 407
                 $query = $this->model->withTrashed()->whereIn('id', $ids);
408 408
                 $objects = $query->get();
409 409
 
410 410
                 $query->restore();
411 411
 
412
-                $objects->each(function ($object) {
412
+                $objects->each(function($object) {
413 413
                     $this->afterRestore($object);
414 414
                 });
415 415
             } catch (\Exception $e) {
@@ -684,7 +684,7 @@  discard block
 block discarded – undo
684 684
     {
685 685
         if (isset($scopes[$scopeField])) {
686 686
             $id = $scopes[$scopeField];
687
-            $query->whereHas($scopeRelation, function ($query) use ($id, $scopeField) {
687
+            $query->whereHas($scopeRelation, function($query) use ($id, $scopeField) {
688 688
                 $query->where($scopeField, $id);
689 689
             });
690 690
             unset($scopes[$scopeField]);
@@ -715,7 +715,7 @@  discard block
 block discarded – undo
715 715
     {
716 716
 
717 717
         if (isset($scopes[$scopeField]) && is_string($scopes[$scopeField])) {
718
-            $query->where(function ($query) use (&$scopes, $scopeField, $orFields) {
718
+            $query->where(function($query) use (&$scopes, $scopeField, $orFields) {
719 719
                 foreach ($orFields as $field) {
720 720
                     $query->orWhere($field, $this->getLikeOperator(), '%' . $scopes[$scopeField] . '%');
721 721
                     unset($scopes[$field]);
@@ -791,11 +791,11 @@  discard block
 block discarded – undo
791 791
 
792 792
         $uniqueTraits = array_unique(array_map('class_basename', $traits));
793 793
 
794
-        $methods = array_map(function (string $trait) use ($method) {
794
+        $methods = array_map(function(string $trait) use ($method) {
795 795
             return $method . $trait;
796 796
         }, $uniqueTraits);
797 797
 
798
-        return array_filter($methods, function (string $method) {
798
+        return array_filter($methods, function(string $method) {
799 799
             return method_exists(get_called_class(), $method);
800 800
         });
801 801
     }
Please login to merge, or discard this patch.
src/ValidationServiceProvider.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -16,23 +16,23 @@
 block discarded – undo
16 16
      */
17 17
     public function boot()
18 18
     {
19
-        Validator::extend('absolute_or_relative_url', function ($attribute, $value, $parameters, $validator) {
19
+        Validator::extend('absolute_or_relative_url', function($attribute, $value, $parameters, $validator) {
20 20
             return Str::startsWith($value, '/') || Validator::make([$attribute => $value], [$attribute => 'url'])->passes();
21 21
         }, 'The :attribute should be a valid url (absolute or relative)');
22 22
 
23
-        Validator::extend('relative_or_secure_url', function ($attribute, $value, $parameters) {
23
+        Validator::extend('relative_or_secure_url', function($attribute, $value, $parameters) {
24 24
             return Str::startsWith($value, '/') || filter_var($value, FILTER_VALIDATE_URL) !== false && Str::startsWith($value, 'https');
25 25
         }, 'The :attribute should be a valid url (relative or https)');
26 26
 
27
-        Validator::extend('web_color', function ($attribute, $value, $parameters, $validator) {
27
+        Validator::extend('web_color', function($attribute, $value, $parameters, $validator) {
28 28
             return preg_match('/^([A-Fa-f0-9]{6}|[A-Fa-f0-9]{3})$/i', $value);
29 29
         });
30 30
 
31
-        Validator::extend('phone_number', function ($attribute, $value, $parameters) {
31
+        Validator::extend('phone_number', function($attribute, $value, $parameters) {
32 32
             return preg_match("/^[+]?[0-9\-\ ]*$/", $value);
33 33
         });
34 34
 
35
-        Validator::extend('validBlocks', function ($attribute, $value, $parameters, $validator) {
35
+        Validator::extend('validBlocks', function($attribute, $value, $parameters, $validator) {
36 36
             foreach ($value as $block) {
37 37
                 $cmsBlock = $this->app->make(BlockRepository::class)->buildFromCmsArray($block, false);
38 38
 
Please login to merge, or discard this patch.