You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ca...@apache.org on 2006/09/01 22:27:54 UTC

svn commit: r439464 - /maven/shared/trunk/maven-user/maven-user-model/src/main/java/org/apache/maven/user/model/impl/DefaultUserManager.java

Author: carlos
Date: Fri Sep  1 13:27:54 2006
New Revision: 439464

URL: http://svn.apache.org/viewvc?rev=439464&view=rev
Log:
Fix previous password check error.
Submitted by: Lester Ecarma

Modified:
    maven/shared/trunk/maven-user/maven-user-model/src/main/java/org/apache/maven/user/model/impl/DefaultUserManager.java

Modified: maven/shared/trunk/maven-user/maven-user-model/src/main/java/org/apache/maven/user/model/impl/DefaultUserManager.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-user/maven-user-model/src/main/java/org/apache/maven/user/model/impl/DefaultUserManager.java?rev=439464&r1=439463&r2=439464&view=diff
==============================================================================
--- maven/shared/trunk/maven-user/maven-user-model/src/main/java/org/apache/maven/user/model/impl/DefaultUserManager.java (original)
+++ maven/shared/trunk/maven-user/maven-user-model/src/main/java/org/apache/maven/user/model/impl/DefaultUserManager.java Fri Sep  1 13:27:54 2006
@@ -159,7 +159,9 @@
         {
             int oldCount = Math.min( securityPolicy.getPreviousPasswordsCount() - 1, 
                                      user.getPreviousEncodedPasswords().size() );
-            List sublist = user.getPreviousEncodedPasswords().subList( 1, oldCount );
+            //modified sublist start index as the previous value results to nothing being added to the list. 
+            //TODO Please check and verify if it satisfies the objective.
+            List sublist = user.getPreviousEncodedPasswords().subList( 0, oldCount );
             previousPasswords.addAll( sublist );
         }
         user.setPreviousEncodedPasswords( previousPasswords );