diff --git a/backend/src/main/java/org/raddatz/familienarchiv/controller/UserController.java b/backend/src/main/java/org/raddatz/familienarchiv/controller/UserController.java index b3753546..b52150f2 100644 --- a/backend/src/main/java/org/raddatz/familienarchiv/controller/UserController.java +++ b/backend/src/main/java/org/raddatz/familienarchiv/controller/UserController.java @@ -80,8 +80,7 @@ public class UserController { @RequirePermission(Permission.ADMIN_USER) public ResponseEntity createUser(Authentication authentication, @Valid @RequestBody CreateUserRequest request) { - AppUser actor = userService.findByEmail(authentication.getName()); - return ResponseEntity.ok(userService.createUserOrUpdate(actor.getId(), request)); + return ResponseEntity.ok(userService.createUserOrUpdate(actorId(authentication), request)); } @PutMapping("/users/{id}") @@ -89,8 +88,7 @@ public class UserController { public ResponseEntity adminUpdateUser(Authentication authentication, @PathVariable UUID id, @RequestBody AdminUpdateUserRequest dto) { - AppUser actor = userService.findByEmail(authentication.getName()); - AppUser updated = userService.adminUpdateUser(actor.getId(), id, dto); + AppUser updated = userService.adminUpdateUser(actorId(authentication), id, dto); updated.setPassword(null); return ResponseEntity.ok(updated); } @@ -99,9 +97,12 @@ public class UserController { @RequirePermission(Permission.ADMIN_USER) public ResponseEntity deleteUser(Authentication authentication, @PathVariable UUID id) { - AppUser actor = userService.findByEmail(authentication.getName()); - userService.deleteUser(actor.getId(), id); + userService.deleteUser(actorId(authentication), id); return ResponseEntity.ok().build(); } + private UUID actorId(Authentication auth) { + return userService.findByEmail(auth.getName()).getId(); + } + }