refactor: updateUser mit Kontext und Mapper re-implementiert

Thomas Ziemer
1 parent 493ca844
Showing 7 changed files with 21 additions and 6 deletions
src/main/java/net/ziemers/swxercise/lg/model/user/Profile.java
src/main/java/net/ziemers/swxercise/lg/model/user/User.java
src/main/java/net/ziemers/swxercise/lg/user/dto/UserDto.java
src/main/java/net/ziemers/swxercise/lg/user/service/SessionContext.java
src/main/java/net/ziemers/swxercise/lg/user/service/UserDtoToEntityContextService.java
src/main/java/net/ziemers/swxercise/lg/user/service/UserDtoToEntityMapper.java
src/main/java/net/ziemers/swxercise/lg/user/service/UserService.java
@@ -145,7 +145,7 @@ public class Profile extends BaseEntity {
         return passwordHash;
     }
-    private void setPassword(String password) {
+    public void setPassword(String password) {
         // das Klartextkennwort wird niemals gespeichert!
         this.passwordHash = cryptString(password);
     }
@@ -18,7 +18,6 @@ public class User extends BaseEntity {
     private String lastname;
-    @NotNull
     private Profile profile;
     private Address address;
@@ -14,7 +14,7 @@ public class UserDto {
     private String username;    // aus dem Profile
     @NotNull
-    private String password;    // aus dem Profile
+    private String password = "";    // aus dem Profile
     private String firstname;
@@ -35,4 +35,5 @@ public class SessionContext implements java.io.Serializable {
         }
         return false;
     }
+
 }
@@ -23,7 +23,7 @@ public class UserDtoToEntityContextService {
     public UserDtoToEntityContext createContext(final UserDto dto) {
         final UserDtoToEntityContext ctx = new UserDtoToEntityContext();
-        // das UserDTO in den Kontext füllen
+        // das übergebene UserDto in den Kontext füllen
         ctx.dto = dto;
         // einen neuen oder einen bereits existierenden Benutzer in den Kontext füllen
@@ -35,6 +35,9 @@ public class UserDtoToEntityContextService {
             // wir füllen das User-Objekt mit Method Chaining
             ctx.user = new User(dto.getFirstname(), dto.getLastname())
                     .withProfile(ctx.profile);
+        } else {
+            ctx.profile = ctx.user.getProfile();
+            ctx.address = ctx.user.getAddress();
         }
         return ctx;
     }
@@ -17,11 +17,20 @@ public class UserDtoToEntityMapper {
      * @return den Kontext.
      */
     public UserDtoToEntityContext map(UserDtoToEntityContext ctx) {
+        // User-Objekt mappen
         ctx.user.setFirstname(ctx.dto.getFirstname());
         ctx.user.setLastname(ctx.dto.getLastname());
+
+        // Profile-Objekt mappen, falls gegeben
         if (ctx.profile != null) {
             ctx.user.setProfile(ctx.profile);
+            if(ctx.dto.getPassword().length() > 0) {
+                ctx.profile.setPassword(ctx.dto.getPassword());
+            }
+            ctx.profile.setMailaddress(ctx.dto.getMailaddress());
         }
+
+        // Address-Objekt mappen, falls gegeben
         if (ctx.address != null) {
             ctx.user.setAddress(ctx.address);
         }
@@ -123,8 +123,11 @@ public class UserService {
         // ist zurzeit ein Benutzer angemeldet, können wir ihn aktualisieren
         final User user = sessionContext.getUser();
         if (user != null) {
-            // TODO noch zu implementieren
-            return false;
+            // der Benutzername darf sich beim Aktualisieren nicht mehr ändern!
+            dto.withUsername(user.getProfile().getUsername());
+            final UserDtoToEntityContext ctx = ctxService.createContext(dto);
+            mapper.map(ctx);
+            return dao.saveOrUpdate(ctx.user) != null;
         }
         return false;
     }