mmm-go / oppmg · 提交

转到一个项目

作者 tangxvhui
提交 fe3c7a633a848514f3ab0d58dc57603d15c43110 1 个父辈 161e8359

修复bug

内嵌 并排对比
正在显示 4 个修改的文件 包含 18 行增加 和 0 行删除
@@ -52,6 +52,10 @@ func (t *Role) ValidateTypes() bool { @@ -52,6 +52,10 @@ func (t *Role) ValidateTypes() bool {
52 return false 52 return false
53 } 53 }
54 54
  55 +func (t *Role) IsDefaultRole() bool {
  56 + return t.IsDefault == ROLE_DEFAULR
  57 +}
  58 +
55 func (t *Role) ValidatePid() (*Role, error) { 59 func (t *Role) ValidatePid() (*Role, error) {
56 if t.Pid == 0 { 60 if t.Pid == 0 {
57 return nil, nil 61 return nil, nil
@@ -9,6 +9,8 @@ var errmessge ErrorMap = map[string]string{ @@ -9,6 +9,8 @@ var errmessge ErrorMap = map[string]string{
9 "10002": "请先删除该角色下的人员", 9 "10002": "请先删除该角色下的人员",
10 "10003": "无效角色", 10 "10003": "无效角色",
11 "10004": "角色已存在", 11 "10004": "角色已存在",
  12 + "10005": "默认角色组不能修改",
  13 + "10006": "默认角色不能修改",
12 //职位相关 14 //职位相关
13 "10011": "该职位已被使用无法删除", 15 "10011": "该职位已被使用无法删除",
14 "10012": "超过10级的职位限制，请重新选择", 16 "10012": "超过10级的职位限制，请重新选择",
@@ -60,6 +60,9 @@ func RoleDelete(param protocol.RequestRoleDelete) error { @@ -60,6 +60,9 @@ func RoleDelete(param protocol.RequestRoleDelete) error {
60 log.Error(e.Error()) 60 log.Error(e.Error())
61 return protocol.NewErrWithMessage("1", e) 61 return protocol.NewErrWithMessage("1", e)
62 } 62 }
  63 + if ok := role.IsDefaultRole(); ok {
  64 + return protocol.NewErrWithMessage("10006")
  65 + }
63 if role.CompanyId != param.CompanyID { 66 if role.CompanyId != param.CompanyID {
64 e := fmt.Errorf("role.CompanyId(%d) != param.CompanyID(%d)", role.CompanyId, param.CompanyID) 67 e := fmt.Errorf("role.CompanyId(%d) != param.CompanyID(%d)", role.CompanyId, param.CompanyID)
65 log.Error(e.Error()) 68 log.Error(e.Error())
@@ -98,6 +101,12 @@ func RoleEdit(param protocol.RequestRoleEdit) (*protocol.ResponseRoleInfo, error @@ -98,6 +101,12 @@ func RoleEdit(param protocol.RequestRoleEdit) (*protocol.ResponseRoleInfo, error
98 log.Error(e.Error()) 101 log.Error(e.Error())
99 return nil, protocol.NewErrWithMessage("1", e) 102 return nil, protocol.NewErrWithMessage("1", e)
100 } 103 }
  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")
  109 + }
101 if role.CompanyId != param.CompanyID { 110 if role.CompanyId != param.CompanyID {
102 e := fmt.Errorf("role.CompanyId(%d) != param.CompanyID(%d)", role.CompanyId, param.CompanyID) 111 e := fmt.Errorf("role.CompanyId(%d) != param.CompanyID(%d)", role.CompanyId, param.CompanyID)
103 log.Error(e.Error()) 112 log.Error(e.Error())
@@ -142,6 +151,9 @@ func RoleGroupDelete(param protocol.RequestRoleDelete) error { @@ -142,6 +151,9 @@ func RoleGroupDelete(param protocol.RequestRoleDelete) error {
142 log.Error(e.Error()) 151 log.Error(e.Error())
143 return protocol.NewErrWithMessage("1", e) 152 return protocol.NewErrWithMessage("1", e)
144 } 153 }
  154 + if ok := roleinfo.IsDefaultRole(); ok {
  155 + return protocol.NewErrWithMessage("10005")
  156 + }
145 if roleinfo.CompanyId != param.CompanyID { 157 if roleinfo.CompanyId != param.CompanyID {
146 e := fmt.Errorf("roleinfo.CompanyId(%d) != param.CompanyID(%d)", roleinfo.CompanyId, param.CompanyID) 158 e := fmt.Errorf("roleinfo.CompanyId(%d) != param.CompanyID(%d)", roleinfo.CompanyId, param.CompanyID)
147 log.Error(e.Error()) 159 log.Error(e.Error())