From f9f8f90ac63d6ce3274410d3721b173f40db6e41 Mon Sep 17 00:00:00 2001 From: jinpengyong <jpy123456> Date: Fri, 25 Aug 2023 14:12:48 +0800 Subject: [PATCH] chore:过滤海城小时数据 --- 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 5b50ee5..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.getId(), 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("id���" + manageAccount.getId() + ";"); - logUtils.saveOperationForManage(request, content.toString()); - 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("id���" + existManageAccount.getId() + ";"); - logUtils.saveOperationForManage(request, content.toString()); - + 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("id���" + oldAccount.getId() + ";"); - //������������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()); + 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