You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2010/09/27 15:04:43 UTC
svn commit: r1001722 -
/directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/name/DN.java
Author: elecharny
Date: Mon Sep 27 13:04:42 2010
New Revision: 1001722
URL: http://svn.apache.org/viewvc?rev=1001722&view=rev
Log:
Fixed a bug in the way we normalized DN after having modified it.
Modified:
directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/name/DN.java
Modified: directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/name/DN.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/name/DN.java?rev=1001722&r1=1001721&r2=1001722&view=diff
==============================================================================
--- directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/name/DN.java (original)
+++ directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/name/DN.java Mon Sep 27 13:04:42 2010
@@ -1202,6 +1202,7 @@ public class DN implements Cloneable, Se
DN clonedDn = clone();
clonedDn.rdns.add( 0, newRdn.clone() );
+ clonedDn.normalized.getAndSet( false );
// FIXME this try-catch block shouldn't be here
// instead this method should throw the LdapInvalidDnException
@@ -1559,13 +1560,12 @@ public class DN implements Cloneable, Se
*/
public DN normalize( Map<String, OidNormalizer> oidsMap ) throws LdapInvalidDnException
{
-
if ( ( oidsMap == null ) || ( oidsMap.isEmpty() ) )
{
return this;
}
- if( normalized.get() )
+ if ( normalized.get() )
{
return this;
}