Merge pull request #1231 from SliverHorn/fix

fixed: c.ShouldBindXxx 未处理错误 以及注释错误

Author: SliverHorn

server/api/v1/system/sys_api.go

@@ -121,18 +121,23 @@ func (s *SystemApiApi) GetApiList(c *gin.Context) {
 // @Router    /api/getApiById [post]
 func (s *SystemApiApi) GetApiById(c *gin.Context) {
 	var idInfo request.GetById
-	_ = c.ShouldBindJSON(&idInfo)
-	if err := utils.Verify(idInfo, utils.IdVerify); err != nil {
+	err := c.ShouldBindJSON(&idInfo)
+	if err != nil {
+		response.FailWithMessage(err.Error(), c)
+		return
+	}
+	err = utils.Verify(idInfo, utils.IdVerify)
+	if err != nil {
 		response.FailWithMessage(err.Error(), c)
 		return
 	}
 	api, err := apiService.GetApiById(idInfo.ID)
 	if err != nil {
 		global.GVA_LOG.Error("获取失败!", zap.Error(err))
 		response.FailWithMessage("获取失败", c)
-	} else {
-		response.OkWithDetailed(systemRes.SysAPIResponse{Api: api}, "获取成功", c)
+		return
 	}
+	response.OkWithDetailed(systemRes.SysAPIResponse{Api: api}, "获取成功", c)
 }
 
 // UpdateApi

server/api/v1/system/sys_menu.go

@@ -64,7 +64,11 @@ func (a *AuthorityMenuApi) GetBaseMenuTree(c *gin.Context) {
 // @Router    /menu/addMenuAuthority [post]
 func (a *AuthorityMenuApi) AddMenuAuthority(c *gin.Context) {
 	var authorityMenu systemReq.AddMenuAuthorityInfo
-	_ = c.ShouldBindJSON(&authorityMenu)
+	err := c.ShouldBindJSON(&authorityMenu)
+	if err != nil {
+		response.FailWithMessage(err.Error(), c)
+		return
+	}
 	if err := utils.Verify(authorityMenu, utils.AuthorityIdVerify); err != nil {
 		response.FailWithMessage(err.Error(), c)
 		return

server/api/v1/system/sys_user.go

@@ -178,7 +178,6 @@ func (b *BaseApi) ChangePassword(c *gin.Context) {
 	response.OkWithMessage("修改成功", c)
 }
 
-// GetUserList
 // GetUserList
 // @Tags      SysUser
 // @Summary   分页获取用户列表