fix: 不返回root用户及root分组，并禁止修改root用户分组

Author: colorful3

src/main/java/io/github/talelin/merak/controller/cms/AdminController.java

@@ -63,7 +63,9 @@ public PageResponseVO getUsers(
             List<GroupDO> groups = groupService.getUserGroupsByUserId(user.getId());
             return new UserInfoVO(user, groups);
         }).collect(Collectors.toList());
-        return ResponseUtil.generatePageResult(iPage.getTotal(), userInfos, page, count);
+        boolean existedRoot = userInfos.removeIf(userInfo -> groupService.checkIsRootByUserId(userInfo.getId()));
+        long total = existedRoot ? iPage.getTotal() - 1 : iPage.getTotal();
+        return ResponseUtil.generatePageResult(total, userInfos, page, count);
     }
 
     @PutMapping("/user/{id}/password")

src/main/java/io/github/talelin/merak/service/GroupService.java

@@ -93,4 +93,6 @@ public interface GroupService extends IService<GroupDO> {
      * @return 用户id
      */
     List<Long> getGroupUserIds(Long id);
+
+    Long getRootGroupId();
 }

src/main/java/io/github/talelin/merak/service/impl/AdminServiceImpl.java

@@ -166,7 +166,9 @@ public boolean removePermissions(RemovePermissionsDTO dto) {
 
     @Override
     public List<GroupDO> getAllGroups() {
-        return groupService.list();
+        List<GroupDO> groups = groupService.list();
+        groups.removeIf(group -> group.getId().equals(groupService.getRootGroupId()));
+        return groups;
     }
 
     @Override

src/main/java/io/github/talelin/merak/service/impl/GroupServiceImpl.java

@@ -87,6 +87,9 @@ public boolean checkIsRootByUserId(Long userId) {
     public boolean deleteUserGroupRelations(Long userId, List<Long> deleteIds) {
         if (deleteIds == null || deleteIds.isEmpty())
             return true;
+        if (checkIsRootByUserId(userId)) {
+            throw new ForbiddenException("can't modify the root user's group", 10078);
+        }
         QueryWrapper<UserGroupDO> wrapper = new QueryWrapper<>();
         wrapper.lambda()
                 .eq(UserGroupDO::getUserId, userId)
@@ -114,6 +117,11 @@ public List<Long> getGroupUserIds(Long id) {
         return list.stream().map(UserGroupDO::getUserId).collect(Collectors.toList());
     }
 
+    @Override
+    public Long getRootGroupId() {
+        return rootGroupId;
+    }
+
     private boolean checkGroupExistByIds(List<Long> ids) {
         return ids.stream().allMatch(this::checkGroupExistById);
     }

src/main/resources/code.properties

@@ -46,6 +46,7 @@ lin.cms.code-message[10074]=root\u5206\u7EC4\u4E0D\u53EF\u5220\u9664
 lin.cms.code-message[10075]=guest\u5206\u7EC4\u4E0D\u53EF\u5220\u9664
 lin.cms.code-message[10076]=\u90AE\u7BB1\u5DF2\u88AB\u4F7F\u7528\uFF0C\u8BF7\u91CD\u65B0\u586B\u5165\u65B0\u7684\u90AE\u7BB1
 lin.cms.code-message[10077]=\u4E0D\u53EF\u5C06\u7528\u6237\u5206\u914D\u7ED9\u4E0D\u5B58\u5728\u7684\u5206\u7EC4
+lin.cms.code-message[10078]=\u4E0D\u53EF\u4FEE\u6539root\u7528\u6237\u7684\u5206\u7EC4
 # lin.cms.code-message[10080]=\u8BF7\u6C42\u65B9\u6CD5\u4E0D\u5141\u8BB8
 # lin.cms.code-message[10100]=\u5237\u65B0\u4EE4\u724C\u83B7\u53D6\u5931\u8D25
 # lin.cms.code-message[10110]=\u6587\u4EF6\u4F53\u79EF\u8FC7\u5927