bugfix: Angemeldeten Benutzer aktualisieren

Thomas Ziemer
1 parent f8f21bfa
Showing 3 changed files with 29 additions and 3 deletions
src/main/java/net/ziemers/swxercise/lg/user/service/SessionContext.java
src/main/java/net/ziemers/swxercise/lg/user/service/UserService.java
src/main/java/net/ziemers/swxercise/ui/UserViewController.java
@@ -86,6 +86,19 @@ public class SessionContext implements java.io.Serializable {
     }
     /**
+     * Aktualisiert das Benutzerobjekt dieses Session-Kontexts.
+     *
+     * @return <code>true</code>, wenn die Aktualisierung erfolgreich durchgeführt werden konnte.
+     */
+    public boolean refresh(final User user) {
+        if (getUser() != null) {
+            setUser(user);
+            return true;
+        }
+        return false;
+    }
+
+    /**
      * Meldet einen Benutzer von diesem Session-Kontext ab.
      *
      * @return <code>true</code>, wenn die Abmeldung erfolgreich durchgeführt werden konnte.
@@ -127,7 +127,15 @@ public class UserService {
             final UserDtoToEntityContext ctx = ctxService.createContext(dto);
             mapper.map(ctx);
-            return dao.saveOrUpdate(ctx.user) != null;
+
+            if (dao.saveOrUpdate(ctx.user) != null) {
+                // handelt es sich um das Benutzerobjekt des angemeldeten Benutzers?
+                if (dto.getUsername().equals(ctx.user.getProfile().getUsername())) {
+                    // aktualisiert das Benutzerobjekt des Session-Kontexts
+                    return sessionContext.refresh(ctx.user);
+                }
+                return true;
+            }
         }
         return false;
     }
@@ -140,7 +148,11 @@ public class UserService {
      */
     public boolean updateUser(final UserDto dto) {
         final User user = sessionContext.getUser();
-        return updateUser(user.getId(), dto);
+        if (updateUser(user.getId(), dto)) {
+            // aktualisiert das Benutzerobjekt des Session-Kontexts
+            return sessionContext.refresh(dao.findById(user.getId()));
+        }
+        return false;
     }
     /**
@@ -101,7 +101,8 @@ public class UserViewController {
     public User getUser() {
         logger.info("Trying to get user's profile info.");
-        return userService.findUser(); }
+        return userService.findUser();
+    }
     /**
      * Erstellt ein neues User-Objekt mit den gewünschten Eigenschaften, welche mittels {@link UserDto} definiert werden.