From a4d06b1354d48d94c8e07d716863bcb7d3876a07 Mon Sep 17 00:00:00 2001
From: jinpengyong <jpy123456>
Date: Wed, 23 Jun 2021 15:18:40 +0800
Subject: [PATCH] Merge remote-tracking branch 'origin/dev' into dev_

---
 screen-api/src/main/java/com/moral/api/service/impl/UserGroupServiceImpl.java |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/screen-api/src/main/java/com/moral/api/service/impl/UserGroupServiceImpl.java b/screen-api/src/main/java/com/moral/api/service/impl/UserGroupServiceImpl.java
index 4515798..75a25f0 100644
--- a/screen-api/src/main/java/com/moral/api/service/impl/UserGroupServiceImpl.java
+++ b/screen-api/src/main/java/com/moral/api/service/impl/UserGroupServiceImpl.java
@@ -20,6 +20,7 @@
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
+import org.springframework.util.ObjectUtils;
 import org.springframework.web.context.request.RequestContextHolder;
 import org.springframework.web.context.request.ServletRequestAttributes;
 
@@ -62,7 +63,7 @@
         deleteWrapper.eq("user_id", userId);
         userGroupMapper.delete(deleteWrapper);
         List<String> groups = new ArrayList<>();
-        if (groupIds != null && !groupIds.isEmpty()) {
+        if (!ObjectUtils.isEmpty(groupIds)) {
             groupIds.forEach(groupId -> {
                 groups.add(groupMapper.selectById(groupId).getGroupName());
                 UserGroup userGroup = new UserGroup();
@@ -75,7 +76,7 @@
 
         //������
         String account = userMapper.selectById((Integer) parameters.get("userId")).getAccount();
-        String content = "������������" + account + "���������������" + groups.toString();
+        String content = "������������" + account + "���������������" + groups;
         operationLogUtils.insertLog(request, content, Constants.UPDATE_OPERATE_TYPE);
     }
 

--
Gitblit v1.8.0