From c7dd4a195d8d893d92c49963447cdf6486844584 Mon Sep 17 00:00:00 2001
From: cjl <276999030@qq.com>
Date: Fri, 20 Oct 2023 09:45:53 +0800
Subject: [PATCH] Merge remote-tracking branch 'origin/wb' into cjl
---
screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java | 309 ++++++++++++++++++--------------------------------
1 files changed, 113 insertions(+), 196 deletions(-)
diff --git a/screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java b/screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java
index 6bbb7cf..f62a6f5 100644
--- a/screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java
+++ b/screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java
@@ -11,10 +11,9 @@
import com.moral.api.entity.ManageRole;
import com.moral.api.mapper.ManageAccountMapper;
import com.moral.api.mapper.ManageAccountRoleMapper;
-import com.moral.api.mapper.ManageMenuMapper;
import com.moral.api.mapper.ManageRoleMapper;
import com.moral.api.pojo.dto.account.*;
-import com.moral.api.pojo.dto.login.AccountInfoDTO;
+import com.moral.api.pojo.redisBean.AccountInfoDTO;
import com.moral.api.pojo.dto.login.LoginDTO;
import com.moral.api.pojo.form.account.AccountDeleteForm;
import com.moral.api.pojo.form.account.AccountInsertForm;
@@ -26,9 +25,11 @@
import com.baomidou.mybatisplus.extension.service.impl.ServiceImpl;
import com.moral.api.config.mybatis.wrapper.NullFilterWrapper;
import com.moral.api.service.ManageMenuService;
+import com.moral.api.util.CompareFieldUtils;
import com.moral.api.util.LogUtils;
import com.moral.constant.Constants;
import com.moral.constant.ResponseCodeEnum;
+import com.moral.pojo.CompareFieldResult;
import com.moral.util.*;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.context.properties.ConfigurationProperties;
@@ -40,6 +41,7 @@
import javax.servlet.http.HttpServletRequest;
import java.util.*;
+import java.util.function.Predicate;
/**
* <p>
@@ -50,7 +52,6 @@
* @since 2021-03-09
*/
@Service
-@ConfigurationProperties(prefix = "log-aspect")
public class ManageAccountServiceImpl extends ServiceImpl<ManageAccountMapper, ManageAccount> implements ManageAccountService {
@@ -62,14 +63,6 @@
ManageMenuService manageMenuService;
@Autowired
ManageAccountRoleMapper manageAccountRoleMapper;
- @Autowired
- LogUtils logUtils;
-
- Map<String, String> manageAccountFormMap;
-
- public void setManageAccountFormMap(Map<String, String> manageAccountFormMap) {
- this.manageAccountFormMap = manageAccountFormMap;
- }
/**
* @Description: ������������
@@ -89,21 +82,10 @@
//������������������
QueryWrapper<ManageAccount> wrapper = new QueryWrapper<>();
wrapper.eq("account", account);
- List<ManageAccount> manageAccounts = manageAccountMapper.selectList(wrapper);
- if (ObjectUtils.isEmpty(manageAccounts)) {
- loginDTO.setCode(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getCode());
- loginDTO.setMsg(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
- return loginDTO;
- }
- //������������������������
- ManageAccount manageAccount = null;
- for (ManageAccount value : manageAccounts) {
- if (Constants.NOT_DELETE.equals(value.getIsDelete()))
- manageAccount = value;
- }
-
+ wrapper.eq("is_delete", Constants.NOT_DELETE);
+ ManageAccount manageAccount = manageAccountMapper.selectOne(wrapper);
if (ObjectUtils.isEmpty(manageAccount)) {
- loginDTO.setCode(ResponseCodeEnum.ACCOUNT_IS_DELETE.getCode());
+ loginDTO.setCode(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getCode());
loginDTO.setMsg(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
return loginDTO;
}
@@ -118,7 +100,7 @@
List<ManageRole> roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
//������������
- List<ManageMenu> menus = null;
+ List<ManageMenu> menus = new ArrayList<>();
if (!ObjectUtils.isEmpty(roles)) {
menus = manageMenuService.getParentChildrenMenusByRoles(roles);
}
@@ -131,18 +113,16 @@
//������token ������������������
String token = TokenUtils.getToken(String.valueOf(manageAccount.getId()), accountInfoDTO);
-
//������������������
loginDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
loginDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
- loginDTO.setAccountInfoDTO(accountInfoDTO);
loginDTO.setToken(token);
//������������������
HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
StringBuilder content = new StringBuilder();
content.append(manageAccount.getUserName()).append("���������������������;");
- logUtils.saveLoginForManage(request, content.toString(), manageAccount, manageAccount.getUserName());
+ LogUtils.saveLoginForManage(request, content.toString(), manageAccount, manageAccount.getUserName());
return loginDTO;
}
@@ -175,7 +155,7 @@
AccountDTO accountDTO = new AccountDTO();
//������
ManageAccount manageAccount = accountInsertForm.formConvertEntity();
- List<String> roleIdsStr = accountInsertForm.getRoleIds();
+ List<Integer> roleIds = accountInsertForm.getRoleIds();
/*������������������������*/
ManageAccount existAccount = new ManageAccount();
@@ -193,38 +173,23 @@
//������
manageAccountMapper.insert(manageAccount);
- //������account_role
- /*String to Integer*/
- List<ManageRole> roles = null;
//������������������������������������������������������
- if (!ObjectUtils.isEmpty(roleIdsStr)) {
- List<Integer> roleIds = new ArrayList<>();
- roleIdsStr.forEach(str -> roleIds.add(Integer.parseInt(str)));
-
+ if (!ObjectUtils.isEmpty(roleIds)) {
Integer accountId = manageAccount.getId();
- roleIdsStr.forEach(
+ roleIds.forEach(
value -> {
ManageAccountRole manageAccountRole = new ManageAccountRole();
manageAccountRole.setAccountId(accountId);
- manageAccountRole.setRoleId(Integer.parseInt(value));
+ manageAccountRole.setRoleId(value);
manageAccountRoleMapper.insert(manageAccountRole);
}
);
- roles = manageRoleMapper.selectBatchIds(roleIds);
}
+ //������������������
+ insertLog(manageAccount);
//������������������
- accountDTO.setAccount(manageAccount);
- accountDTO.setRoles(roles);
accountDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
accountDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
-
- //������������������
- HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
- StringBuilder content = new StringBuilder();
- content.append("������������������").append(manageAccount.getUserName() + ";")
- .append("account���" + manageAccount.getAccount() + ";");
- logUtils.saveOperationForManage(request, content.toString(),Constants.INSERT_OPERATE_TYPE);
-
return accountDTO;
}
@@ -240,7 +205,7 @@
AccountQueryDTO accountQueryDTO = new AccountQueryDTO();
//������
- Integer pageCount = accountQueryForm.getPage();
+ Integer page = accountQueryForm.getPage();
Integer size = accountQueryForm.getSize();
String userName = accountQueryForm.getUserName();
String email = accountQueryForm.getEmail();
@@ -250,10 +215,10 @@
String order = accountQueryForm.getOrder();
String orderType = accountQueryForm.getOrderType();
Date createStartTime = accountQueryForm.getCreateStartTime();
- Date createEndTime = DateUtils.getDateOfDay(accountQueryForm.getCreateEndTime(), 1);
+ Date createEndTime = accountQueryForm.getCreateEndTime();
//������������������
- Page<ManageAccount> page = new Page<>(pageCount, size);
+ Page<ManageAccount> queryPage = new Page<>(page, size);
NullFilterWrapper<ManageAccount> wrapper = new NullFilterWrapper<>();
wrapper.like("user_name", userName);
@@ -275,20 +240,37 @@
wrapper.eq("is_delete", Constants.NOT_DELETE);
//������������
- Page resultPage = manageAccountMapper.selectPage(page, wrapper);
-
- //���������������������������
+ Page resultPage = manageAccountMapper.selectPage(queryPage, wrapper);
List<ManageAccount> accounts = resultPage.getRecords();
- List<AccountDTO> accountDTOS = new ArrayList<>();
- if (!ObjectUtils.isEmpty(accounts)) {
- for (ManageAccount manageAccount : accounts) {
- AccountDTO accountDTO = new AccountDTO();
- List<ManageRole> roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
- accountDTO.setRoles(roles);
- accountDTO.setAccount(manageAccount);
- accountDTOS.add(accountDTO);
- }
+ //������������������������������admin������admin���������������������
+ AccountInfoDTO accountInfoDTO = (AccountInfoDTO) TokenUtils.getUserInfo();
+ List<ManageRole> ownRoles = accountInfoDTO.getRoles();
+ boolean ownAdminRole = false;
+ for (ManageRole ownRole : ownRoles) {
+ String roleName = ownRole.getName();
+ if ("admin".equals(roleName))
+ ownAdminRole = true;
}
+ if (!ownAdminRole) {
+ accounts.removeIf(new Predicate<ManageAccount>() {
+ @Override
+ public boolean test(ManageAccount manageAccount) {
+ if (manageAccount.getAccount().equals("admin"))
+ return true;
+ return false;
+ }
+ });
+ }
+ //���������������������������
+ List<AccountDTO> accountDTOS = new ArrayList<>();
+ for (ManageAccount manageAccount : accounts) {
+ AccountDTO accountDTO = new AccountDTO();
+ List<ManageRole> roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
+ accountDTO.setRoles(roles);
+ accountDTO.setAccount(manageAccount);
+ accountDTOS.add(accountDTO);
+ }
+
//������������������
accountQueryDTO.setAccountDTOS(accountDTOS);
@@ -315,7 +297,7 @@
AccountDTO accountDTO = new AccountDTO();
//������
Integer accountId = accountDeleteForm.getAccountId();
- //������������������
+ //���������������������������������������������
ManageAccount manageAccount = new ManageAccount();
manageAccount.setIsDelete(Constants.NOT_DELETE);
manageAccount.setId(accountId);
@@ -327,6 +309,13 @@
accountDTO.setMsg(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
return accountDTO;
}
+ //���������������������������
+ AccountInfoDTO accountInfo = (AccountInfoDTO) TokenUtils.getUserInfo();
+ if(accountInfo.getAccount().getId().equals(accountId)){
+ accountDTO.setCode(ResponseCodeEnum.CANNOT_DELETE_ONESELF.getCode());
+ accountDTO.setMsg(ResponseCodeEnum.CANNOT_DELETE_ONESELF.getMsg());
+ return accountDTO;
+ }
//������������������
UpdateWrapper<ManageAccount> deleteAccountWrapper = new UpdateWrapper<>();
deleteAccountWrapper.eq("id", accountId);
@@ -336,18 +325,13 @@
UpdateWrapper<ManageAccountRole> deleteManageAccountRoleWrapper = new UpdateWrapper<>();
deleteManageAccountRoleWrapper.set("is_delete", Constants.DELETE).eq("account_id", manageAccount.getId());
manageAccountRoleMapper.update(null, deleteManageAccountRoleWrapper);
+ //������token
+ TokenUtils.destoryToken(accountId);
//������������������
accountDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
accountDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
- accountDTO.setAccount(existManageAccount);
-
//������������������
- HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
- StringBuilder content = new StringBuilder();
- content.append("������������������").append(existManageAccount.getUserName() + ";")
- .append("���������" + existManageAccount.getAccount() + ";");
- logUtils.saveOperationForManage(request, content.toString(),Constants.DELETE_OPERATE_TYPE);
-
+ deleteLog(existManageAccount);
return accountDTO;
}
@@ -364,144 +348,77 @@
AccountDTO accountDTO = new AccountDTO();
//������
ManageAccount manageAccount = accountUpdateForm.formConvertEntity();
- List<Integer> roleIds = accountUpdateForm.getRoleIds();
- //������������������������������������
- QueryWrapper<ManageAccount> exitWrapper = new QueryWrapper<>();
- ManageAccount exitManageAccount = new ManageAccount();
- exitManageAccount.setId(manageAccount.getId());
- exitManageAccount.setIsDelete(Constants.NOT_DELETE);
- exitWrapper.setEntity(exitManageAccount);
- exitManageAccount = manageAccountMapper.selectOne(exitWrapper);
- if (ObjectUtils.isEmpty(exitManageAccount)) {
+ //������������������������������������������
+ QueryWrapper<ManageAccount> oldAccountWrapper = new QueryWrapper<>();
+ ManageAccount oldManageAccount = new ManageAccount();
+ oldManageAccount.setId(manageAccount.getId());
+ oldManageAccount.setIsDelete(Constants.NOT_DELETE);
+ oldAccountWrapper.setEntity(oldManageAccount);
+ oldManageAccount = manageAccountMapper.selectOne(oldAccountWrapper);
+ if (ObjectUtils.isEmpty(oldManageAccount)) {
accountDTO.setCode(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getCode());
accountDTO.setMsg(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
return accountDTO;
}
//������ManageAccount���
- Map manageAccountMap = JSONObject.parseObject(JSON.toJSONString(manageAccount),Map.class);//���������Map���������������������������
- if (manageAccountMap.size()>2) {//������������account������������������������������������������
- manageAccountMapper.updateById(manageAccount);
- }
- //������������������������
- manageAccount = manageAccountMapper.selectById(manageAccount.getId());
- //������������������������������
- List<ManageRole> oldRoles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
-
- //���������������������������������ManageAccountRole���
- /*
- * ������roleIds���null������������������������������������������
- * ������roleIds���������������������������������������������������
- * ������roleIds���������������������������������������������
- * */
- List<ManageRole> roles = null;
- if (roleIds != null) {
- UpdateWrapper<ManageAccountRole> deleteWrapper = new UpdateWrapper<>();
- deleteWrapper.eq("account_id", manageAccount.getId()).set("is_delete", Constants.DELETE);
- manageAccountRoleMapper.update(null, deleteWrapper);
- /*������������������*/
- for (Integer roleId : roleIds) {
- ManageAccountRole manageAccountRole = new ManageAccountRole();
- manageAccountRole.setAccountId(manageAccount.getId());
- manageAccountRole.setRoleId(roleId);
- manageAccountRoleMapper.insert(manageAccountRole);
- }
- //������������������������
- if (roleIds.size() != 0)
- roles = manageRoleMapper.selectBatchIds(roleIds);
- } else {
- roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
- }
- accountDTO.setRoles(roles);
+ manageAccountMapper.updateById(manageAccount);
+ //������������������
+ updateLog(oldManageAccount,manageAccount);
+ //������token
+ TokenUtils.destoryToken(manageAccount.getId());
//������������������
accountDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
accountDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
- accountDTO.setAccount(manageAccount);
-
- //������������������
- insertUpdateLog(accountUpdateForm, exitManageAccount, oldRoles);
-
return accountDTO;
}
/**
- * @Description: ���������������������������
- * @Param: [form, newAccount, oldAccount]
- * @return: void
- * @Author: ���������
- * @Date: 2021/4/8
- */
- private void insertUpdateLog(AccountUpdateForm updateForm, ManageAccount oldAccount, List<ManageRole> oldRoles) {
- //������������������
- HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
+ * @Description: ������������������������
+ * @Param: [oldAccount, newAccount]
+ * @return: void
+ * @Author: ���������
+ * @Date: 2021/8/25
+ */
+ private void updateLog(ManageAccount oldAccount,ManageAccount newAccount){
+ List<CompareFieldResult> results = CompareFieldUtils.compare(ManageAccount.class, oldAccount, newAccount);
+ StringBuilder content = new StringBuilder("���������������������"+";");
+ content.append("���������"+oldAccount.getAccount()+";");
+ //������������������
+ if(!ObjectUtils.isEmpty(newAccount.getPassword()))
+ content.append("���������������");
+ String resultContent = CompareFieldUtils.resultsConvertContent(results,content.toString());
+ LogUtils.saveOperationForManage(resultContent, Constants.UPDATE_OPERATE_TYPE);
+ }
+
+ /**
+ * @Description: ������������������������
+ * @Param: [account]
+ * @return: void
+ * @Author: ���������
+ * @Date: 2021/8/25
+ */
+ private void insertLog(ManageAccount manageAccount){
StringBuilder content = new StringBuilder();
- content.append("������������������").append(oldAccount.getUserName() + ";")
- .append("���������" + oldAccount.getAccount() + ";");
- //������������Map,���������������������������������
- Map<String, Object> newParameters = JSONObject.parseObject(JSON.toJSONString(updateForm), Map.class);
- Map<String, Object> oldParameters = JSONObject.parseObject(JSON.toJSONString(oldAccount), Map.class);
- //������������������������Map���������������������������
- Set<String> keys = manageAccountFormMap.keySet();
- for (String key : keys) {
- String value = manageAccountFormMap.get(key);//���������������������
- if ("password".equals(key)) {//������������������������������������������
- if (!ObjectUtils.isEmpty(updateForm.getPassword())) {//���������������������������������
- content.append("���������������;");
- }
- } else if ("roleIds".equals(key)) {//������������������������Id���������������������
- List<Integer> newRoleIds = updateForm.getRoleIds();
-
- if (newRoleIds != null && 0 == newRoleIds.size()) {//���������������������������������������������������������������
- content.append("���������������������;");
- }
- if ((newRoleIds != null) && (newRoleIds.size() != 0)) {//���������������������������������������������null���������������������������
- StringBuilder oldRolesName = new StringBuilder("���");
- StringBuilder newRolesName = new StringBuilder("���");
-
- List<ManageRole> newRoles = null;
- if (!ObjectUtils.isEmpty(updateForm.getRoleIds())) {
- newRoles = manageRoleMapper.selectBatchIds(updateForm.getRoleIds());
- }
-
- if (!ObjectUtils.isEmpty(oldRoles)) {
- oldRolesName.deleteCharAt(oldRolesName.length()-1);//������ "���"
- oldRolesName.append("[");
- for (ManageRole role : oldRoles) {
- oldRolesName.append(role.getName() + ",");
- }
- oldRolesName.deleteCharAt(oldRolesName.length() - 1);//������������������������
- oldRolesName.append("]");
- }
-
- if (!ObjectUtils.isEmpty(newRoles)) {
- newRolesName.deleteCharAt(newRolesName.length()-1);//������ "���"
- newRolesName.append("[");
- for (ManageRole role : newRoles) {
- newRolesName.append(role.getName() + ",");
- }
- newRolesName.deleteCharAt(newRolesName.length() - 1);//������������������������
- newRolesName.append("]");
- }
- //������������content
- content.append(value+":"+oldRolesName + "->" + newRolesName + ";");
- }
- } else {//������������������
- if (newParameters.get(key) != null) {
- String newValue = "���";
- String oldValue = "���";
- if (newParameters.get(key) != null && !newParameters.get(key).equals(" ")) {
- newValue = String.valueOf(newParameters.get(key));
- }
- if (oldParameters.get(key) != null && !oldParameters.get(key).equals(" ")) {
- oldValue = String.valueOf(oldParameters.get(key));
- }
- content.append(value + ":" + oldValue + "->" + newValue + ";");
- }
- }
- }
- logUtils.saveOperationForManage(request, content.toString(), Constants.UPDATE_OPERATE_TYPE);
+ content.append("������������������������").append(manageAccount.getUserName() + ";")
+ .append("account���" + manageAccount.getAccount() + ";");
+ LogUtils.saveOperationForManage(content.toString(), Constants.INSERT_OPERATE_TYPE);
}
+ /**
+ * @Description: ������������������������
+ * @Param: [manageAccount]
+ * @return: void
+ * @Author: ���������
+ * @Date: 2021/8/25
+ */
+ private void deleteLog(ManageAccount manageAccount){
+ StringBuilder content = new StringBuilder();
+ content.append("������������������").append(manageAccount.getUserName() + ";")
+ .append("������:" + manageAccount.getAccount() + ";");
+ LogUtils.saveOperationForManage( content.toString(), Constants.DELETE_OPERATE_TYPE);
+ }
}
+
--
Gitblit v1.8.0