diff --git a/bonus-modules/bonus-system/src/main/java/com/bonus/system/controller/SysPostController.java b/bonus-modules/bonus-system/src/main/java/com/bonus/system/controller/SysPostController.java index bfd3c68..2f056aa 100644 --- a/bonus-modules/bonus-system/src/main/java/com/bonus/system/controller/SysPostController.java +++ b/bonus-modules/bonus-system/src/main/java/com/bonus/system/controller/SysPostController.java @@ -96,8 +96,6 @@ public class SysPostController extends BaseController { @SysLog(title = "岗位管理", businessType = OperaType.INSERT, logType = 0, module = "系统管理->岗位管理", details = "新增岗位") public AjaxResult add(@Validated @RequestBody SysPost post) { try { - Long companyId = SecurityUtils.getLoginUser().getSysUser().getCompanyId(); - post.setCompanyId(companyId); if (!postService.checkPostNameUnique(post)) { return error("新增岗位'" + post.getPostName() + "'失败,岗位名称已存在"); } else if (!postService.checkPostCodeUnique(post)) { diff --git a/bonus-modules/bonus-system/src/main/java/com/bonus/system/mapper/SysPostMapper.java b/bonus-modules/bonus-system/src/main/java/com/bonus/system/mapper/SysPostMapper.java index ef966a8..8be623b 100644 --- a/bonus-modules/bonus-system/src/main/java/com/bonus/system/mapper/SysPostMapper.java +++ b/bonus-modules/bonus-system/src/main/java/com/bonus/system/mapper/SysPostMapper.java @@ -85,16 +85,16 @@ public interface SysPostMapper /** * 校验岗位名称 * - * @param postName 岗位名称 + * @param post * @return 结果 */ - public SysPost checkPostNameUnique(String postName); + public SysPost checkPostNameUnique(SysPost post); /** * 校验岗位编码 * - * @param postCode 岗位编码 + * @param post * @return 结果 */ - public SysPost checkPostCodeUnique(String postCode); + public SysPost checkPostCodeUnique(SysPost post); } diff --git a/bonus-modules/bonus-system/src/main/java/com/bonus/system/mapper/SysRoleMapper.java b/bonus-modules/bonus-system/src/main/java/com/bonus/system/mapper/SysRoleMapper.java index dfbfbeb..e9c39b9 100644 --- a/bonus-modules/bonus-system/src/main/java/com/bonus/system/mapper/SysRoleMapper.java +++ b/bonus-modules/bonus-system/src/main/java/com/bonus/system/mapper/SysRoleMapper.java @@ -60,18 +60,18 @@ public interface SysRoleMapper /** * 校验角色名称是否唯一 * - * @param roleName 角色名称 + * @param role 角色名称 * @return 角色信息 */ - public SysRole checkRoleNameUnique(String roleName); + public SysRole checkRoleNameUnique(SysRole role); /** * 校验角色权限是否唯一 * - * @param roleKey 角色权限 + * @param role 角色权限 * @return 角色信息 */ - public SysRole checkRoleKeyUnique(String roleKey); + public SysRole checkRoleKeyUnique(SysRole role); /** * 修改角色信息 diff --git a/bonus-modules/bonus-system/src/main/java/com/bonus/system/service/impl/SysDeptServiceImpl.java b/bonus-modules/bonus-system/src/main/java/com/bonus/system/service/impl/SysDeptServiceImpl.java index 138f1cd..5d0c714 100644 --- a/bonus-modules/bonus-system/src/main/java/com/bonus/system/service/impl/SysDeptServiceImpl.java +++ b/bonus-modules/bonus-system/src/main/java/com/bonus/system/service/impl/SysDeptServiceImpl.java @@ -460,8 +460,9 @@ public class SysDeptServiceImpl implements ISysDeptService private void createCompanyAdminRole(SysDept sysDept, SysUser user) { log.info("部门信息{},公司管理员用户{}",sysDept,user); SysRole role = new SysRole(); - role.setRoleKey("company_"+sysDept.getDeptId()+"_admin"); - role.setRoleName("company_"+sysDept.getDeptId()+"_管理员"); + role.setRoleKey("company_admin"); + role.setRoleName("公司管理员"); + role.setRemark("公司管理员"); role.setCompanyId(sysDept.getDeptId()); role.setRoleSort(0); role.setStatus("0"); diff --git a/bonus-modules/bonus-system/src/main/java/com/bonus/system/service/impl/SysPostServiceImpl.java b/bonus-modules/bonus-system/src/main/java/com/bonus/system/service/impl/SysPostServiceImpl.java index 707e841..513dd3e 100644 --- a/bonus-modules/bonus-system/src/main/java/com/bonus/system/service/impl/SysPostServiceImpl.java +++ b/bonus-modules/bonus-system/src/main/java/com/bonus/system/service/impl/SysPostServiceImpl.java @@ -94,8 +94,10 @@ public class SysPostServiceImpl implements ISysPostService @Override public boolean checkPostNameUnique(SysPost post) { + Long companyId = SecurityUtils.getLoginUser().getSysUser().getCompanyId(); + post.setCompanyId(companyId); Long postId = StringUtils.isNull(post.getPostId()) ? -1L : post.getPostId(); - SysPost info = postMapper.checkPostNameUnique(post.getPostName()); + SysPost info = postMapper.checkPostNameUnique(post); if (StringUtils.isNotNull(info) && info.getPostId().longValue() != postId.longValue()) { return UserConstants.NOT_UNIQUE; @@ -112,8 +114,10 @@ public class SysPostServiceImpl implements ISysPostService @Override public boolean checkPostCodeUnique(SysPost post) { + Long companyId = SecurityUtils.getLoginUser().getSysUser().getCompanyId(); + post.setCompanyId(companyId); Long postId = StringUtils.isNull(post.getPostId()) ? -1L : post.getPostId(); - SysPost info = postMapper.checkPostCodeUnique(post.getPostCode()); + SysPost info = postMapper.checkPostCodeUnique(post); if (StringUtils.isNotNull(info) && info.getPostId().longValue() != postId.longValue()) { return UserConstants.NOT_UNIQUE; diff --git a/bonus-modules/bonus-system/src/main/java/com/bonus/system/service/impl/SysRoleServiceImpl.java b/bonus-modules/bonus-system/src/main/java/com/bonus/system/service/impl/SysRoleServiceImpl.java index 5a9218e..e5a68af 100644 --- a/bonus-modules/bonus-system/src/main/java/com/bonus/system/service/impl/SysRoleServiceImpl.java +++ b/bonus-modules/bonus-system/src/main/java/com/bonus/system/service/impl/SysRoleServiceImpl.java @@ -167,8 +167,10 @@ public class SysRoleServiceImpl implements ISysRoleService @Override public boolean checkRoleNameUnique(SysRole role) { + Long companyId = SecurityUtils.getLoginUser().getSysUser().getCompanyId(); + role.setCompanyId(companyId); Long roleId = StringUtils.isNull(role.getRoleId()) ? -1L : role.getRoleId(); - SysRole info = roleMapper.checkRoleNameUnique(role.getRoleName()); + SysRole info = roleMapper.checkRoleNameUnique(role); if (StringUtils.isNotNull(info) && info.getRoleId().longValue() != roleId.longValue()) { return UserConstants.NOT_UNIQUE; @@ -185,8 +187,10 @@ public class SysRoleServiceImpl implements ISysRoleService @Override public boolean checkRoleKeyUnique(SysRole role) { + Long companyId = SecurityUtils.getLoginUser().getSysUser().getCompanyId(); + role.setCompanyId(companyId); Long roleId = StringUtils.isNull(role.getRoleId()) ? -1L : role.getRoleId(); - SysRole info = roleMapper.checkRoleKeyUnique(role.getRoleKey()); + SysRole info = roleMapper.checkRoleKeyUnique(role); if (StringUtils.isNotNull(info) && info.getRoleId().longValue() != roleId.longValue()) { return UserConstants.NOT_UNIQUE; diff --git a/bonus-modules/bonus-system/src/main/resources/mapper/system/SysPostMapper.xml b/bonus-modules/bonus-system/src/main/resources/mapper/system/SysPostMapper.xml index c8bfbed..6a4deea 100644 --- a/bonus-modules/bonus-system/src/main/resources/mapper/system/SysPostMapper.xml +++ b/bonus-modules/bonus-system/src/main/resources/mapper/system/SysPostMapper.xml @@ -71,7 +71,7 @@ PUBLIC "-//mybatis.org//DTD Mapper 3.0//EN" where u.user_name = #{userName} - where post_name=#{postName} @@ -80,7 +80,7 @@ PUBLIC "-//mybatis.org//DTD Mapper 3.0//EN" limit 1 - where post_code=#{postCode} diff --git a/bonus-modules/bonus-system/src/main/resources/mapper/system/SysRoleMapper.xml b/bonus-modules/bonus-system/src/main/resources/mapper/system/SysRoleMapper.xml index 3459b64..1a8ccc1 100644 --- a/bonus-modules/bonus-system/src/main/resources/mapper/system/SysRoleMapper.xml +++ b/bonus-modules/bonus-system/src/main/resources/mapper/system/SysRoleMapper.xml @@ -88,14 +88,22 @@ PUBLIC "-//mybatis.org//DTD Mapper 3.0//EN" WHERE r.del_flag = '0' and u.user_name = #{userName} - - where r.role_name=#{roleName} and r.del_flag = '0' limit 1 + where r.role_name=#{roleName} and r.del_flag = '0' + + AND company_id = #{companyId} + + limit 1 - - where r.role_key=#{roleKey} and r.del_flag = '0' limit 1 + where r.role_key=#{roleKey} and r.del_flag = '0' + + AND company_id = #{companyId} + + limit 1