Passed
Push — pr/513 ( 76ed50 )
by Quentin
11:53 queued 01:01
created
src/Http/Controllers/Admin/BlocksController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -56,16 +56,16 @@
 block discarded – undo
56 56
 
57 57
         $blocksCollection->push($newBlock);
58 58
 
59
-        $block['blocks']->each(function ($childBlock) use ($newBlock, $blocksCollection, $blockRepository) {
59
+        $block['blocks']->each(function($childBlock) use ($newBlock, $blocksCollection, $blockRepository) {
60 60
             $childBlock['parent_id'] = $newBlock->id;
61 61
             $newChildBlock = $blockRepository->createForPreview($childBlock);
62 62
             $blocksCollection->push($newChildBlock);
63 63
         });
64 64
 
65
-        $renderedBlocks = $blocksCollection->where('parent_id', null)->map(function ($block) use ($blocksCollection, $viewFactory, $config) {
65
+        $renderedBlocks = $blocksCollection->where('parent_id', null)->map(function($block) use ($blocksCollection, $viewFactory, $config) {
66 66
             if ($config->get('twill.block_editor.block_preview_render_childs') ?? true) {
67 67
                 $childBlocks = $blocksCollection->where('parent_id', $block->id);
68
-                $renderedChildViews = $childBlocks->map(function ($childBlock) use ($viewFactory, $config) {
68
+                $renderedChildViews = $childBlocks->map(function($childBlock) use ($viewFactory, $config) {
69 69
                     $view = $this->getBlockView($childBlock->type, $config);
70 70
 
71 71
                     return $viewFactory->exists($view) ? $viewFactory->make($view, [
Please login to merge, or discard this patch.
src/Http/Controllers/Admin/DashboardController.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -108,14 +108,14 @@  discard block
 block discarded – undo
108 108
     {
109 109
         $modules = Collection::make($this->config->get('twill.dashboard.modules'));
110 110
 
111
-        return $modules->filter(function ($module) {
111
+        return $modules->filter(function($module) {
112 112
             return ($module['search'] ?? false);
113
-        })->map(function ($module) use ($request) {
113
+        })->map(function($module) use ($request) {
114 114
             $repository = $this->getRepository($module['name']);
115 115
 
116 116
             $found = $repository->cmsSearch($request->get('search'), $module['search_fields'] ?? ['title'])->take(10);
117 117
 
118
-            return $found->map(function ($item) use ($module) {
118
+            return $found->map(function($item) use ($module) {
119 119
                 try {
120 120
                     $author = $item->revisions()->latest()->first()->user->name ?? 'Admin';
121 121
                 } catch (\Exception $e) {
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
      */
143 143
     private function getAllActivities()
144 144
     {
145
-        return Activity::take(20)->latest()->get()->map(function ($activity) {
145
+        return Activity::take(20)->latest()->get()->map(function($activity) {
146 146
             return $this->formatActivity($activity);
147 147
         })->filter()->values();
148 148
     }
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
      */
153 153
     private function getLoggedInUserActivities()
154 154
     {
155
-        return Activity::where('causer_id', $this->authFactory->guard('twill_users')->user()->id)->take(20)->latest()->get()->map(function ($activity) {
155
+        return Activity::where('causer_id', $this->authFactory->guard('twill_users')->user()->id)->take(20)->latest()->get()->map(function($activity) {
156 156
             return $this->formatActivity($activity);
157 157
         })->filter()->values();
158 158
     }
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
             return [];
202 202
         }
203 203
 
204
-        $statsByDate = Collection::make($response['rows'] ?? [])->map(function (array $dateRow) {
204
+        $statsByDate = Collection::make($response['rows'] ?? [])->map(function(array $dateRow) {
205 205
             return [
206 206
                 'date' => $dateRow[0],
207 207
                 'users' => (int) $dateRow[1],
@@ -216,7 +216,7 @@  discard block
 block discarded – undo
216 216
             'yesterday',
217 217
             'week',
218 218
             'month',
219
-        ])->mapWithKeys(function ($period) use ($statsByDate) {
219
+        ])->mapWithKeys(function($period) use ($statsByDate) {
220 220
             $stats = $this->getPeriodStats($period, $statsByDate);
221 221
             return [
222 222
                 $period => [
@@ -253,10 +253,10 @@  discard block
 block discarded – undo
253 253
                 'stats' => $stats = $statsByDate->first(),
254 254
                 'moreUsers' => $stats['users'] > $statsByDate->get(1)['users'],
255 255
                 'morePageViews' => $stats['pageViews'] > $statsByDate->get(1)['pageViews'],
256
-                'usersData' => $statsByDate->take(7)->map(function ($stat) {
256
+                'usersData' => $statsByDate->take(7)->map(function($stat) {
257 257
                     return $stat['users'];
258 258
                 }),
259
-                'pageViewsData' => $statsByDate->take(7)->map(function ($stat) {
259
+                'pageViewsData' => $statsByDate->take(7)->map(function($stat) {
260 260
                     return $stat['pageViews'];
261 261
                 }),
262 262
             ];
@@ -265,10 +265,10 @@  discard block
 block discarded – undo
265 265
                 'stats' => $stats = $statsByDate->get(1),
266 266
                 'moreUsers' => $stats['users'] > $statsByDate->get(2)['users'],
267 267
                 'morePageViews' => $stats['pageViews'] > $statsByDate->get(2)['pageViews'],
268
-                'usersData' => $statsByDate->slice(1)->take(7)->map(function ($stat) {
268
+                'usersData' => $statsByDate->slice(1)->take(7)->map(function($stat) {
269 269
                     return $stat['users'];
270 270
                 }),
271
-                'pageViewsData' => $statsByDate->slice(1)->take(7)->map(function ($stat) {
271
+                'pageViewsData' => $statsByDate->slice(1)->take(7)->map(function($stat) {
272 272
                     return $stat['pageViews'];
273 273
                 }),
274 274
             ];
@@ -291,10 +291,10 @@  discard block
 block discarded – undo
291 291
                 'stats' => $stats,
292 292
                 'moreUsers' => $stats['users'] > $compareStats['users'],
293 293
                 'morePageViews' => $stats['pageViews'] > $compareStats['pageViews'],
294
-                'usersData' => $statsByDate->slice(1)->take(29)->map(function ($stat) {
294
+                'usersData' => $statsByDate->slice(1)->take(29)->map(function($stat) {
295 295
                     return $stat['users'];
296 296
                 }),
297
-                'pageViewsData' => $statsByDate->slice(1)->take(29)->map(function ($stat) {
297
+                'pageViewsData' => $statsByDate->slice(1)->take(29)->map(function($stat) {
298 298
                     return $stat['pageViews'];
299 299
                 }),
300 300
             ];
@@ -317,10 +317,10 @@  discard block
 block discarded – undo
317 317
                 'stats' => $stats,
318 318
                 'moreUsers' => $stats['users'] > $compareStats['users'],
319 319
                 'morePageViews' => $stats['pageViews'] > $compareStats['pageViews'],
320
-                'usersData' => $statsByDate->slice(1)->take(29)->map(function ($stat) {
320
+                'usersData' => $statsByDate->slice(1)->take(29)->map(function($stat) {
321 321
                     return $stat['users'];
322 322
                 }),
323
-                'pageViewsData' => $statsByDate->slice(1)->take(29)->map(function ($stat) {
323
+                'pageViewsData' => $statsByDate->slice(1)->take(29)->map(function($stat) {
324 324
                     return $stat['pageViews'];
325 325
                 }),
326 326
             ];
@@ -346,9 +346,9 @@  discard block
 block discarded – undo
346 346
      */
347 347
     private function getShortcuts($modules)
348 348
     {
349
-        return $modules->filter(function ($module) {
349
+        return $modules->filter(function($module) {
350 350
             return ($module['count'] ?? false) || ($module['create'] ?? false);
351
-        })->map(function ($module) {
351
+        })->map(function($module) {
352 352
             $repository = $this->getRepository($module['name']);
353 353
 
354 354
             $moduleOptions = [
@@ -385,14 +385,14 @@  discard block
 block discarded – undo
385 385
      */
386 386
     private function getDrafts($modules)
387 387
     {
388
-        return $modules->filter(function ($module) {
388
+        return $modules->filter(function($module) {
389 389
             return ($module['draft'] ?? false);
390
-        })->map(function ($module) {
390
+        })->map(function($module) {
391 391
             $repository = $this->getRepository($module['name']);
392 392
 
393 393
             $drafts = $repository->draft()->mine()->limit(3)->latest()->get();
394 394
 
395
-            return $drafts->map(function ($draft) use ($module) {
395
+            return $drafts->map(function($draft) use ($module) {
396 396
                 return [
397 397
                     'type' => ucfirst($module['label_singular'] ?? Str::singular($module['name'])),
398 398
                     'name' => $draft->titleInDashboard ?? $draft->title,
Please login to merge, or discard this patch.
src/Http/Requests/Admin/UserRequest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@
 block discarded – undo
40 40
                         'name' => 'required',
41 41
                         'role' => 'not_in:SUPERADMIN',
42 42
                         'email' => 'required|email|unique:' . config('twill.users_table', 'twill_users') . ',email,' . $this->route('user'),
43
-                        'verify-code' => function ($attribute, $value, $fail) {
43
+                        'verify-code' => function($attribute, $value, $fail) {
44 44
                             $user = Auth::guard('twill_users')->user();
45 45
                             $with2faSettings = config('twill.enabled.users-2fa') && $user->id == $this->route('user');
46 46
 
Please login to merge, or discard this patch.
src/Commands/GenerateBlocks.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@
 block discarded – undo
64 64
 
65 65
         $this->info(self::SCANNING_BLOCKS);
66 66
 
67
-        Collection::make($this->filesystem->files($path))->each(function ($viewFile) {
67
+        Collection::make($this->filesystem->files($path))->each(function($viewFile) {
68 68
             $blockName = $viewFile->getBasename('.blade.php');
69 69
 
70 70
             $vueBlockTemplate = $this->viewFactory->make('admin.blocks.' . $blockName, ['renderForBlocks' => true])->render();
Please login to merge, or discard this patch.
src/AuthServiceProvider.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -31,72 +31,72 @@
 block discarded – undo
31 31
 
32 32
     public function boot()
33 33
     {
34
-        Gate::define('list', function ($user) {
35
-            return $this->authorize($user, function ($user) {
34
+        Gate::define('list', function($user) {
35
+            return $this->authorize($user, function($user) {
36 36
                 return $this->userHasRole($user, [UserRole::VIEWONLY, UserRole::PUBLISHER, UserRole::ADMIN]);
37 37
             });
38 38
         });
39 39
 
40
-        Gate::define('edit', function ($user) {
41
-            return $this->authorize($user, function ($user) {
40
+        Gate::define('edit', function($user) {
41
+            return $this->authorize($user, function($user) {
42 42
                 return $this->userHasRole($user, [UserRole::PUBLISHER, UserRole::ADMIN]);
43 43
             });
44 44
         });
45 45
 
46
-        Gate::define('reorder', function ($user) {
47
-            return $this->authorize($user, function ($user) {
46
+        Gate::define('reorder', function($user) {
47
+            return $this->authorize($user, function($user) {
48 48
                 return $this->userHasRole($user, [UserRole::PUBLISHER, UserRole::ADMIN]);
49 49
             });
50 50
         });
51 51
 
52
-        Gate::define('publish', function ($user) {
53
-            return $this->authorize($user, function ($user) {
52
+        Gate::define('publish', function($user) {
53
+            return $this->authorize($user, function($user) {
54 54
                 return $this->userHasRole($user, [UserRole::PUBLISHER, UserRole::ADMIN]);
55 55
             });
56 56
         });
57 57
 
58
-        Gate::define('feature', function ($user) {
59
-            return $this->authorize($user, function ($user) {
58
+        Gate::define('feature', function($user) {
59
+            return $this->authorize($user, function($user) {
60 60
                 return $this->userHasRole($user, [UserRole::PUBLISHER, UserRole::ADMIN]);
61 61
             });
62 62
         });
63 63
 
64
-        Gate::define('delete', function ($user) {
65
-            return $this->authorize($user, function ($user) {
64
+        Gate::define('delete', function($user) {
65
+            return $this->authorize($user, function($user) {
66 66
                 return $this->userHasRole($user, [UserRole::PUBLISHER, UserRole::ADMIN]);
67 67
             });
68 68
         });
69 69
 
70
-        Gate::define('upload', function ($user) {
71
-            return $this->authorize($user, function ($user) {
70
+        Gate::define('upload', function($user) {
71
+            return $this->authorize($user, function($user) {
72 72
                 return $this->userHasRole($user, [UserRole::PUBLISHER, UserRole::ADMIN]);
73 73
             });
74 74
         });
75 75
 
76
-        Gate::define('manage-users', function ($user) {
77
-            return $this->authorize($user, function ($user) {
76
+        Gate::define('manage-users', function($user) {
77
+            return $this->authorize($user, function($user) {
78 78
                 return $this->userHasRole($user, [UserRole::ADMIN]);
79 79
             });
80 80
         });
81 81
 
82 82
         // As an admin, I can edit users, except superadmins
83 83
         // As a non-admin, I can edit myself only
84
-        Gate::define('edit-user', function ($user, $editedUser = null) {
85
-            return $this->authorize($user, function ($user) use ($editedUser) {
84
+        Gate::define('edit-user', function($user, $editedUser = null) {
85
+            return $this->authorize($user, function($user) use ($editedUser) {
86 86
                 $editedUserObject = User::find($editedUser);
87 87
                 return ($this->userHasRole($user, [UserRole::ADMIN]) || $user->id == $editedUser)
88 88
                     && ($editedUserObject ? $editedUserObject->role !== self::SUPERADMIN : true);
89 89
             });
90 90
         });
91 91
 
92
-        Gate::define('publish-user', function ($user) {
93
-            return $this->authorize($user, function ($user) {
92
+        Gate::define('publish-user', function($user) {
93
+            return $this->authorize($user, function($user) {
94 94
                 $editedUserObject = User::find(request('id'));
95 95
                 return $this->userHasRole($user, [UserRole::ADMIN]) && ($editedUserObject ? $user->id !== $editedUserObject->id && $editedUserObject->role !== self::SUPERADMIN : false);
96 96
             });
97 97
         });
98 98
 
99
-        Gate::define('impersonate', function ($user) {
99
+        Gate::define('impersonate', function($user) {
100 100
             return $user->role === self::SUPERADMIN;
101 101
         });
102 102
 
Please login to merge, or discard this patch.
src/Repositories/SettingRepository.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
      */
35 35
     public function byKey($key, $section = null)
36 36
     {
37
-        return $this->model->when($section, function ($query) use ($section) {
37
+        return $this->model->when($section, function($query) use ($section) {
38 38
             $query->where('section', $section);
39 39
         })->where('key', $key)->exists() ? $this->model->where('key', $key)->with('translations')->first()->value : null;
40 40
     }
@@ -45,15 +45,15 @@  discard block
 block discarded – undo
45 45
      */
46 46
     public function getFormFields($section = null)
47 47
     {
48
-        $settings = $this->model->when($section, function ($query) use ($section) {
48
+        $settings = $this->model->when($section, function($query) use ($section) {
49 49
             $query->where('section', $section);
50 50
         })->with('translations', 'medias')->get();
51 51
 
52
-        $medias = $settings->mapWithKeys(function ($setting) {
52
+        $medias = $settings->mapWithKeys(function($setting) {
53 53
             return [$setting->key => parent::getFormFields($setting)['medias'][$setting->key] ?? null];
54 54
         })->filter()->toArray();
55 55
 
56
-        return $settings->mapWithKeys(function ($setting) {
56
+        return $settings->mapWithKeys(function($setting) {
57 57
             $settingValue = [];
58 58
 
59 59
             foreach ($setting->translations as $translation) {
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
             $this->updateOrCreate($section + ['key' => $role], $section + [
102 102
                 'key' => $role,
103 103
                 'medias' => [
104
-                    $role => Collection::make($settingsFields['medias'][$role])->map(function ($media) {
104
+                    $role => Collection::make($settingsFields['medias'][$role])->map(function($media) {
105 105
                         return json_decode($media, true);
106 106
                     })->values()->filter()->toArray(),
107 107
                 ],
Please login to merge, or discard this patch.
src/Repositories/UserRepository.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@
 block discarded – undo
62 62
      */
63 63
     public function filter($query, array $scopes = [])
64 64
     {
65
-        $query->when(isset($scopes['role']), function ($query) use ($scopes) {
65
+        $query->when(isset($scopes['role']), function($query) use ($scopes) {
66 66
             $query->where('role', $scopes['role']);
67 67
         });
68 68
         $query->where('role', '<>', 'SUPERADMIN');
Please login to merge, or discard this patch.
src/Repositories/Behaviors/HandleTranslations.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -23,12 +23,12 @@  discard block
 block discarded – undo
23 23
 
24 24
             $submittedLanguages = Collection::make($fields['languages'] ?? []);
25 25
 
26
-            $atLeastOneLanguageIsPublished = $submittedLanguages->contains(function ($language) {
26
+            $atLeastOneLanguageIsPublished = $submittedLanguages->contains(function($language) {
27 27
                 return $language['published'];
28 28
             });
29 29
 
30 30
             foreach ($locales as $index => $locale) {
31
-                $submittedLanguage = Arr::first($submittedLanguages->filter(function ($lang) use ($locale) {
31
+                $submittedLanguage = Arr::first($submittedLanguages->filter(function($lang) use ($locale) {
32 32
                     return $lang['value'] == $locale;
33 33
                 }));
34 34
 
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 
39 39
                 $fields[$locale] = [
40 40
                     'active' => $activeField,
41
-                ] + $attributes->mapWithKeys(function ($attribute) use (&$fields, $locale, $localesCount, $index) {
41
+                ] + $attributes->mapWithKeys(function($attribute) use (&$fields, $locale, $localesCount, $index) {
42 42
                     $attributeValue = $fields[$attribute] ?? null;
43 43
 
44 44
                     // if we are at the last locale,
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
     {
80 80
         if (property_exists($this->model, 'translatedAttributes')) {
81 81
             $attributes = $this->model->translatedAttributes;
82
-            $query->whereHas('translations', function ($q) use ($scopes, $attributes) {
82
+            $query->whereHas('translations', function($q) use ($scopes, $attributes) {
83 83
                 foreach ($attributes as $attribute) {
84 84
                     if (isset($scopes[$attribute]) && is_string($scopes[$attribute])) {
85 85
                         $q->where($attribute, 'like', '%' . $scopes[$attribute] . '%');
Please login to merge, or discard this patch.
src/Repositories/Behaviors/HandleSlugs.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@
 block discarded – undo
78 78
         }
79 79
 
80 80
         if (!$item && config('translatable.use_property_fallback', false)) {
81
-            $item = (clone $query)->orWhere(function ($query) {
81
+            $item = (clone $query)->orWhere(function($query) {
82 82
                 return $query->withActiveTranslations(config('translatable.fallback_locale'));
83 83
             })->forFallbackLocaleSlug($slug)->first();
84 84
 
Please login to merge, or discard this patch.