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 );