registration logic simplifications

Author: eugenp

spring-security-login-and-registration/src/main/java/org/baeldung/persistence/service/UserService.java

@@ -12,14 +12,16 @@
 import org.springframework.stereotype.Service;
 
 @Service
+@Transactional
 public class UserService implements IUserService {
     @Autowired
     private UserRepository repository;
 
     @Autowired
     private VerificationTokenRepository tokenRepository;
 
-    @Transactional
+    // API
+
     @Override
     public User registerNewUserAccount(UserDto accountDto) throws EmailExistsException {
         if (emailExist(accountDto.getEmail())) {
@@ -34,14 +36,6 @@ public User registerNewUserAccount(UserDto accountDto) throws EmailExistsExcepti
         return repository.save(user);
     }
 
-    private boolean emailExist(String email) {
-        User user = repository.findByEmail(email);
-        if (user != null) {
-            return true;
-        }
-        return false;
-    }
-
     @Override
     public User getUser(String verificationToken) {
         User user = tokenRepository.findByToken(verificationToken).getUser();
@@ -53,23 +47,30 @@ public VerificationToken getVerificationToken(String VerificationToken) {
         return tokenRepository.findByToken(VerificationToken);
     }
 
-    @Transactional
     @Override
     public void saveRegisteredUser(User user) {
         repository.save(user);
     }
 
-    @Transactional
     @Override
     public void deleteUser(User user) {
         repository.delete(user);
     }
 
-    @Transactional
     @Override
     public void createVerificationTokenForUser(User user, String token) {
         VerificationToken myToken = new VerificationToken(token, user);
         tokenRepository.save(myToken);
     }
-    
+
+    //
+
+    private boolean emailExist(String email) {
+        User user = repository.findByEmail(email);
+        if (user != null) {
+            return true;
+        }
+        return false;
+    }
+
 }

spring-security-login-and-registration/src/main/java/org/baeldung/security/MyUserDetailsService.java

@@ -47,12 +47,8 @@ public UserDetails loadUserByUsername(String email) throws UsernameNotFoundExcep
             if (user == null) {
                 return new org.springframework.security.core.userdetails.User(" ", " ", enabled, true, true, true, getAuthorities(new Integer(1)));
             }
-            if (!user.isEnabled()) {
-                enabled = false;
-                return new org.springframework.security.core.userdetails.User(" ", " ", enabled, true, true, true, getAuthorities(new Integer(1)));
-            }
-            return new org.springframework.security.core.userdetails.User(user.getEmail(), user.getPassword().toLowerCase(), enabled, accountNonExpired, credentialsNonExpired, accountNonLocked, getAuthorities(user.getRole().getRole()));
-
+            
+            return new org.springframework.security.core.userdetails.User(user.getEmail(), user.getPassword().toLowerCase(), user.isEnabled(), accountNonExpired, credentialsNonExpired, accountNonLocked, getAuthorities(user.getRole().getRole()));
         } catch (Exception e) {
             throw new RuntimeException(e);
         }