refactored UserController and UserService for Adminpanel

maxih4 · Jan 8, 2024 · b94b013 · b94b013
1 parent e1c19ac
commit b94b013
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/main/java/com/example/kickevent/controller/UserController.java b/src/main/java/com/example/kickevent/controller/UserController.java
@@ -55,7 +55,8 @@ User updateUser(@RequestBody User newUser, @PathVariable Long id) {
         return userService.findById(id).map(User -> {
             User.setUserName(newUser.getUserName());
             User.setRoles(newUser.getRoles());
-            return userService.save(User);
+            return userService.update(User);
+
         }).orElseGet(() -> {
             newUser.setId(id);
             return userService.save(newUser);

diff --git a/src/main/java/com/example/kickevent/services/UserService.java b/src/main/java/com/example/kickevent/services/UserService.java
@@ -61,6 +61,11 @@ public User save(User user) {
 
     }
 
+    public User update(User user){
+        if (userRepository.findByUserName(user.getUserName()).isPresent() && !Objects.equals(userRepository.findByUserName(user.getUserName()).get().getId(), user.getId())) throw new UsernameAlreadyTakenException(user.getUserName());
+        return userRepository.save(user);
+    }
+
     @Transactional
     public User delete(User user) {
         user.setRoles(null);