cleanup in the registration process

eugenp · eugenp · commit ff38ebf0dd2f · 2015-01-04T13:39:09.000+02:00
diff --git a/spring-security-login-and-registration/src/main/java/org/baeldung/persistence/service/UserService.java b/spring-security-login-and-registration/src/main/java/org/baeldung/persistence/service/UserService.java
@@ -24,6 +24,8 @@ public class UserService implements IUserService {
     @Autowired
     private PasswordEncoder passwordEncoder;
 
+    // API
+
     @Override
     public User registerNewUserAccount(UserDto accountDto) throws EmailExistsException {
         if (emailExist(accountDto.getEmail())) {
diff --git a/spring-security-login-and-registration/src/main/java/org/baeldung/spring/SecSecurityConfig.java b/spring-security-login-and-registration/src/main/java/org/baeldung/spring/SecSecurityConfig.java
@@ -16,17 +16,13 @@
 public class SecSecurityConfig {
 
     @Autowired
-    UserDetailsService userDetailsService;
+    private UserDetailsService userDetailsService;
 
     public SecSecurityConfig() {
         super();
     }
 
-    @Bean
-    public BCryptPasswordEncoder encoder() {
-        BCryptPasswordEncoder encoder = new BCryptPasswordEncoder(11);
-        return encoder;
-    }
+    // beans
 
     @Bean
     public DaoAuthenticationProvider authProvider() {
@@ -37,8 +33,8 @@ public DaoAuthenticationProvider authProvider() {
     }
 
     @Bean
-    public PasswordEncoder passwordEncoder() {
-        return new BCryptPasswordEncoder();
+    public PasswordEncoder encoder() {
+        return new BCryptPasswordEncoder(11);
     }
 
 }

Original file line number	Diff line number	Diff line change
`@@ -16,17 +16,13 @@`
`16`	`16`	`public class SecSecurityConfig {`
`17`	`17`
`18`	`18`	`@Autowired`
`19`		`- UserDetailsService userDetailsService;`
	`19`	`+ private UserDetailsService userDetailsService;`
`20`	`20`
`21`	`21`	`public SecSecurityConfig() {`
`22`	`22`	`super();`
`23`	`23`	`}`
`24`	`24`
`25`		`- @Bean`
`26`		`- public BCryptPasswordEncoder encoder() {`
`27`		`- BCryptPasswordEncoder encoder = new BCryptPasswordEncoder(11);`
`28`		`- return encoder;`
`29`		`- }`
	`25`	`+ // beans`
`30`	`26`
`31`	`27`	`@Bean`
`32`	`28`	`public DaoAuthenticationProvider authProvider() {`
`@@ -37,8 +33,8 @@ public DaoAuthenticationProvider authProvider() {`
`37`	`33`	`}`
`38`	`34`
`39`	`35`	`@Bean`
`40`		`- public PasswordEncoder passwordEncoder() {`
`41`		`- return new BCryptPasswordEncoder();`
	`36`	`+ public PasswordEncoder encoder() {`
	`37`	`+ return new BCryptPasswordEncoder(11);`
`42`	`38`	`}`
`43`	`39`
`44`	`40`	`}`