Passed
Push — feature/duplication ( 011e93 )
by
unknown
11:47
created
src/Models/Behaviors/HasTranslation.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
         if (method_exists($query->getModel(), 'translations')) {
20 20
             $locale = $locale == null ? app()->getLocale() : $locale;
21 21
 
22
-            $query->whereHas('translations', function ($query) use ($locale) {
22
+            $query->whereHas('translations', function($query) use ($locale) {
23 23
                 $query->whereActive(true);
24 24
                 $query->whereLocale($locale);
25 25
 
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
                 }
29 29
             });
30 30
 
31
-            return $query->with(['translations' => function ($query) use ($locale) {
31
+            return $query->with(['translations' => function($query) use ($locale) {
32 32
                 $query->whereActive(true);
33 33
                 $query->whereLocale($locale);
34 34
 
@@ -48,14 +48,14 @@  discard block
 block discarded – undo
48 48
         $locale = $locale == null ? app()->getLocale() : $locale;
49 49
 
50 50
         return $query
51
-            ->join($translationTable, function (JoinClause $join) use ($translationTable, $localeKey, $table, $keyName) {
51
+            ->join($translationTable, function(JoinClause $join) use ($translationTable, $localeKey, $table, $keyName) {
52 52
                 $join
53
-                    ->on($translationTable.'.'.$this->getRelationKey(), '=', $table.'.'.$keyName)
54
-                    ->where($translationTable.'.'.$localeKey, $this->locale());
53
+                    ->on($translationTable . '.' . $this->getRelationKey(), '=', $table . '.' . $keyName)
54
+                    ->where($translationTable . '.' . $localeKey, $this->locale());
55 55
             })
56
-            ->where($translationTable.'.'.$this->getLocaleKey(), $locale)
57
-            ->orderBy($translationTable.'.'.$orderField, $orderType)
58
-            ->select($table.'.*')
56
+            ->where($translationTable . '.' . $this->getLocaleKey(), $locale)
57
+            ->orderBy($translationTable . '.' . $orderField, $orderType)
58
+            ->select($table . '.*')
59 59
             ->with('translations');
60 60
     }
61 61
 
@@ -91,14 +91,14 @@  discard block
 block discarded – undo
91 91
 
92 92
     public function getActiveLanguages()
93 93
     {
94
-        return $this->translations->map(function ($translation) {
94
+        return $this->translations->map(function($translation) {
95 95
             return [
96 96
                 'shortlabel' => strtoupper($translation->locale),
97 97
                 'label' => getLanguageLabelFromLocaleCode($translation->locale),
98 98
                 'value' => $translation->locale,
99 99
                 'published' => $translation->active ?? false,
100 100
             ];
101
-        })->sortBy(function ($translation) {
101
+        })->sortBy(function($translation) {
102 102
             $localesOrdered = config('translatable.locales');
103 103
             return array_search($translation['value'], $localesOrdered);
104 104
         })->values();
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
 
107 107
     public function translatedAttribute($key)
108 108
     {
109
-        return $this->translations->mapWithKeys(function ($translation) use ($key) {
109
+        return $this->translations->mapWithKeys(function($translation) use ($key) {
110 110
             return [$translation->locale => $this->translate($translation->locale)->$key];
111 111
         });
112 112
     }
Please login to merge, or discard this patch.
src/Http/ViewComposers/MediasUploaderConfig.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -51,13 +51,13 @@
 block discarded – undo
51 51
         // anonymous functions are used to let configuration dictate
52 52
         // the execution of the appropriate implementation
53 53
         $endpointByType = [
54
-            'local' => function () {
54
+            'local' => function() {
55 55
                 return $this->urlGenerator->route('admin.media-library.medias.store');
56 56
             },
57
-            's3' => function () use ($libraryDisk) {
57
+            's3' => function() use ($libraryDisk) {
58 58
                 return s3Endpoint($libraryDisk);
59 59
             },
60
-            'azure' => function () use ($libraryDisk) {
60
+            'azure' => function() use ($libraryDisk) {
61 61
                 return azureEndpoint($libraryDisk);
62 62
             },
63 63
         ];
Please login to merge, or discard this patch.
src/Http/ViewComposers/FilesUploaderConfig.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -51,13 +51,13 @@
 block discarded – undo
51 51
         // anonymous functions are used to let configuration dictate
52 52
         // the execution of the appropriate implementation
53 53
         $endpointByType = [
54
-            'local' => function () {
54
+            'local' => function() {
55 55
                 return $this->urlGenerator->route('admin.file-library.files.store');
56 56
             },
57
-            's3' => function () use ($libraryDisk) {
57
+            's3' => function() use ($libraryDisk) {
58 58
                 return s3Endpoint($libraryDisk);
59 59
             },
60
-            'azure' => function () use ($libraryDisk) {
60
+            'azure' => function() use ($libraryDisk) {
61 61
                 return azureEndpoint($libraryDisk);
62 62
             },
63 63
         ];
Please login to merge, or discard this patch.
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/Repositories/Behaviors/HandleFieldsGroups.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
             if ($object->$group) {
39 39
                 $decoded_fields = json_decode($object->$group, true);
40 40
                 // In case that some field read the value through $item->$name
41
-                foreach($decoded_fields as $field_name => $field_value) {
41
+                foreach ($decoded_fields as $field_name => $field_value) {
42 42
                     $object->setAttribute($field_name, $field_value);
43 43
                 }
44 44
                 $fields = array_merge($fields, $decoded_fields);
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
 
51 51
     protected function handleFieldsGroups($fields) {
52 52
         foreach ($this->fieldsGroups as $group => $groupFields) {
53
-            $fields[$group] = Arr::where(Arr::only($fields, $groupFields), function ($value, $key) {
53
+            $fields[$group] = Arr::where(Arr::only($fields, $groupFields), function($value, $key) {
54 54
                 return !empty($value);
55 55
             });
56 56
             Arr::forget($fields, $groupFields);
Please login to merge, or discard this patch.