Passed
Push — master ( 66283a...fa07c5 )
by Al Amin
05:24
created
src/Http/routes.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -8,10 +8,10 @@
 block discarded – undo
8 8
     Route::get('roles/menus', 'RoleController@menus')->name('roles.menus');
9 9
     Route::get('menus/{node}/tree', 'MenuController@tree')->name('menus.tree');
10 10
     Route::resources([
11
-         'roles'   => 'RoleController',
12
-         'sections' => 'SectionController',
13
-         'menus'    => 'MenuController'
14
-     ]);
11
+            'roles'   => 'RoleController',
12
+            'sections' => 'SectionController',
13
+            'menus'    => 'MenuController'
14
+        ]);
15 15
     Route::get('selected-menus', 'MenuController@selected')->name('menus.selected');
16 16
 
17 17
     Route::resource('users', 'UserController')->only(['index', 'edit', 'update']);
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@
 block discarded – undo
1 1
 <?php
2 2
 
3
-Route::name('menu-maker::')->group(function () {
3
+Route::name('menu-maker::')->group(function() {
4 4
     Route::redirect('/', config('menu.path') . '/users');
5 5
 
6 6
     Route::put('roles/assign', 'RoleController@assign')->name('roles.assign');
Please login to merge, or discard this patch.
src/MenuServiceProvider.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -24,11 +24,11 @@  discard block
 block discarded – undo
24 24
         Role::observe(RoleObserver::class);
25 25
         Menu::observe(MenuObserver::class);
26 26
 
27
-        AuthGuard::macro('menus', function ($section) {
27
+        AuthGuard::macro('menus', function($section) {
28 28
             return $this->user()->menus($section);
29 29
         });
30 30
 
31
-        Blade::if('approve', function ($menu) {
31
+        Blade::if ('approve', function($menu) {
32 32
             return auth()->user()->approve($menu);
33 33
         });
34 34
 
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
      */
53 53
     private function registerRoutes()
54 54
     {
55
-        Route::group($this->routeConfiguration(), function () {
55
+        Route::group($this->routeConfiguration(), function() {
56 56
             $this->loadRoutesFrom(__DIR__ . '/Http/routes.php');
57 57
         });
58 58
     }
@@ -104,11 +104,11 @@  discard block
 block discarded – undo
104 104
             ], 'menu-config');
105 105
 
106 106
             $this->publishes([
107
-                __DIR__.'/Storage/seeds' => database_path('seeds'),
107
+                __DIR__ . '/Storage/seeds' => database_path('seeds'),
108 108
             ], 'menu-seeder');
109 109
 
110 110
             $this->publishes([
111
-                __DIR__.'/../resources/views' => resource_path('views/menu-maker'),
111
+                __DIR__ . '/../resources/views' => resource_path('views/menu-maker'),
112 112
             ], 'menu-views');
113 113
         }
114 114
     }
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
      */
121 121
     public function register()
122 122
     {
123
-        $this->app->bind('userModel', function () {
123
+        $this->app->bind('userModel', function() {
124 124
             $class = '\\' . ltrim(config('auth.providers.users.model'), '\\');
125 125
 
126 126
             return new $class;
Please login to merge, or discard this patch.
Storage/migrations/2019_02_10_060024_create_pcmm_menu_role_pivot_table.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
      */
14 14
     public function up()
15 15
     {
16
-        Schema::create('pcmm_menu_role', function (Blueprint $table) {
16
+        Schema::create('pcmm_menu_role', function(Blueprint $table) {
17 17
             $table->integer('role_id')->unsigned();
18 18
             $table->integer('menu_id')->unsigned();
19 19
 
Please login to merge, or discard this patch.
src/Storage/migrations/2019_02_10_054600_create_pcmm_menus_table.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@
 block discarded – undo
15 15
      */
16 16
     public function up()
17 17
     {
18
-        Schema::create('pcmm_menus', function (Blueprint $table) {
18
+        Schema::create('pcmm_menus', function(Blueprint $table) {
19 19
             $table->increments('id');
20 20
             $table->nestedSet();
21 21
             $table->string('name', 100);
Please login to merge, or discard this patch.
src/Storage/migrations/2019_02_10_054420_create_pcmm_roles_table.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
      */
14 14
     public function up()
15 15
     {
16
-        Schema::create('pcmm_roles', function (Blueprint $table) {
16
+        Schema::create('pcmm_roles', function(Blueprint $table) {
17 17
             $table->increments('id');
18 18
             $table->string('name', 100);
19 19
             $table->boolean('is_active')->default(true);
Please login to merge, or discard this patch.
Storage/migrations/2019_02_10_054421_create_pcmm_role_user_pivot_table.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
      */
14 14
     public function up()
15 15
     {
16
-        Schema::create('pcmm_role_user', function (Blueprint $table) {
16
+        Schema::create('pcmm_role_user', function(Blueprint $table) {
17 17
             $table->integer('role_id')->unsigned();
18 18
             $table->integer('user_id')->unsigned();
19 19
 
Please login to merge, or discard this patch.
src/Storage/migrations/2019_02_10_060134_create_pcmm_permissions_table.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
      */
14 14
     public function up()
15 15
     {
16
-        Schema::create('pcmm_permissions', function (Blueprint $table) {
16
+        Schema::create('pcmm_permissions', function(Blueprint $table) {
17 17
             $table->increments('id');
18 18
             $table->integer('menu_id')->unsigned()->index();
19 19
             $table->string('namespace', 255)->index();
Please login to merge, or discard this patch.
src/Storage/factories/MenuFactory.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
 use Faker\Generator as Faker;
5 5
 use PhpCollective\MenuMaker\Storage\Menu;
6 6
 
7
-$factory->define(Menu::class, function (Faker $faker) {
7
+$factory->define(Menu::class, function(Faker $faker) {
8 8
     return [
9 9
         'name'   => $faker->word,
10 10
         'alias' => $faker->unique()->slug,
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
     ];
16 16
 });
17 17
 
18
-$factory->state(Menu::class, 'section', function ($faker) {
18
+$factory->state(Menu::class, 'section', function($faker) {
19 19
     return [
20 20
         'name'   => $faker->word,
21 21
         'alias' => $faker->unique()->slug,
Please login to merge, or discard this patch.
src/MenuMaker.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
     {
25 25
         $this->section = Menu::whereAlias($alias)->first();
26 26
 
27
-        if (! $this->section) {
27
+        if (!$this->section) {
28 28
             return collect([]);
29 29
         }
30 30
         return $this->getMenuItemsWithActive();
@@ -37,11 +37,11 @@  discard block
 block discarded – undo
37 37
         $currentPath = request()->path();
38 38
         $menus = $this->getMenuItems();
39 39
 
40
-        $traverse = function ($menus) use (&$traverse, $currentRoute, $currentPath) {
40
+        $traverse = function($menus) use (&$traverse, $currentRoute, $currentPath) {
41 41
             $checked = collect([]);
42 42
             foreach ($menus as $key => $menu) {
43 43
                 $menu['children'] = $traverse($menu['children'], $currentRoute, $currentPath);
44
-                if((is_array($menu['routes']) && in_array($currentRoute, $menu['routes']))
44
+                if ((is_array($menu['routes']) && in_array($currentRoute, $menu['routes']))
45 45
                     || $currentPath === $menu['link']
46 46
                     || $menu['children']->where('active', true)->count())
47 47
                 {
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
 
59 59
     protected function getMenuItems()
60 60
     {
61
-        return Cache::rememberForever('menus.section.' . optional($this->section)->id . '.user.' . $this->id, function () {
61
+        return Cache::rememberForever('menus.section.' . optional($this->section)->id . '.user.' . $this->id, function() {
62 62
             return $this->admin()
63 63
                 ? $this->getAdminMenuItems()
64 64
                 : $this->getUserMenuItems();
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
     private function getAdminMenuItems()
69 69
     {
70 70
         $this->section->load([
71
-            'descendants' => function ($query) {
71
+            'descendants' => function($query) {
72 72
                 $query->visible();
73 73
             }
74 74
         ]);
@@ -79,9 +79,9 @@  discard block
 block discarded – undo
79 79
     private function getUserMenuItems()
80 80
     {
81 81
         $this->section->load([
82
-            'descendants' => function ($query) {
82
+            'descendants' => function($query) {
83 83
                 $query->with([
84
-                    'ancestors' => function ($query) {
84
+                    'ancestors' => function($query) {
85 85
                         $query->descendantsOf($this->section);
86 86
                     }
87 87
                 ])->select('pcmm_menus.*')
@@ -90,9 +90,9 @@  discard block
 block discarded – undo
90 90
                     ->leftJoin('pcmm_role_user', 'pcmm_roles.id', '=', 'pcmm_role_user.role_id')
91 91
                     ->leftJoin($this->getTable(), 'pcmm_role_user.user_id', '=', $this->getTable() . '.id')
92 92
                     ->visible()
93
-                    ->where(function ($query) {
93
+                    ->where(function($query) {
94 94
                         $query->wherePrivilege('PUBLIC')
95
-                            ->orWhere(function ($query) {
95
+                            ->orWhere(function($query) {
96 96
                                 $query->where('pcmm_menus.privilege', 'PROTECTED')
97 97
                                     ->where('pcmm_roles.is_active', true)
98 98
                                     ->where($this->getTable() . '.id', $this->id);
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
             }
102 102
         ]);
103 103
 
104
-        $items = $this->section->descendants->reject(function ($item) {
104
+        $items = $this->section->descendants->reject(function($item) {
105 105
             return $item->ancestors->contains('privilege', 'PRIVATE')
106 106
                 || $item->ancestors->contains('visible', false);
107 107
         });
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
 
196 196
     public function approve($alias)
197 197
     {
198
-        return $this->admin() || $this->whereHas('roles.menus', function ($query) use ($alias) {
198
+        return $this->admin() || $this->whereHas('roles.menus', function($query) use ($alias) {
199 199
                 $query->whereAlias($alias);
200 200
             })->exists();
201 201
     }
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
203 203
     public function authorize($request)
204 204
     {
205 205
         $route = explode_route($request->route());
206
-        return $this->whereHas('roles.menus.permissions', function ($query) use ($route) {
206
+        return $this->whereHas('roles.menus.permissions', function($query) use ($route) {
207 207
             $query->where('namespace', $route['namespace'])
208 208
                 ->where('controller', $route['controller'])
209 209
                 ->where('method', $route['method'])
Please login to merge, or discard this patch.