作者 唐旭辉

bug 修复

@@ -38,7 +38,11 @@ func (c *RbacController) RoleAdd() { @@ -38,7 +38,11 @@ func (c *RbacController) RoleAdd() {
38 msg = protocol.BadRequestParam("1") 38 msg = protocol.BadRequestParam("1")
39 return 39 return
40 } 40 }
41 - 41 + n := []rune(param.Name)
  42 + if len(n) > 10 || len(n) <= 0 {
  43 + msg = protocol.BadRequestParam("10008")
  44 + return
  45 + }
42 r, err := serverbac.RoleAdd(param) 46 r, err := serverbac.RoleAdd(param)
43 msg = protocol.NewReturnResponse(r, err) 47 msg = protocol.NewReturnResponse(r, err)
44 return 48 return
@@ -63,6 +67,11 @@ func (c *RbacController) RoleUpdate() { @@ -63,6 +67,11 @@ func (c *RbacController) RoleUpdate() {
63 msg = protocol.BadRequestParam("1") 67 msg = protocol.BadRequestParam("1")
64 return 68 return
65 } 69 }
  70 + n := []rune(param.Name)
  71 + if len(n) > 10 || len(n) <= 0 {
  72 + msg = protocol.BadRequestParam("10008")
  73 + return
  74 + }
66 param.Types = models.ROLETYPES_ROLE 75 param.Types = models.ROLETYPES_ROLE
67 r, err := serverbac.RoleEdit(param) 76 r, err := serverbac.RoleEdit(param)
68 msg = protocol.NewReturnResponse(r, err) 77 msg = protocol.NewReturnResponse(r, err)
@@ -114,6 +123,11 @@ func (c *RbacController) RoleGroupAdd() { @@ -114,6 +123,11 @@ func (c *RbacController) RoleGroupAdd() {
114 msg = protocol.BadRequestParam("1") 123 msg = protocol.BadRequestParam("1")
115 return 124 return
116 } 125 }
  126 + n := []rune(param.Name)
  127 + if len(n) > 10 || len(n) <= 0 {
  128 + msg = protocol.BadRequestParam("10007")
  129 + return
  130 + }
117 param.Pid = 0 131 param.Pid = 0
118 param.Types = models.ROLETYPES_GROUP 132 param.Types = models.ROLETYPES_GROUP
119 roleinfo, err := serverbac.RoleAdd(param) 133 roleinfo, err := serverbac.RoleAdd(param)
@@ -140,6 +154,11 @@ func (c *RbacController) RoleGroupUpdate() { @@ -140,6 +154,11 @@ func (c *RbacController) RoleGroupUpdate() {
140 msg = protocol.BadRequestParam("1") 154 msg = protocol.BadRequestParam("1")
141 return 155 return
142 } 156 }
  157 + n := []rune(param.Name)
  158 + if len(n) > 10 || len(n) <= 0 {
  159 + msg = protocol.BadRequestParam("10007")
  160 + return
  161 + }
143 param.Pid = 0 162 param.Pid = 0
144 param.Types = models.ROLETYPES_GROUP 163 param.Types = models.ROLETYPES_GROUP
145 roleinfo, err := serverbac.RoleEdit(param) 164 roleinfo, err := serverbac.RoleEdit(param)
@@ -9,8 +9,10 @@ var errmessge ErrorMap = map[string]string{ @@ -9,8 +9,10 @@ var errmessge ErrorMap = map[string]string{
9 "10002": "请先删除该角色下的人员", 9 "10002": "请先删除该角色下的人员",
10 "10003": "无效角色", 10 "10003": "无效角色",
11 "10004": "角色已存在", 11 "10004": "角色已存在",
12 - "10005": "默认角色组不能修改", 12 + "10005": "默认角色组不能删除",
13 "10006": "默认角色不能修改", 13 "10006": "默认角色不能修改",
  14 + "10007": "角色组名称最多10个字符",
  15 + "10008": "角色名称最多10个字符",
14 //职位相关 16 //职位相关
15 "10011": "该职位已被使用无法删除", 17 "10011": "该职位已被使用无法删除",
16 "10012": "超过10级的职位限制,请重新选择", 18 "10012": "超过10级的职位限制,请重新选择",
@@ -102,11 +102,9 @@ func RoleEdit(param protocol.RequestRoleEdit) (*protocol.ResponseRoleInfo, error @@ -102,11 +102,9 @@ func RoleEdit(param protocol.RequestRoleEdit) (*protocol.ResponseRoleInfo, error
102 return nil, protocol.NewErrWithMessage("1", e) 102 return nil, protocol.NewErrWithMessage("1", e)
103 } 103 }
104 if ok := role.IsDefaultRole(); ok && role.Types == models.ROLETYPES_ROLE { 104 if ok := role.IsDefaultRole(); ok && role.Types == models.ROLETYPES_ROLE {
105 - return nil, protocol.NewErrWithMessage("10005")  
106 - }  
107 - if ok := role.IsDefaultRole(); ok && role.Types == models.ROLETYPES_GROUP {  
108 return nil, protocol.NewErrWithMessage("10006") 105 return nil, protocol.NewErrWithMessage("10006")
109 } 106 }
  107 +
110 if role.CompanyId != param.CompanyID { 108 if role.CompanyId != param.CompanyID {
111 e := fmt.Errorf("role.CompanyId(%d) != param.CompanyID(%d)", role.CompanyId, param.CompanyID) 109 e := fmt.Errorf("role.CompanyId(%d) != param.CompanyID(%d)", role.CompanyId, param.CompanyID)
112 log.Error(e.Error()) 110 log.Error(e.Error())