Merge pull request #54 from balajidharma/2.x-Changes

#53 - Fixed

Author: balajidharma

app/Http/Controllers/Admin/MenuItemController.php

@@ -61,15 +61,13 @@ public function create(Menu $menu)
      */
     public function store(StoreMenuItemRequest $request, Menu $menu)
     {
-        $menu->menuItems()->create($request->except(['roles']));
+        $item = $menu->menuItems()->create($request->except(['roles']));
 
         $roles = $request->roles ?? [];
-        if(!empty($roles)) {
-            $menu->assignRole($roles);
-        }
+        $item->assignRole(array_map('intval', $roles));
 
         return redirect()->route('admin.menu.item.index', $menu->id)
-            ->with('message', 'Menu created successfully.');
+            ->with('message', 'Menu Item created successfully.');
     }
 
     /**
@@ -96,7 +94,7 @@ public function update(UpdateMenuItemRequest $request, Menu $menu, MenuItem $ite
         $item->update($request->except(['roles']));
 
         $roles = $request->roles ?? [];
-        $item->syncRoles($roles);
+        $item->syncRoles(array_map('intval', $roles));
 
         return redirect()->route('admin.menu.item.index', $menu->id)
             ->with('message', 'Menu Item updated successfully.');