diff --git a/Yi.Framework/Yi.Framework.ApiMicroservice/Controllers/MenuController.cs b/Yi.Framework/Yi.Framework.ApiMicroservice/Controllers/MenuController.cs
index bfdc2523..c44f1d02 100644
--- a/Yi.Framework/Yi.Framework.ApiMicroservice/Controllers/MenuController.cs
+++ b/Yi.Framework/Yi.Framework.ApiMicroservice/Controllers/MenuController.cs
@@ -100,7 +100,12 @@ namespace Yi.Framework.ApiMicroservice.Controllers
         /// <returns></returns>
         [HttpPost]
         public async Task<Result> AddChildrenMenu(ChildrenDto<menu> childrenDto)
-        { 
+        {
+            var _menu = await _menuService.GetEntityById(childrenDto.parentId);
+          var _children= await _menuService.AddChildrenMenu(_menu);
+            _children = childrenDto.data;
+            return Result.Success();
+
         }
     }
 }
diff --git a/Yi.Framework/Yi.Framework.ApiMicroservice/Controllers/UserController.cs b/Yi.Framework/Yi.Framework.ApiMicroservice/Controllers/UserController.cs
index a1d38072..7dfeda30 100644
--- a/Yi.Framework/Yi.Framework.ApiMicroservice/Controllers/UserController.cs
+++ b/Yi.Framework/Yi.Framework.ApiMicroservice/Controllers/UserController.cs
@@ -92,8 +92,7 @@ namespace Yi.Framework.ApiMicroservice.Controllers
         [HttpPost]
         public async Task<Result> SetRoleByUser(IdsListDto<int> idsListDto)
         {
-            有bug：无法运行，500报错;
-            var _user = this.HttpContext.GetCurrentUserInfo();
+          
           await  _userService.SetRolesByUser(idsListDto.ids2, idsListDto.ids1);
             return Result.Success();
         }
diff --git a/Yi.Framework/Yi.Framework.ApiMicroservice/YIDB.db b/Yi.Framework/Yi.Framework.ApiMicroservice/YIDB.db
index 8932385e..b2c233d8 100644
Binary files a/Yi.Framework/Yi.Framework.ApiMicroservice/YIDB.db and b/Yi.Framework/Yi.Framework.ApiMicroservice/YIDB.db differ
diff --git a/Yi.Framework/Yi.Framework.Interface/IMenuService.cs b/Yi.Framework/Yi.Framework.Interface/IMenuService.cs
index a39e9b6e..fc480933 100644
--- a/Yi.Framework/Yi.Framework.Interface/IMenuService.cs
+++ b/Yi.Framework/Yi.Framework.Interface/IMenuService.cs
@@ -45,6 +45,6 @@ namespace Yi.Framework.Interface
         /// </summary>
         /// <param name="_menu"></param>
         /// <returns></returns>
-        Task<bool> AddChildrenMenu(menu _menu);
+        Task<menu> AddChildrenMenu(menu _menu);
     }
 }
diff --git a/Yi.Framework/Yi.Framework.Service/MenuService.cs b/Yi.Framework/Yi.Framework.Service/MenuService.cs
index c0bf6d9d..001f32d9 100644
--- a/Yi.Framework/Yi.Framework.Service/MenuService.cs
+++ b/Yi.Framework/Yi.Framework.Service/MenuService.cs
@@ -13,15 +13,14 @@ namespace Yi.Framework.Service
     {
         public MenuService(DbContext Db) : base(Db) { }
 
-        public async Task<bool> AddChildrenMenu(menu _menu)
+        public async Task<menu> AddChildrenMenu(menu _menu)
         {
-            var menu_data = await GetEntity(u=>u.id==_menu.id&& u.is_delete == (short)Common.Enum.DelFlagEnum.Normal);
-            if (menu_data == null)
-            {
-                return false;
-            }         
-            menu_data.children.Add(new menu()) ;
-            return await AddAsync(_menu);
+            var menu_data = await _Db.Set<menu>().Include(u=>u.children)
+                .Where(u=>u.id==_menu.id&& u.is_delete == (short)Common.Enum.DelFlagEnum.Normal).FirstOrDefaultAsync();
+            var _children = menu_data.children.ToList();
+            _children.Add(new menu());
+            await UpdateAsync(menu_data);
+            return menu_data;
         }
 
         public async Task<bool> DelListByUpdateAsync(List<int> _ids)
diff --git a/Yi.Framework/Yi.Framework.Service/UserService.cs b/Yi.Framework/Yi.Framework.Service/UserService.cs
index b4448d3e..004001dd 100644
--- a/Yi.Framework/Yi.Framework.Service/UserService.cs
+++ b/Yi.Framework/Yi.Framework.Service/UserService.cs
@@ -92,7 +92,7 @@ namespace Yi.Framework.Service
             {
                 return false;
             }           
-            var roleList = _Db.Set<role>().Where(u => roleIds.Contains(u.id)).ToList();
+            var roleList =await _Db.Set<role>().Where(u => roleIds.Contains(u.id) && u.is_delete == (short)Common.Enum.DelFlagEnum.Normal).ToListAsync();
             foreach(var item in user_data)
             {
                 item.roles = roleList;