From 891a929e1fceb8602275866683437fa0911872bf Mon Sep 17 00:00:00 2001
From: "B.E.N.S.O.N" <bensonhkbus@gmail.com>
Date: Tue, 12 May 2026 18:04:29 +0800
Subject: [PATCH] User Page Update

---
 .../ffii/fpsms/modules/user/service/GroupService.java  | 10 ++++++++++
 .../ffii/fpsms/modules/user/service/UserService.java   |  8 ++++----
 .../ffii/fpsms/modules/user/web/GroupController.java   | 10 +++++++++-
 .../ffii/fpsms/modules/user/web/UserController.java    |  7 +------
 4 files changed, 24 insertions(+), 11 deletions(-)

diff --git a/src/main/java/com/ffii/fpsms/modules/user/service/GroupService.java b/src/main/java/com/ffii/fpsms/modules/user/service/GroupService.java
index 2d41d9e..79a4f5c 100644
--- a/src/main/java/com/ffii/fpsms/modules/user/service/GroupService.java
+++ b/src/main/java/com/ffii/fpsms/modules/user/service/GroupService.java
@@ -1,6 +1,7 @@
 package com.ffii.fpsms.modules.user.service;
 
 import java.util.Date;
+import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.stream.Collectors;
@@ -209,4 +210,13 @@ public class GroupService extends AbstractBaseEntityService<Group, Long, GroupRe
 		return jdbcDao.queryForList(sql.toString(), args);
 	}
 
+	@Transactional(rollbackFor = Exception.class)
+	public Map<Integer, List<Map<String, Object>>> listAuthForUsers(List<Integer> userIds) {
+		Map<Integer, List<Map<String, Object>>> result = new LinkedHashMap<>();
+		for (Integer userId : userIds) {
+			result.put(userId, listAuth(Map.of("userId", userId)));
+		}
+		return result;
+	}
+
 }
diff --git a/src/main/java/com/ffii/fpsms/modules/user/service/UserService.java b/src/main/java/com/ffii/fpsms/modules/user/service/UserService.java
index fa9e7df..9c57e79 100644
--- a/src/main/java/com/ffii/fpsms/modules/user/service/UserService.java
+++ b/src/main/java/com/ffii/fpsms/modules/user/service/UserService.java
@@ -185,8 +185,8 @@ public class UserService extends AbstractBaseEntityService<User, Long, UserRepos
         if (!authBatchDeleteValues.isEmpty()) {
             jdbcDao.batchUpdate(
                     "DELETE FROM user_authority"
-                            + " WHERE userId = :userId ",
-//							+ "AND authId = :authId",
+                            + " WHERE userId = :userId "
+                            + " AND authId = :authId",
                     authBatchDeleteValues);
         }
         if (!authBatchInsertValues.isEmpty()) {
@@ -228,8 +228,8 @@ public class UserService extends AbstractBaseEntityService<User, Long, UserRepos
         if (!authBatchDeleteValues.isEmpty()) {
             jdbcDao.batchUpdate(
                     "DELETE FROM user_authority"
-                            + " WHERE userId = :userId ",
-//							+ "AND authId = :authId",
+                            + " WHERE userId = :userId "
+                            + " AND authId = :authId",
                     authBatchDeleteValues);
         }
         if (!authBatchInsertValues.isEmpty()) {
diff --git a/src/main/java/com/ffii/fpsms/modules/user/web/GroupController.java b/src/main/java/com/ffii/fpsms/modules/user/web/GroupController.java
index aee7d7c..d200d85 100644
--- a/src/main/java/com/ffii/fpsms/modules/user/web/GroupController.java
+++ b/src/main/java/com/ffii/fpsms/modules/user/web/GroupController.java
@@ -1,6 +1,7 @@
 package com.ffii.fpsms.modules.user.web;
 
 import java.util.HashMap;
+import java.util.List;
 import java.util.Map;
 
 import org.apache.commons.logging.Log;
@@ -11,6 +12,7 @@ import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.PostMapping;
+import org.springframework.web.bind.annotation.RequestParam;
 import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.ResponseStatus;
@@ -80,7 +82,6 @@ public class GroupController{
 
 	@GetMapping("/auth/{target}/{id}")
 	public RecordsRes<Map<String, Object>> authComboJson(HttpServletRequest request, @PathVariable("id") int id, @PathVariable("target") String target) throws ServletRequestBindingException {
-		System.out.println(request);
 		Map<String, Object> args = new HashMap<>();
 		if (id != 0){
 			if (target.equals("group")){
@@ -94,4 +95,11 @@ public class GroupController{
 		return new RecordsRes<>(groupService.listAuth(args));
 	}
 
+	@GetMapping("/auth/user-batch")
+	public Map<Integer, List<Map<String, Object>>> authBatchByUserIds(
+			@RequestParam("userIds") List<Integer> userIds
+	) {
+		return groupService.listAuthForUsers(userIds);
+	}
+
 }
diff --git a/src/main/java/com/ffii/fpsms/modules/user/web/UserController.java b/src/main/java/com/ffii/fpsms/modules/user/web/UserController.java
index af4abda..c464cda 100644
--- a/src/main/java/com/ffii/fpsms/modules/user/web/UserController.java
+++ b/src/main/java/com/ffii/fpsms/modules/user/web/UserController.java
@@ -78,7 +78,6 @@ public class UserController{
 	@GetMapping
 //	@PreAuthorize("hasAuthority('VIEW_USER')")
 	public ResponseEntity<List<UserRecord>> list(@ModelAttribute @Valid SearchUserReq req) {
-        logger.info("Test List user");
         return ResponseEntity.ok(userService.search(req));
 	}
 
@@ -120,13 +119,10 @@ public class UserController{
 	@GetMapping("/{id}")
 	@PreAuthorize("hasAuthority('VIEW_USER')")
 	public LoadUserRes load(@PathVariable long id) {
-		LoadUserRes test = new LoadUserRes(
+		return new LoadUserRes(
 											userService.find(id).orElseThrow(NotFoundException::new),
 											userService.listUserAuthId(id),
 											userService.listUserGroupId(id));
-											logger.info("Test List user2");
-				logger.info(test);
-		return test;
 	}
 	@GetMapping("/user-info/{id}")
 //	@PreAuthorize("hasAuthority('VIEW_USER')")
@@ -147,7 +143,6 @@ public class UserController{
 //	@ResponseStatus(HttpStatus.CREATED)
 //	@PreAuthorize("hasAuthority('MAINTAIN_USER')")
 	public IdRes newRecord(@RequestBody @Valid NewUserReq req) throws UnsupportedEncodingException {
-		System.out.println(req.getUsername());
 		return new IdRes(userService.newRecord(req).getId());
 	}