Passed
Pull Request — 2.x (#729)
by Antonio Carlos
06:30
created
src/Commands/ModuleMakeDeprecated.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@
 block discarded – undo
37 37
     {
38 38
         $this->call('twill:make:module', [
39 39
             'moduleName' => $this->argument('moduleName'),
40
-        ] + collect($this->options())->mapWithKeys(function ($value, $key) {
40
+        ] + collect($this->options())->mapWithKeys(function($value, $key) {
41 41
             return ["--{$key}" => $value];
42 42
         })->toArray());
43 43
     }
Please login to merge, or discard this patch.
src/Models/Media.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -22,18 +22,18 @@  discard block
 block discarded – undo
22 22
 
23 23
     public function __construct(array $attributes = [])
24 24
     {
25
-        $this->fillable(array_merge($this->fillable, Collection::make(config('twill.media_library.extra_metadatas_fields'))->map(function ($field) {
25
+        $this->fillable(array_merge($this->fillable, Collection::make(config('twill.media_library.extra_metadatas_fields'))->map(function($field) {
26 26
             return $field['name'];
27 27
         })->toArray()));
28 28
 
29
-        Collection::make(config('twill.media_library.translatable_metadatas_fields'))->each(function ($field) {
29
+        Collection::make(config('twill.media_library.translatable_metadatas_fields'))->each(function($field) {
30 30
             $this->casts[$field] = 'json';
31 31
         });
32 32
 
33 33
         parent::__construct($attributes);
34 34
     }
35 35
 
36
-    public function scopeUnused ($query)
36
+    public function scopeUnused($query)
37 37
     {
38 38
         $usedIds = DB::table(config('twill.mediables_table'))->get()->pluck('media_id');
39 39
         return $query->whereNotIn('id', $usedIds->toArray())->get();
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
             'medium' => ImageService::getUrl($this->uuid, ["h" => "430"]),
70 70
             'width' => $this->width,
71 71
             'height' => $this->height,
72
-            'tags' => $this->tags->map(function ($tag) {
72
+            'tags' => $this->tags->map(function($tag) {
73 73
                 return $tag->name;
74 74
             }),
75 75
             'deleteUrl' => $this->canDeleteSafely() ? moduleRoute('medias', 'media-library', 'destroy', $this->id) : null,
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
                     'caption' => $this->caption,
82 82
                     'altText' => $this->alt_text,
83 83
                     'video' => null,
84
-                ] + Collection::make(config('twill.media_library.extra_metadatas_fields'))->mapWithKeys(function ($field) {
84
+                ] + Collection::make(config('twill.media_library.extra_metadatas_fields'))->mapWithKeys(function($field) {
85 85
                     return [
86 86
                         $field['name'] => $this->{$field['name']},
87 87
                     ];
Please login to merge, or discard this patch.
src/Models/File.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
         return DB::table(config('twill.fileables_table', 'twill_fileables'))->where('file_id', $this->id)->count() === 0;
26 26
     }
27 27
 
28
-    public function scopeUnused ($query)
28
+    public function scopeUnused($query)
29 29
     {
30 30
         $usedIds = DB::table(config('twill.fileables_table'))->get()->pluck('file_id');
31 31
         return $query->whereNotIn('id', $usedIds->toArray())->get();
Please login to merge, or discard this patch.
src/Helpers/helpers.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -9,7 +9,7 @@  discard block
 block discarded – undo
9 9
 if (!function_exists('dumpUsableSqlQuery')) {
10 10
     function dumpUsableSqlQuery($query)
11 11
     {
12
-        dd(vsprintf(str_replace('?', '%s', $query->toSql()), array_map(function ($binding) {
12
+        dd(vsprintf(str_replace('?', '%s', $query->toSql()), array_map(function($binding) {
13 13
             return is_numeric($binding) ? $binding : "'{$binding}'";
14 14
         }, $query->getBindings())));
15 15
     }
@@ -201,12 +201,12 @@  discard block
 block discarded – undo
201 201
     {
202 202
         $blockList = app(BlockCollection::class)->getBlockList();
203 203
 
204
-        $appBlocksList = $blockList->filter(function ($block) {
204
+        $appBlocksList = $blockList->filter(function($block) {
205 205
             return $block['source'] !== A17\Twill\Services\Blocks\Block::SOURCE_TWILL;
206 206
         });
207 207
 
208 208
         return $blockList->filter(
209
-            function ($block) use ($blocks, $groups, $appBlocksList) {
209
+            function($block) use ($blocks, $groups, $appBlocksList) {
210 210
                 if ($block['group'] === A17\Twill\Services\Blocks\Block::SOURCE_TWILL) {
211 211
                     if (!collect(
212 212
                         config('twill.block_editor.use_twill_blocks')
@@ -215,7 +215,7 @@  discard block
 block discarded – undo
215 215
                     }
216 216
 
217 217
                     if (count($appBlocksList) > 0 && $appBlocksList->contains(
218
-                        function ($appBlock) use ($block) {
218
+                        function($appBlock) use ($block) {
219 219
                             return $appBlock['name'] === $block['name'];
220 220
                         })
221 221
                     ) {
Please login to merge, or discard this patch.
src/Http/Controllers/Admin/IconsController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -20,12 +20,12 @@  discard block
 block discarded – undo
20 20
     {
21 21
         $icons = collect(
22 22
             config('twill.block_editor.directories.source.icons')
23
-        )->reduce(function (Collection $keep, $path) {
23
+        )->reduce(function(Collection $keep, $path) {
24 24
             if (!$this->files->exists($path)) {
25 25
                 return $keep;
26 26
             }
27 27
 
28
-            $files = collect($this->files->files($path))->map(function (
28
+            $files = collect($this->files->files($path))->map(function(
29 29
                 SplFileInfo $file
30 30
             ) {
31 31
                 if (in_array($file->getFilename(), config('twill.internal_icons'))) {
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
             abort(404);
55 55
         }
56 56
 
57
-        return response()->stream(function () use ($file) {
57
+        return response()->stream(function() use ($file) {
58 58
             echo $this->files->get($file);
59 59
         }, 200, ["Content-Type" => "image/svg+xml"]);
60 60
 
Please login to merge, or discard this patch.
src/Commands/ListBlocks.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
     {
48 48
         $this->blockCollection
49 49
             ->getMissingDirectories()
50
-            ->each(function ($directory) {
50
+            ->each(function($directory) {
51 51
                 $this->error("Directory not found: {$directory}");
52 52
             });
53 53
     }
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
         return $blockCollection
62 62
             ->first()
63 63
             ->keys()
64
-            ->map(function ($key) {
64
+            ->map(function($key) {
65 65
                 return Str::studly($key);
66 66
             })
67 67
             ->toArray();
@@ -81,20 +81,20 @@  discard block
 block discarded – undo
81 81
 
82 82
         return $this->blockCollection
83 83
             ->collect()
84
-            ->reject(function (Block $block) use ($sourceFiltered) {
84
+            ->reject(function(Block $block) use ($sourceFiltered) {
85 85
                 return $sourceFiltered && !$this->option($block->source);
86 86
             })
87
-            ->reject(function (Block $block) use ($typeFiltered) {
87
+            ->reject(function(Block $block) use ($typeFiltered) {
88 88
                 return $this->dontPassTextFilter($block) ||
89 89
                     ($typeFiltered &&
90 90
                     !$this->option(Str::plural($block->type)));
91 91
             })
92
-            ->map(function (Block $block) {
92
+            ->map(function(Block $block) {
93 93
                 return $this->colorize(
94 94
                     $this->option('short') ? $block->toShortList() : $block->toList()
95 95
                 );
96 96
             })
97
-            ->sortBy(function ($block) {
97
+            ->sortBy(function($block) {
98 98
                 return [$block['group'], $block['title']];
99 99
             });
100 100
     }
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
         if (filled($filter = $this->argument('filter'))) {
153 153
             return !$block
154 154
                 ->toList()
155
-                ->reduce(function ($keep, $element) use ($filter) {
155
+                ->reduce(function($keep, $element) use ($filter) {
156 156
                     return $keep ||
157 157
                     Str::contains(
158 158
                         Str::lower($element),
Please login to merge, or discard this patch.
src/Services/Blocks/BlockMaker.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -353,7 +353,7 @@  discard block
 block discarded – undo
353 353
 
354 354
         return collect(
355 355
             config('twill.block_editor.directories.source.icons')
356
-        )->reduce(function ($keep, $path) use ($icon) {
356
+        )->reduce(function($keep, $path) use ($icon) {
357 357
             if ($keep) {
358 358
                 return $keep;
359 359
             }
@@ -362,7 +362,7 @@  discard block
 block discarded – undo
362 362
                 return null;
363 363
             }
364 364
 
365
-            return collect($this->files->files($path))->reduce(function ($keep, SplFileInfo $file) use ($icon) {
365
+            return collect($this->files->files($path))->reduce(function($keep, SplFileInfo $file) use ($icon) {
366 366
                 if ($keep) {
367 367
                     return $keep;
368 368
                 }
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
@@ -46,8 +46,8 @@  discard block
 block discarded – undo
46 46
         if (Schema::hasTable(config('twill.related_table', 'twill_related'))) {
47 47
             $relatedItems = Collection::make();
48 48
 
49
-            Collection::make($fields['browsers'])->each(function ($items, $browserName) use (&$relatedItems) {
50
-                Collection::make($items)->each(function ($item) use ($browserName, &$relatedItems) {
49
+            Collection::make($fields['browsers'])->each(function($items, $browserName) use (&$relatedItems) {
50
+                Collection::make($items)->each(function($item) use ($browserName, &$relatedItems) {
51 51
                     try {
52 52
                         $repository = $this->getModelRepository($item['endpointType'] ?? $browserName);
53 53
                         $relatedItems->push((object) [
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
     {
76 76
         if (Schema::hasTable(config('twill.related_table', 'twill_related'))) {
77 77
             if (isset($fields['browsers'])) {
78
-                Collection::make($fields['browsers'])->each(function ($items, $browserName) use ($object) {
78
+                Collection::make($fields['browsers'])->each(function($items, $browserName) use ($object) {
79 79
                     $object->saveRelated($items, $browserName);
80 80
                 });
81 81
             }
@@ -107,14 +107,14 @@  discard block
 block discarded – undo
107 107
             $blocksList = app(BlockCollection::class)->getBlockList();
108 108
         }
109 109
 
110
-        $block['type'] = $blocksList->keyBy('name')->search(function ($blockConfig) use ($block) {
110
+        $block['type'] = $blocksList->keyBy('name')->search(function($blockConfig) use ($block) {
111 111
             return $blockConfig['component'] === $block['type'];
112 112
         });
113 113
 
114 114
         $block['content'] = empty($block['content']) ? new \stdClass : (object) $block['content'];
115 115
 
116 116
         if ($block['browsers']) {
117
-            $browsers = Collection::make($block['browsers'])->map(function ($items) {
117
+            $browsers = Collection::make($block['browsers'])->map(function($items) {
118 118
                 return Collection::make($items)->pluck('id');
119 119
             })->toArray();
120 120
 
Please login to merge, or discard this patch.
src/Http/Controllers/Admin/ModuleController.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -331,7 +331,7 @@  discard block
 block discarded – undo
331 331
             "$this->viewPrefix.index",
332 332
             "twill::$this->moduleName.index",
333 333
             "twill::layouts.listing",
334
-        ])->first(function ($view) {
334
+        ])->first(function($view) {
335 335
             return View::exists($view);
336 336
         });
337 337
 
@@ -442,7 +442,7 @@  discard block
 block discarded – undo
442 442
             "$this->viewPrefix.form",
443 443
             "twill::$this->moduleName.form",
444 444
             "twill::layouts.form",
445
-        ])->first(function ($view) {
445
+        ])->first(function($view) {
446 446
             return View::exists($view);
447 447
         });
448 448
 
@@ -469,7 +469,7 @@  discard block
 block discarded – undo
469 469
             "$this->viewPrefix.form",
470 470
             "twill::$this->moduleName.form",
471 471
             "twill::layouts.form",
472
-        ])->first(function ($view) {
472
+        ])->first(function($view) {
473 473
             return View::exists($view);
474 474
         });
475 475
 
@@ -592,7 +592,7 @@  discard block
 block discarded – undo
592 592
             "$this->viewPrefix.form",
593 593
             "twill::$this->moduleName.form",
594 594
             "twill::layouts.form",
595
-        ])->first(function ($view) {
595
+        ])->first(function($view) {
596 596
             return View::exists($view);
597 597
         });
598 598
 
@@ -839,7 +839,7 @@  discard block
 block discarded – undo
839 839
         $query = $this->request->input('q');
840 840
         $tags = $this->repository->getTags($query);
841 841
 
842
-        return Response::json(['items' => $tags->map(function ($tag) {
842
+        return Response::json(['items' => $tags->map(function($tag) {
843 843
             return $tag->name;
844 844
         })], 200);
845 845
     }
@@ -927,8 +927,8 @@  discard block
 block discarded – undo
927 927
     protected function getIndexTableData($items)
928 928
     {
929 929
         $translated = $this->moduleHas('translations');
930
-        return $items->map(function ($item) use ($translated) {
931
-            $columnsData = Collection::make($this->indexColumns)->mapWithKeys(function ($column) use ($item) {
930
+        return $items->map(function($item) use ($translated) {
931
+            $columnsData = Collection::make($this->indexColumns)->mapWithKeys(function($column) use ($item) {
932 932
                 return $this->getItemColumnData($item, $column);
933 933
             })->toArray();
934 934
 
@@ -1188,7 +1188,7 @@  discard block
 block discarded – undo
1188 1188
             'feature',
1189 1189
             'bulkFeature',
1190 1190
             'bulkDelete',
1191
-        ])->mapWithKeys(function ($endpoint) {
1191
+        ])->mapWithKeys(function($endpoint) {
1192 1192
             return [
1193 1193
                 $endpoint . 'Url' => $this->getIndexOption($endpoint) ? moduleRoute(
1194 1194
                     $this->moduleName, $this->routePrefix, $endpoint,
@@ -1204,7 +1204,7 @@  discard block
 block discarded – undo
1204 1204
      */
1205 1205
     protected function getIndexOption($option)
1206 1206
     {
1207
-        return once(function () use ($option) {
1207
+        return once(function() use ($option) {
1208 1208
             $customOptionNamesMapping = [
1209 1209
                 'store' => 'create',
1210 1210
             ];
@@ -1261,8 +1261,8 @@  discard block
 block discarded – undo
1261 1261
     {
1262 1262
         $withImage = $this->moduleHas('medias');
1263 1263
 
1264
-        return $items->map(function ($item) use ($withImage) {
1265
-            $columnsData = Collection::make($this->browserColumns)->mapWithKeys(function ($column) use ($item) {
1264
+        return $items->map(function($item) use ($withImage) {
1265
+            $columnsData = Collection::make($this->browserColumns)->mapWithKeys(function($column) use ($item) {
1266 1266
                 return $this->getItemColumnData($item, $column);
1267 1267
             })->toArray();
1268 1268
 
@@ -1328,7 +1328,7 @@  discard block
 block discarded – undo
1328 1328
                     $fieldSplitted = explode('|', $field);
1329 1329
                     if (count($fieldSplitted) > 1) {
1330 1330
                         $requestValue = $requestFilters[$key];
1331
-                        Collection::make($fieldSplitted)->each(function ($scopeKey) use (&$scope, $requestValue) {
1331
+                        Collection::make($fieldSplitted)->each(function($scopeKey) use (&$scope, $requestValue) {
1332 1332
                             $scope[$scopeKey] = $requestValue;
1333 1333
                         });
1334 1334
                     } else {
@@ -1505,11 +1505,11 @@  discard block
 block discarded – undo
1505 1505
      */
1506 1506
     protected function validateFormRequest()
1507 1507
     {
1508
-        $unauthorizedFields = Collection::make($this->fieldsPermissions)->filter(function ($permission, $field) {
1508
+        $unauthorizedFields = Collection::make($this->fieldsPermissions)->filter(function($permission, $field) {
1509 1509
             return Auth::guard('twill_users')->user()->cannot($permission);
1510 1510
         })->keys();
1511 1511
 
1512
-        $unauthorizedFields->each(function ($field) {
1512
+        $unauthorizedFields->each(function($field) {
1513 1513
             $this->request->offsetUnset($field);
1514 1514
         });
1515 1515
 
@@ -1755,7 +1755,7 @@  discard block
 block discarded – undo
1755 1755
      */
1756 1756
     public function getRepeaterList()
1757 1757
     {
1758
-        return app(BlockCollection::class)->getRepeaterList()->mapWithKeys(function ($repeater) {
1758
+        return app(BlockCollection::class)->getRepeaterList()->mapWithKeys(function($repeater) {
1759 1759
             return [$repeater['name'] => $repeater];
1760 1760
         });
1761 1761
     }
Please login to merge, or discard this patch.