KAA-1293: Move email and user existence check methods from Web UI to …

…REST API method
kaaproject · Sep 14, 2016 · ffdac54 · ffdac54
1 parent 52f0884
commit ffdac54
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 18 deletions.
diff --git a/server/node/src/main/java/org/kaaproject/kaa/server/admin/services/UserServiceImpl.java b/server/node/src/main/java/org/kaaproject/kaa/server/admin/services/UserServiceImpl.java
@@ -29,12 +29,9 @@
 
 import java.util.ArrayList;
 import java.util.List;
-import java.util.Set;
 import java.util.stream.Collectors;
 
-import static org.kaaproject.kaa.server.admin.services.util.Utils.checkEmailUniquieness;
-import static org.kaaproject.kaa.server.admin.services.util.Utils.checkNotNull;
-import static org.kaaproject.kaa.server.admin.services.util.Utils.checkEmailFormat;
+import static org.kaaproject.kaa.server.admin.services.util.Utils.checkFieldUniquieness;
 import static org.kaaproject.kaa.server.admin.services.util.Utils.getCurrentUser;
 import static org.kaaproject.kaa.server.admin.shared.util.Utils.isEmpty;
 
@@ -107,8 +104,8 @@ public org.kaaproject.kaa.common.dto.admin.UserDto getUser(String userId) throws
     public org.kaaproject.kaa.common.dto.admin.UserDto editUser(org.kaaproject.kaa.common.dto.admin.UserDto user)
             throws KaaAdminServiceException {
         try {
-            User stored = userFacade.findByUserName(user.getUsername());
-            boolean createNewUser = (stored == null);
+
+            boolean createNewUser = (user.getId() == null);
 
             String tempPassword = null;
             if (createNewUser)  {
@@ -121,22 +118,22 @@ public org.kaaproject.kaa.common.dto.admin.UserDto editUser(org.kaaproject.kaa.c
                     }
                 }
 
-                checkEmailFormat(user.getMail());
-
-                checkEmailUniquieness(
+                checkFieldUniquieness(
                         user.getMail(),
                         userFacade.getAll().stream().map(u -> u.getMail()).collect(Collectors.toSet())
                 );
-
+
+                checkFieldUniquieness(
+                        user.getUsername(),
+                        userFacade.getAll().stream().map(u -> u.getUsername()).collect(Collectors.toSet())
+                );
+
                 CreateUserResult result = userFacade.saveUserDto(user, passwordEncoder);
                 user.setExternalUid(result.getUserId().toString());
                 tempPassword = result.getPassword();
             } else {
-                if (isEmpty(user.getId())) {
-                    controlService.getUsers().stream()
-                            .filter(u -> u.getUsername().equals(user.getUsername())).findFirst()
-                            .ifPresent(u -> user.setId(u.getId()));
-                }
+                User stored = userFacade.findByUserName(user.getUsername());
+
                 user.setExternalUid(String.valueOf(stored.getId()));
                 checkUserId(user.getId());
 

diff --git a/server/node/src/main/java/org/kaaproject/kaa/server/admin/services/util/Utils.java b/server/node/src/main/java/org/kaaproject/kaa/server/admin/services/util/Utils.java
@@ -77,11 +77,11 @@ public static <T> T checkNotNull(T reference) throws KaaAdminServiceException {
         return reference;
     }
 
-    public static void checkEmailUniquieness(String email, Set<String> storedEmails) throws KaaAdminServiceException {
+    public static void checkFieldUniquieness(String email, Set<String> storedEmails) throws KaaAdminServiceException {
         checkNotNull(email);
         boolean isAdded = storedEmails.add(email);
         if (!isAdded) {
-            throw new KaaAdminServiceException("Entered email is already used by another user!", ServiceErrorCode.INVALID_ARGUMENTS);
+            throw new KaaAdminServiceException("Entered property is already used by another user!", ServiceErrorCode.INVALID_ARGUMENTS);
         }
     }
 

diff --git a/server/node/src/main/java/org/kaaproject/kaa/server/admin/shared/services/UserService.java b/server/node/src/main/java/org/kaaproject/kaa/server/admin/shared/services/UserService.java
@@ -39,4 +39,4 @@ public interface UserService extends RemoteService {
     void deleteUser(String userId) throws KaaAdminServiceException;
 
     List<UserDto> findAllTenantAdminsByTenantId(String id) throws KaaAdminServiceException;
-}
+}