From 7232a74d17635defd5e7e39fb7a8892f12fa3931 Mon Sep 17 00:00:00 2001
From: kaiyu <404897439@qq.com>
Date: Thu, 06 May 2021 17:50:21 +0800
Subject: [PATCH] screen-manage 优化后台账号增加，更新，删除逻辑
---
/dev/null | 58 -------------------
screen-common/src/main/java/com/moral/util/AESUtils.java | 3
screen-manage/src/main/java/com/moral/api/pojo/form/account/AccountInsertForm.java | 2
screen-manage/src/main/java/com/moral/api/controller/AccountController.java | 20 +-----
screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java | 41 +++----------
5 files changed, 16 insertions(+), 108 deletions(-)
diff --git a/screen-common/src/main/java/com/moral/util/AESUtils.java b/screen-common/src/main/java/com/moral/util/AESUtils.java
index 34f760b..b887ea9 100644
--- a/screen-common/src/main/java/com/moral/util/AESUtils.java
+++ b/screen-common/src/main/java/com/moral/util/AESUtils.java
@@ -81,7 +81,8 @@
}
public static void main(String[] args) {
- System.out.println(encrypt("chenkaiyu111","AD42F7787B035B7580000EF93BE20BAD"));
+ System.out.println(encrypt("123456","AD42F7787B035B7580000EF93BE20BAD"));
+ //123456 KoWjfDMZQhJMLlG1crBPqQ==
}
}
diff --git a/screen-manage/src/main/java/com/moral/api/controller/AccountController.java b/screen-manage/src/main/java/com/moral/api/controller/AccountController.java
index 0c39f9a..53620af 100644
--- a/screen-manage/src/main/java/com/moral/api/controller/AccountController.java
+++ b/screen-manage/src/main/java/com/moral/api/controller/AccountController.java
@@ -1,18 +1,14 @@
package com.moral.api.controller;
-import com.moral.api.entity.SysArea;
import com.moral.api.pojo.dto.account.*;
-import com.moral.api.pojo.dto.organization.OrganizationDTO;
import com.moral.api.pojo.form.account.AccountDeleteForm;
import com.moral.api.pojo.form.account.AccountInsertForm;
import com.moral.api.pojo.form.account.AccountQueryForm;
import com.moral.api.pojo.form.account.AccountUpdateForm;
import com.moral.api.pojo.vo.account.AccountDeleteVO;
-import com.moral.api.pojo.vo.account.AccountInsertVO;
import com.moral.api.pojo.vo.account.AccountQueryVO;
import com.moral.api.pojo.vo.account.AccountUpdateVO;
import com.moral.api.service.ManageAccountService;
-import com.moral.api.service.SysAreaService;
import com.moral.constant.ResponseCodeEnum;
import com.moral.constant.ResultMessage;
import io.swagger.annotations.*;
@@ -20,7 +16,6 @@
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.web.bind.annotation.*;
-import java.util.List;
@Slf4j
@Api(tags = {"������������������"})
@@ -47,10 +42,7 @@
//������������������
AccountDTO dto = accountService.insertAccount(form);
- //������������������������
- AccountInsertVO accountInsertVO = AccountInsertVO.convert(dto);
-
- return new ResultMessage(dto.getCode(), dto.getMsg(), accountInsertVO);
+ return new ResultMessage(dto.getCode(), dto.getMsg(), null);
}
@PostMapping("update")
@@ -69,10 +61,7 @@
//������������������
AccountDTO dto = accountService.updateAccount(form);
- //���������������������������
- AccountUpdateVO vo = AccountUpdateVO.convert(dto);
-
- return new ResultMessage(dto.getCode(), dto.getMsg(), vo);
+ return new ResultMessage(dto.getCode(), dto.getMsg(), null);
}
@PostMapping("delete")
@@ -85,10 +74,7 @@
//������������������
AccountDTO dto = accountService.deleteAccount(form);
- //���������������������������
- AccountDeleteVO vo = AccountDeleteVO.convert(dto);
-
- return new ResultMessage(dto.getCode(), dto.getMsg(), vo);
+ return new ResultMessage(dto.getCode(), dto.getMsg(), null);
}
@GetMapping("query")
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/form/account/AccountInsertForm.java b/screen-manage/src/main/java/com/moral/api/pojo/form/account/AccountInsertForm.java
index 08b8cf9..89c43fd 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/form/account/AccountInsertForm.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/form/account/AccountInsertForm.java
@@ -39,7 +39,7 @@
private String wechat;
- private List<String> roleIds;
+ private List<Integer> roleIds;
public boolean valid() {
if (
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountInsertVO.java b/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountInsertVO.java
deleted file mode 100644
index aa12050..0000000
--- a/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountInsertVO.java
+++ /dev/null
@@ -1,58 +0,0 @@
-package com.moral.api.pojo.vo.account;
-
-import com.fasterxml.jackson.annotation.JsonInclude;
-import com.moral.api.entity.ManageRole;
-import com.moral.api.pojo.dto.account.AccountDTO;
-import com.moral.constant.ResponseCodeEnum;
-import lombok.Data;
-import org.springframework.util.ObjectUtils;
-
-import java.util.ArrayList;
-import java.util.List;
-
-/**
- * @ClassName AccountVO
- * @Description TODO
- * @Author ���������
- * @Date 2021/3/13 17:25
- * @Version TODO
- **/
-@Data
-@JsonInclude(JsonInclude.Include.NON_EMPTY)
-public class AccountInsertVO extends AccountVO {
-
- /**
- * @Description: ������������������������������������
- * @Param: [dto]
- * @return: com.moral.api.pojo.vo.account.AccountVO
- * @Author: ���������
- * @Date: 2021/3/15
- */
-
- public static AccountInsertVO convert(AccountDTO dto) {
- if (dto.getCode() != ResponseCodeEnum.SUCCESS.getCode())
- return null;
-
- AccountInsertVO vo = new AccountInsertVO();
- Integer id = dto.getAccount().getId();
- String userName = dto.getAccount().getUserName();
- String email = dto.getAccount().getEmail();
- String mobile = dto.getAccount().getMobile();
- String wechat = dto.getAccount().getWechat();
- //������������������������
- List<ManageRole> roles = dto.getRoles();
- List<String> roleNames = new ArrayList<>();
- if (!ObjectUtils.isEmpty(roles)) {
- roles.forEach(role -> roleNames.add(role.getName()));
- }
- vo.setId(id);
- vo.setUserName(userName);
- vo.setEmail(email);
- vo.setMobile(mobile);
- vo.setWechat(wechat);
- vo.setRoleNames(roleNames);
-
- return vo;
- }
-
-}
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 5bfa013..f01c6b1 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
@@ -175,7 +175,7 @@
AccountDTO accountDTO = new AccountDTO();
//������
ManageAccount manageAccount = accountInsertForm.formConvertEntity();
- List<String> roleIdsStr = accountInsertForm.getRoleIds();
+ List<Integer> roleIds = accountInsertForm.getRoleIds();
/*������������������������*/
ManageAccount existAccount = new ManageAccount();
@@ -193,28 +193,19 @@
//������
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);
}
//������������������
- accountDTO.setAccount(manageAccount);
- accountDTO.setRoles(roles);
accountDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
accountDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
@@ -314,7 +305,7 @@
AccountDTO accountDTO = new AccountDTO();
//������
Integer accountId = accountDeleteForm.getAccountId();
- //������������������
+ //���������������������������������������������
ManageAccount manageAccount = new ManageAccount();
manageAccount.setIsDelete(Constants.NOT_DELETE);
manageAccount.setId(accountId);
@@ -338,13 +329,11 @@
//������������������
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() + ";");
+ .append("������:" + existManageAccount.getAccount() + ";");
logUtils.saveOperationForManage(request, content.toString(), Constants.DELETE_OPERATE_TYPE);
return accountDTO;
@@ -383,8 +372,7 @@
if (manageAccountMap.size() > 1) {//������������������������id���������������������������������
manageAccountMapper.updateById(manageAccount);
}
- //������������������������
- manageAccount = manageAccountMapper.selectById(manageAccount.getId());
+
//������������������������������
List<ManageRole> oldRoles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
@@ -394,8 +382,8 @@
* ������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);
@@ -406,21 +394,12 @@
manageAccountRole.setRoleId(roleId);
manageAccountRoleMapper.insert(manageAccountRole);
}
- //������������������������
- if (roleIds.size() != 0)
- roles = manageRoleMapper.selectBatchIds(roleIds);
- } else {
- roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
}
- accountDTO.setRoles(roles);
//������������������
accountDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
accountDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
- accountDTO.setAccount(manageAccount);
-
//������������������
insertUpdateLog(accountUpdateForm, oldManageAccount, oldRoles);
-
return accountDTO;
}
@@ -436,7 +415,7 @@
HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
StringBuilder content = new StringBuilder();
content.append("������������������").append(oldAccount.getUserName() + ";")
- .append("���������" + oldAccount.getAccount() + ";");
+ .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);
@@ -452,7 +431,7 @@
List<Integer> newRoleIds = updateForm.getRoleIds();
if (newRoleIds != null && 0 == newRoleIds.size()) {//���������������������������������������������������������������
- content.append("���������������������;");
+ content.append("������������������������������;");
}
if ((newRoleIds != null) && (newRoleIds.size() != 0)) {//���������������������������������������������null���������������������������
StringBuilder oldRolesName = new StringBuilder("���");
--
Gitblit v1.8.0