From ac38f7b9099d96165e248209c9c8fd127cb22972 Mon Sep 17 00:00:00 2001 From: zendwang <wangzhenxian@idadt.com> Date: 星期四, 02 十二月 2021 18:34:49 +0800 Subject: [PATCH] fix 为Transactional注解设置rollback属性 --- ruoyi-system/src/main/java/com/ruoyi/system/service/impl/SysRoleServiceImpl.java | 24 +++++++++++++----------- 1 files changed, 13 insertions(+), 11 deletions(-) diff --git a/ruoyi-system/src/main/java/com/ruoyi/system/service/impl/SysRoleServiceImpl.java b/ruoyi-system/src/main/java/com/ruoyi/system/service/impl/SysRoleServiceImpl.java index 7dd14af..9096f23 100644 --- a/ruoyi-system/src/main/java/com/ruoyi/system/service/impl/SysRoleServiceImpl.java +++ b/ruoyi-system/src/main/java/com/ruoyi/system/service/impl/SysRoleServiceImpl.java @@ -141,9 +141,10 @@ @Override public String checkRoleNameUnique(SysRole role) { Long roleId = StringUtils.isNull(role.getRoleId()) ? -1L : role.getRoleId(); - SysRole info = getOne(new LambdaQueryWrapper<SysRole>() - .eq(SysRole::getRoleName, role.getRoleName()).last("limit 1")); - if (StringUtils.isNotNull(info) && info.getRoleId().longValue() != roleId.longValue()) { + long count = count(new LambdaQueryWrapper<SysRole>() + .eq(SysRole::getRoleName, role.getRoleName()) + .ne(SysRole::getRoleId, roleId)); + if (count > 0) { return UserConstants.NOT_UNIQUE; } return UserConstants.UNIQUE; @@ -158,9 +159,10 @@ @Override public String checkRoleKeyUnique(SysRole role) { Long roleId = StringUtils.isNull(role.getRoleId()) ? -1L : role.getRoleId(); - SysRole info = getOne(new LambdaQueryWrapper<SysRole>() - .eq(SysRole::getRoleKey, role.getRoleKey()).last("limit 1")); - if (StringUtils.isNotNull(info) && info.getRoleId().longValue() != roleId.longValue()) { + long count = count(new LambdaQueryWrapper<SysRole>() + .eq(SysRole::getRoleKey, role.getRoleKey()) + .ne(SysRole::getRoleId, roleId)); + if (count > 0) { return UserConstants.NOT_UNIQUE; } return UserConstants.UNIQUE; @@ -213,7 +215,7 @@ * @return 缁撴灉 */ @Override - @Transactional + @Transactional(rollbackFor = { Exception.class }) public int insertRole(SysRole role) { // 鏂板瑙掕壊淇℃伅 baseMapper.insert(role); @@ -227,7 +229,7 @@ * @return 缁撴灉 */ @Override - @Transactional + @Transactional(rollbackFor = { Exception.class }) public int updateRole(SysRole role) { // 淇敼瑙掕壊淇℃伅 baseMapper.updateById(role); @@ -254,7 +256,7 @@ * @return 缁撴灉 */ @Override - @Transactional + @Transactional(rollbackFor = { Exception.class }) public int authDataScope(SysRole role) { // 淇敼瑙掕壊淇℃伅 baseMapper.updateById(role); @@ -313,7 +315,7 @@ * @return 缁撴灉 */ @Override - @Transactional + @Transactional(rollbackFor = { Exception.class }) public int deleteRoleById(Long roleId) { // 鍒犻櫎瑙掕壊涓庤彍鍗曞叧鑱� roleMenuMapper.delete(new LambdaQueryWrapper<SysRoleMenu>().eq(SysRoleMenu::getRoleId, roleId)); @@ -329,7 +331,7 @@ * @return 缁撴灉 */ @Override - @Transactional + @Transactional(rollbackFor = { Exception.class }) public int deleteRoleByIds(Long[] roleIds) { for (Long roleId : roleIds) { checkRoleAllowed(new SysRole(roleId)); -- Gitblit v1.9.3