directory-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From elecha...@apache.org
Subject svn commit: r980948 [1/3] - in /directory/apacheds/trunk: all/ core-annotations/ core-api/ core-api/src/main/java/org/apache/directory/server/core/ core-api/src/main/java/org/apache/directory/server/core/interceptor/context/ core-api/src/main/java/org/...
Date Fri, 30 Jul 2010 20:45:00 GMT
Author: elecharny
Date: Fri Jul 30 20:44:57 2010
New Revision: 980948

URL: http://svn.apache.org/viewvc?rev=980948&view=rev
Log:
o Merged the modifications made by Kiran in the dnFactoryExperient branch.
o DN is now immutable.
o We have a DN cache too.

Added:
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/DNFactory.java
      - copied unchanged from r980934, directory/apacheds/branches/apacheds-dnfactory-experiment/core/src/main/java/org/apache/directory/server/core/DNFactory.java
Modified:
    directory/apacheds/trunk/all/   (props changed)
    directory/apacheds/trunk/core/   (props changed)
    directory/apacheds/trunk/core-annotations/   (props changed)
    directory/apacheds/trunk/core-api/   (props changed)
    directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapCoreSessionConnection.java
    directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapPrincipal.java
    directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/interceptor/context/MoveAndRenameOperationContext.java
    directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/interceptor/context/MoveOperationContext.java
    directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/AttributeTypeSynchronizer.java
    directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/ComparatorSynchronizer.java
    directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/MatchingRuleSynchronizer.java
    directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/NormalizerSynchronizer.java
    directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/ObjectClassSynchronizer.java
    directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SchemaSynchronizer.java
    directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SyntaxCheckerSynchronizer.java
    directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SyntaxSynchronizer.java
    directory/apacheds/trunk/core-avl/   (props changed)
    directory/apacheds/trunk/core-constants/   (props changed)
    directory/apacheds/trunk/core-integ/   (props changed)
    directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/authz/MoveRenameAuthorizationIT.java
    directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/authz/SearchAuthorizationIT.java
    directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/collective/CollectiveAttributeServiceIT.java   (props changed)
    directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/exception/ExceptionServiceIT.java
    directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaAttributeTypeHandlerIT.java
    directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaComparatorHandlerIT.java
    directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaMatchingRuleHandlerIT.java
    directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaNormalizerHandlerIT.java
    directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaObjectClassHandlerIT.java
    directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaSyntaxCheckerHandlerIT.java
    directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaSyntaxHandlerIT.java
    directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/ObjectClassCreateIT.java
    directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/trigger/BackupUtilitiesSP.java
    directory/apacheds/trunk/core-jndi/   (props changed)
    directory/apacheds/trunk/core-jndi/src/main/java/org/apache/directory/server/core/jndi/ServerContext.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/DefaultDirectoryService.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/DefaultOperationManager.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/ReferralManagerImpl.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/authn/PasswordPolicyConfiguration.java   (props changed)
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/authz/AciAuthorizationInterceptor.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/authz/DefaultAuthorizationInterceptor.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/authz/GroupCache.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/authz/TupleCache.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/collective/CollectiveAttributeInterceptor.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/exception/ExceptionInterceptor.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/operational/OperationalAttributeInterceptor.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/partition/DefaultPartitionNexus.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/prefs/PreferencesUtils.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/prefs/ServerSystemPreferences.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/referral/ReferralInterceptor.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/schema/DefaultSchemaService.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/schema/SchemaInterceptor.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/schema/SchemaSubentryModifier.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/security/CoreKeyStoreSpi.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/sp/LdapClassLoader.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/sp/StoredProcExecutionManager.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/subtree/SubentryInterceptor.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/subtree/SubtreeEvaluator.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/trigger/AbstractStoredProcedureParameterInjector.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/trigger/AddStoredProcedureParameterInjector.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/trigger/DeleteStoredProcedureParameterInjector.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/trigger/ModifyDNStoredProcedureParameterInjector.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/trigger/ModifyStoredProcedureParameterInjector.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/trigger/SimpleTriggerExecutionAuthorizer.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/trigger/TriggerInterceptor.java
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/trigger/TriggerSpecCache.java
    directory/apacheds/trunk/core/src/test/java/org/apache/directory/server/core/authz/support/MaxImmSubFilterTest.java
    directory/apacheds/trunk/core/src/test/java/org/apache/directory/server/core/authz/support/MaxValueCountFilterTest.java
    directory/apacheds/trunk/core/src/test/java/org/apache/directory/server/core/authz/support/RelatedProtectedItemFilterTest.java
    directory/apacheds/trunk/core/src/test/java/org/apache/directory/server/core/authz/support/RelatedUserClassFilterTest.java
    directory/apacheds/trunk/core/src/test/java/org/apache/directory/server/core/authz/support/RestrictedByFilterTest.java
    directory/apacheds/trunk/core/src/test/java/org/apache/directory/server/core/changelog/MemoryChangeLogStoreTest.java
    directory/apacheds/trunk/core/src/test/java/org/apache/directory/server/core/interceptor/InterceptorChainTest.java
    directory/apacheds/trunk/core/src/test/java/org/apache/directory/server/core/schema/SchemaCheckerTest.java
    directory/apacheds/trunk/core/src/test/java/org/apache/directory/server/core/subtree/SubtreeEvaluatorTest.java

Propchange: directory/apacheds/trunk/all/
------------------------------------------------------------------------------
--- svn:mergeinfo (added)
+++ svn:mergeinfo Fri Jul 30 20:44:57 2010
@@ -0,0 +1,2 @@
+/directory/apacheds/branches/apacheds-dnfactory-experiment/all:980138-980938
+/directory/apacheds/trunk/all:980025-980137

Propchange: directory/apacheds/trunk/core/
------------------------------------------------------------------------------
--- svn:mergeinfo (added)
+++ svn:mergeinfo Fri Jul 30 20:44:57 2010
@@ -0,0 +1 @@
+/directory/apacheds/branches/apacheds-dnfactory-experiment/core:980138-980934

Propchange: directory/apacheds/trunk/core-annotations/
------------------------------------------------------------------------------
--- svn:mergeinfo (added)
+++ svn:mergeinfo Fri Jul 30 20:44:57 2010
@@ -0,0 +1,2 @@
+/directory/apacheds/branches/apacheds-dnfactory-experiment/core-annotations:980138-980934
+/directory/apacheds/trunk/core-annotations:980025-980137

Propchange: directory/apacheds/trunk/core-api/
------------------------------------------------------------------------------
--- svn:mergeinfo (added)
+++ svn:mergeinfo Fri Jul 30 20:44:57 2010
@@ -0,0 +1 @@
+/directory/apacheds/branches/apacheds-dnfactory-experiment/core-api:980138-980934

Modified: directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapCoreSessionConnection.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapCoreSessionConnection.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapCoreSessionConnection.java (original)
+++ directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapCoreSessionConnection.java Fri Jul 30 20:44:57 2010
@@ -174,6 +174,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -198,6 +199,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -235,6 +237,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -257,6 +260,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -279,6 +283,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -319,6 +324,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -351,6 +357,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -373,6 +380,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -468,6 +476,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             // do nothing
             LOG.warn( e.getMessage(), e );
         }
@@ -500,6 +509,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             // do nothing
             LOG.warn( e.getMessage(), e );
         }
@@ -537,6 +547,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -574,6 +585,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -605,6 +617,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -633,6 +646,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -655,6 +669,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -686,6 +701,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -708,6 +724,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -795,6 +812,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             resp.getLdapResult().setResultCode( ResultCodeEnum.getResultCode( e ) );
             resp.getLdapResult().setErrorMessage( e.getMessage() );
         }
@@ -847,6 +865,7 @@ public class LdapCoreSessionConnection i
         }
         catch ( Exception e )
         {
+            //e.printStackTrace();
             LOG.warn( e.getMessage(), e );
         }
 

Modified: directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapPrincipal.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapPrincipal.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapPrincipal.java (original)
+++ directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapPrincipal.java Fri Jul 30 20:44:57 2010
@@ -129,7 +129,7 @@ public final class LdapPrincipal impleme
      */
     public DN getDN()
     {
-        return ( DN ) dn.clone();
+        return dn;
     }
 
 

Modified: directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/interceptor/context/MoveAndRenameOperationContext.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/interceptor/context/MoveAndRenameOperationContext.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/interceptor/context/MoveAndRenameOperationContext.java (original)
+++ directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/interceptor/context/MoveAndRenameOperationContext.java Fri Jul 30 20:44:57 2010
@@ -61,7 +61,7 @@ public class MoveAndRenameOperationConte
     {
         super( session, oldDn, newRdn, delOldRdn );
         this.newSuperiorDn = newSuperiorDn;
-        newDn = ((DN)newSuperiorDn.clone()).add( newRdn );
+        newDn = newSuperiorDn.add( newRdn );
     }
 
 
@@ -85,7 +85,7 @@ public class MoveAndRenameOperationConte
             throwReferral();
         }
         
-        newDn = ((DN)newSuperiorDn.clone()).add(newRdn);
+        newDn = newSuperiorDn.add(newRdn);
         
         try
         {

Modified: directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/interceptor/context/MoveOperationContext.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/interceptor/context/MoveOperationContext.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/interceptor/context/MoveOperationContext.java (original)
+++ directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/interceptor/context/MoveOperationContext.java Fri Jul 30 20:44:57 2010
@@ -68,7 +68,7 @@ public class MoveOperationContext extend
         this.newSuperior = newSuperior;
         oldSuperior = oldDn.getParent();
         rdn = ( RDN )(oldDn.getRdn().clone());
-        newDn = ((DN)(newSuperior.clone())).add( rdn );
+        newDn = newSuperior.add( rdn );
     }
 
     
@@ -100,7 +100,7 @@ public class MoveOperationContext extend
 
         oldSuperior = modifyDnRequest.getName().getParent();
         rdn = ( RDN )(modifyDnRequest.getName().getRdn().clone());
-        newDn = ((DN)(newSuperior.clone())).add( rdn );
+        newDn = newSuperior.add( rdn );
     }
 
 

Modified: directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/AttributeTypeSynchronizer.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/AttributeTypeSynchronizer.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/AttributeTypeSynchronizer.java (original)
+++ directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/AttributeTypeSynchronizer.java Fri Jul 30 20:44:57 2010
@@ -68,8 +68,8 @@ public class AttributeTypeSynchronizer e
     public void add( Entry entry ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=attributetypes,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.ATTRIBUTE_TYPE );
@@ -144,8 +144,8 @@ public class AttributeTypeSynchronizer e
     public void delete( Entry entry, boolean cascade ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=attributetypes,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.ATTRIBUTE_TYPE );
@@ -207,8 +207,8 @@ public class AttributeTypeSynchronizer e
 
         // Inject the new DN
         DN newDn = new DN( targetEntry.getDn() );
-        newDn.remove( newDn.size() - 1 );
-        newDn.add( newRdn );
+        newDn = newDn.remove( newDn.size() - 1 );
+        newDn = newDn.add( newRdn );
         targetEntry.setDn( newDn );
 
         AttributeType at = factory.getAttributeType( schemaManager, targetEntry, schemaManager.getRegistries(),

Modified: directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/ComparatorSynchronizer.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/ComparatorSynchronizer.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/ComparatorSynchronizer.java (original)
+++ directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/ComparatorSynchronizer.java Fri Jul 30 20:44:57 2010
@@ -100,8 +100,8 @@ public class ComparatorSynchronizer exte
     public void add( Entry entry ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=comparators,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.COMPARATOR );
@@ -148,8 +148,8 @@ public class ComparatorSynchronizer exte
     public void delete( Entry entry, boolean cascade ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=comparators,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.COMPARATOR );
@@ -248,8 +248,8 @@ public class ComparatorSynchronizer exte
 
             // Inject the new DN
             DN newDn = new DN( targetEntry.getDn() );
-            newDn.remove( newDn.size() - 1 );
-            newDn.add( newRdn );
+            newDn = newDn.remove( newDn.size() - 1 );
+            newDn = newDn.add( newRdn );
             targetEntry.setDn( newDn );
 
             // Register the new comparator, and unregister the old one

Modified: directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/MatchingRuleSynchronizer.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/MatchingRuleSynchronizer.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/MatchingRuleSynchronizer.java (original)
+++ directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/MatchingRuleSynchronizer.java Fri Jul 30 20:44:57 2010
@@ -97,8 +97,8 @@ public class MatchingRuleSynchronizer ex
     public void add( Entry entry ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=matchingrules,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.MATCHING_RULE );
@@ -145,8 +145,8 @@ public class MatchingRuleSynchronizer ex
     public void delete( Entry entry, boolean cascade ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=matchingrules,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.MATCHING_RULE );

Modified: directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/NormalizerSynchronizer.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/NormalizerSynchronizer.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/NormalizerSynchronizer.java (original)
+++ directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/NormalizerSynchronizer.java Fri Jul 30 20:44:57 2010
@@ -99,8 +99,8 @@ public class NormalizerSynchronizer exte
     public void add( Entry entry ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=normalizers,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.NORMALIZER );
@@ -157,8 +157,8 @@ public class NormalizerSynchronizer exte
     public void delete( Entry entry, boolean cascade ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=normalizers,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.NORMALIZER );
@@ -220,8 +220,8 @@ public class NormalizerSynchronizer exte
 
             // Inject the new DN
             DN newDn = new DN( targetEntry.getDn() );
-            newDn.remove( newDn.size() - 1 );
-            newDn.add( newRdn );
+            newDn = newDn.remove( newDn.size() - 1 );
+            newDn = newDn.add( newRdn );
             targetEntry.setDn( newDn );
 
             Normalizer normalizer = factory.getNormalizer( schemaManager, targetEntry, schemaManager.getRegistries(),

Modified: directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/ObjectClassSynchronizer.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/ObjectClassSynchronizer.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/ObjectClassSynchronizer.java (original)
+++ directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/ObjectClassSynchronizer.java Fri Jul 30 20:44:57 2010
@@ -93,8 +93,8 @@ public class ObjectClassSynchronizer ext
     public void add( Entry entry ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=objectclasses,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.OBJECT_CLASS );
@@ -142,8 +142,8 @@ public class ObjectClassSynchronizer ext
     public void delete( Entry entry, boolean cascade ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=objectclasses,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.OBJECT_CLASS );
@@ -214,8 +214,8 @@ public class ObjectClassSynchronizer ext
 
         // Inject the new DN
         DN newDn = new DN( targetEntry.getDn() );
-        newDn.remove( newDn.size() - 1 );
-        newDn.add( newRdn );
+        newDn = newDn.remove( newDn.size() - 1 );
+        newDn = newDn.add( newRdn );
 
         checkOidIsUnique( newOid );
         ObjectClass oc = factory.getObjectClass( schemaManager, targetEntry, schemaManager.getRegistries(), schemaName );

Modified: directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SchemaSynchronizer.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SchemaSynchronizer.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SchemaSynchronizer.java (original)
+++ directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SchemaSynchronizer.java Fri Jul 30 20:44:57 2010
@@ -160,8 +160,8 @@ public class SchemaSynchronizer implemen
     public void add( Entry entry ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
         parentDn.normalize( registries.getAttributeTypeRegistry().getNormalizerMapping() );
 
         if ( !parentDn.equals( ouSchemaDN ) )

Modified: directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SyntaxCheckerSynchronizer.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SyntaxCheckerSynchronizer.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SyntaxCheckerSynchronizer.java (original)
+++ directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SyntaxCheckerSynchronizer.java Fri Jul 30 20:44:57 2010
@@ -98,8 +98,8 @@ public class SyntaxCheckerSynchronizer e
     public void add( Entry entry ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=syntaxcheckers,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.SYNTAX_CHECKER );
@@ -145,8 +145,8 @@ public class SyntaxCheckerSynchronizer e
     public void delete( Entry entry, boolean cascade ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=syntaxcheckers,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.SYNTAX_CHECKER );

Modified: directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SyntaxSynchronizer.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SyntaxSynchronizer.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SyntaxSynchronizer.java (original)
+++ directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/schema/registries/synchronizers/SyntaxSynchronizer.java Fri Jul 30 20:44:57 2010
@@ -100,8 +100,8 @@ public class SyntaxSynchronizer extends 
     public void add( Entry entry ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=syntaxes,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.SYNTAX );
@@ -200,8 +200,8 @@ public class SyntaxSynchronizer extends 
     public void delete( Entry entry, boolean cascade ) throws LdapException
     {
         DN dn = entry.getDn();
-        DN parentDn = ( DN ) dn.clone();
-        parentDn.remove( parentDn.size() - 1 );
+        DN parentDn = dn;
+        parentDn = parentDn.remove( parentDn.size() - 1 );
 
         // The parent DN must be ou=syntaxes,cn=<schemaName>,ou=schema
         checkParent( parentDn, schemaManager, SchemaConstants.SYNTAX );

Propchange: directory/apacheds/trunk/core-avl/
------------------------------------------------------------------------------
--- svn:mergeinfo (added)
+++ svn:mergeinfo Fri Jul 30 20:44:57 2010
@@ -0,0 +1,2 @@
+/directory/apacheds/branches/apacheds-dnfactory-experiment/core-avl:980138-980934
+/directory/apacheds/trunk/core-avl:980025-980137

Propchange: directory/apacheds/trunk/core-constants/
------------------------------------------------------------------------------
--- svn:mergeinfo (added)
+++ svn:mergeinfo Fri Jul 30 20:44:57 2010
@@ -0,0 +1,2 @@
+/directory/apacheds/branches/apacheds-dnfactory-experiment/core-constants:980138-980934
+/directory/apacheds/trunk/core-constants:980025-980137

Propchange: directory/apacheds/trunk/core-integ/
------------------------------------------------------------------------------
--- svn:mergeinfo (added)
+++ svn:mergeinfo Fri Jul 30 20:44:57 2010
@@ -0,0 +1 @@
+/directory/apacheds/branches/apacheds-dnfactory-experiment/core-integ:980138-980934

Modified: directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/authz/MoveRenameAuthorizationIT.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/authz/MoveRenameAuthorizationIT.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/authz/MoveRenameAuthorizationIT.java (original)
+++ directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/authz/MoveRenameAuthorizationIT.java Fri Jul 30 20:44:57 2010
@@ -164,7 +164,7 @@ public class MoveRenameAuthorizationIT e
         {
             if( isMoved )
             {
-                entryDN.add( 1, newParentRdn );
+                entryDN = entryDN.add( 1, newParentRdn );
                 adminConnection.delete( entryDN );
             }
             

Modified: directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/authz/SearchAuthorizationIT.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/authz/SearchAuthorizationIT.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/authz/SearchAuthorizationIT.java (original)
+++ directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/authz/SearchAuthorizationIT.java Fri Jul 30 20:44:57 2010
@@ -135,8 +135,8 @@ public class SearchAuthorizationIT exten
         for ( int i = 0; ( i < children.length ) && ( count[0] < sizeLimit ); i++ )
         {
             DN childRdn = new DN();
-            childRdn.addAll( parent );
-            childRdn.add( "ou=" + i );
+            childRdn = childRdn.addAll( parent );
+            childRdn = childRdn.add( "ou=" + i );
             childRdns[i] = childRdn;
             children[i].setDn( childRdn );
             reusableAdminCon.add( children[i] );

Propchange: directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/collective/CollectiveAttributeServiceIT.java
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Fri Jul 30 20:44:57 2010
@@ -1,3 +1,4 @@
+/directory/apacheds/branches/apacheds-dnfactory-experiment/core-integ/src/test/java/org/apache/directory/server/core/collective/CollectiveAttributeServiceIT.java:980138-980934
 /directory/apacheds/branches/apacheds-replication/core-integ/src/test/java/org/apache/directory/server/core/collective/CollectiveAttributeServiceIT.java:749790-764110
 /directory/apacheds/branches/apacheds-schema/core-integ/src/test/java/org/apache/directory/server/core/collective/CollectiveAttributeServiceIT.java:806623-896441
 /directory/apacheds/branches/apacheds-subtree/core-integ/src/test/java/org/apache/directory/server/core/collective/CollectiveAttributeServiceIT.java:965202-966561

Modified: directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/exception/ExceptionServiceIT.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/exception/ExceptionServiceIT.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/exception/ExceptionServiceIT.java (original)
+++ directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/exception/ExceptionServiceIT.java Fri Jul 30 20:44:57 2010
@@ -72,8 +72,8 @@ public class ExceptionServiceIT extends 
 
     private AddResponse createSubContext( DN parent, String type, String value ) throws Exception
     {
-        DN dn = ( DN ) parent.clone();
-        dn.add( "ou=" + value );
+        DN dn = parent;
+        dn = dn.add( "ou=" + value );
         Entry entry = new DefaultEntry( dn );
         entry.add( SchemaConstants.OBJECT_CLASS_AT, "person" );
         entry.add( SchemaConstants.OBJECT_CLASS_AT, "OrganizationalPerson" );

Modified: directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaAttributeTypeHandlerIT.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaAttributeTypeHandlerIT.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaAttributeTypeHandlerIT.java (original)
+++ directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaAttributeTypeHandlerIT.java Fri Jul 30 20:44:57 2010
@@ -89,7 +89,7 @@ public class MetaAttributeTypeHandlerIT 
          );
         
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=2.5.4.58" );
+        dn = dn.add( "m-oid=2.5.4.58" );
         
         // Pre-checks
         assertFalse( isOnDisk( dn ) );
@@ -120,7 +120,7 @@ public class MetaAttributeTypeHandlerIT 
             "m-usage: directoryOperation" );
         
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
         
         // Pre-checks
         assertFalse( isOnDisk( dn ) );
@@ -151,7 +151,7 @@ public class MetaAttributeTypeHandlerIT 
             "m-usage: directoryOperation" );
 
         DN dn = getAttributeTypeContainer( "notloaded" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
         
         try
         {
@@ -186,7 +186,7 @@ public class MetaAttributeTypeHandlerIT 
             "m-usage: directoryOperation" );
 
         DN dn = getAttributeTypeContainer( "nis" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
         
         getSchemaContext( service ).createSubcontext( DN.toName( dn ), attrs );
         
@@ -210,7 +210,7 @@ public class MetaAttributeTypeHandlerIT 
         testAddAttributeTypeToEnabledSchema();
 
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
 
         // Check in Registries
         assertTrue( "attributeType should be removed from the registry after being deleted", 
@@ -242,7 +242,7 @@ public class MetaAttributeTypeHandlerIT 
         testAddAttributeTypeToDisabledSchema();
 
         DN dn = getAttributeTypeContainer( "nis" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
 
         // Check in Registries
         assertFalse( "attributeType should be removed from the registry after being deleted", 
@@ -270,7 +270,7 @@ public class MetaAttributeTypeHandlerIT 
         testAddAttributeTypeToEnabledSchema();
 
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
         addDependeeAttributeType();
         
         try
@@ -301,7 +301,7 @@ public class MetaAttributeTypeHandlerIT 
         assertEquals( at.getSyntax().getOid(), SchemaConstants.INTEGER_SYNTAX );
 
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
         
         ModificationItem[] mods = new ModificationItem[2];
         Attribute attr = new BasicAttribute( "m-description", DESCRIPTION1 );
@@ -333,7 +333,7 @@ public class MetaAttributeTypeHandlerIT 
         assertEquals( at.getSyntax().getOid(), SchemaConstants.INTEGER_SYNTAX );
 
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
         
         Attributes mods = new BasicAttributes( true );
         mods.put( "m-description", DESCRIPTION1 );
@@ -363,10 +363,10 @@ public class MetaAttributeTypeHandlerIT 
 
         LdapContext schemaRoot = getSchemaContext( service );
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
         
         DN newdn = getAttributeTypeContainer( "apachemeta" );
-        newdn.add( "m-oid=" + NEW_OID );
+        dn = newdn.add( "m-oid=" + NEW_OID );
         schemaRoot.rename( DN.toName( dn ), DN.toName( newdn ) );
 
         assertFalse( "old attributeType OID should be removed from the registry after being renamed", 
@@ -386,11 +386,11 @@ public class MetaAttributeTypeHandlerIT 
         testAddAttributeTypeToEnabledSchema();
 
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
         addDependeeAttributeType();
         
         DN newdn = getAttributeTypeContainer( "apachemeta" );
-        newdn.add( "m-oid=" + NEW_OID );
+        newdn = newdn.add( "m-oid=" + NEW_OID );
         
         try
         {
@@ -417,10 +417,10 @@ public class MetaAttributeTypeHandlerIT 
         testAddAttributeTypeToEnabledSchema();
         
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
 
         DN newdn = getAttributeTypeContainer( "apache" );
-        newdn.add( "m-oid=" + OID );
+        newdn = newdn.add( "m-oid=" + OID );
         
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
@@ -439,10 +439,10 @@ public class MetaAttributeTypeHandlerIT 
         testAddAttributeTypeToEnabledSchema();
         
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
 
         DN newdn = getAttributeTypeContainer( "apache" );
-        newdn.add( "m-oid=" + NEW_OID );
+        newdn = newdn.add( "m-oid=" + NEW_OID );
         
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
@@ -464,10 +464,10 @@ public class MetaAttributeTypeHandlerIT 
         testAddAttributeTypeToEnabledSchema();
         
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
 
         DN top = new DN();
-        top.add( "m-oid=" + OID );
+        top = top.add( "m-oid=" + OID );
         
         try
         {
@@ -491,10 +491,10 @@ public class MetaAttributeTypeHandlerIT 
         testAddAttributeTypeToEnabledSchema();
         
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
 
         DN newdn = new DN( "ou=comparators,cn=apachemeta" );
-        newdn.add( "m-oid=" + OID );
+        newdn = newdn.add( "m-oid=" + OID );
         
         try
         {
@@ -518,11 +518,11 @@ public class MetaAttributeTypeHandlerIT 
         testAddAttributeTypeToEnabledSchema();
         
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
 
         // nis is inactive by default
         DN newdn = getAttributeTypeContainer( "nis" );
-        newdn.add( "m-oid=" + OID );
+        newdn = newdn.add( "m-oid=" + OID );
         
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
@@ -539,10 +539,10 @@ public class MetaAttributeTypeHandlerIT 
         addDependeeAttributeType();
         
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
 
         DN newdn = getAttributeTypeContainer( "apache" );
-        newdn.add( "m-oid=" + OID );
+        newdn = newdn.add( "m-oid=" + OID );
         
         try
         {
@@ -567,10 +567,10 @@ public class MetaAttributeTypeHandlerIT 
         addDependeeAttributeType();
         
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + OID );
+        dn = dn.add( "m-oid=" + OID );
 
         DN newdn = getAttributeTypeContainer( "apache" );
-        newdn.add( "m-oid=" + NEW_OID );
+        newdn = newdn.add( "m-oid=" + NEW_OID );
         
         try
         {
@@ -605,7 +605,7 @@ public class MetaAttributeTypeHandlerIT 
             "m-supAttributeType", OID );
         
         DN dn = getAttributeTypeContainer( "apachemeta" );
-        dn.add( "m-oid=" + DEPENDEE_OID );
+        dn = dn.add( "m-oid=" + DEPENDEE_OID );
         getSchemaContext( service ).createSubcontext( DN.toName( dn ), attrs );
         
         assertTrue( service.getSchemaManager().getAttributeTypeRegistry().contains( DEPENDEE_OID ) );

Modified: directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaComparatorHandlerIT.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaComparatorHandlerIT.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaComparatorHandlerIT.java (original)
+++ directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaComparatorHandlerIT.java Fri Jul 30 20:44:57 2010
@@ -102,7 +102,7 @@ public class MetaComparatorHandlerIT ext
             "m-description: A test comparator" );
 
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         // Pre-checks
         assertFalse( isOnDisk( dn ) );
@@ -129,7 +129,7 @@ public class MetaComparatorHandlerIT ext
 
         // nis is by default inactive
         DN dn = getComparatorContainer( "nis" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         // Pre-checks
         assertFalse( isOnDisk( dn ) );
@@ -154,7 +154,7 @@ public class MetaComparatorHandlerIT ext
 
         // nis is by default inactive
         DN dn = getComparatorContainer( "notloaded" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         // Pre-checks
         assertFalse( isOnDisk( dn ) );
@@ -195,7 +195,7 @@ public class MetaComparatorHandlerIT ext
                 .toByteArray(), "m-oid", OID, "m-description: A test comparator" );
 
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         // Pre-checks
         assertFalse( isOnDisk( dn ) );
@@ -231,7 +231,7 @@ public class MetaComparatorHandlerIT ext
 
         // nis is by default inactive
         DN dn = getComparatorContainer( "nis" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         // Pre-checks
         assertFalse( isOnDisk( dn ) );
@@ -255,7 +255,7 @@ public class MetaComparatorHandlerIT ext
     public void testDeleteComparatorFromEnabledSchema() throws Exception
     {
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         testAddComparatorToEnabledSchema();
 
@@ -287,7 +287,7 @@ public class MetaComparatorHandlerIT ext
     public void testDeleteComparatorFromDisabledSchema() throws Exception
     {
         DN dn = getComparatorContainer( "nis" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         testAddComparatorToDisabledSchema();
 
         // Pre-checks
@@ -319,7 +319,7 @@ public class MetaComparatorHandlerIT ext
     public void testDeleteComparatorWhenInUse() throws Exception
     {
         DN cDn = getComparatorContainer( "apachemeta" );
-        cDn.add( "m-oid" + "=" + OID );
+        cDn = cDn.add( "m-oid" + "=" + OID );
 
         // Create a new Comparator
         testAddComparatorToEnabledSchema();
@@ -336,7 +336,7 @@ public class MetaComparatorHandlerIT ext
             "m-description: test" );
 
         DN mrDn = getMatchingRuleContainer( "apachemeta" );
-        mrDn.add( "m-oid" + "=" + OID );
+        mrDn = mrDn.add( "m-oid" + "=" + OID );
 
         // Pre-checks
         assertFalse( isOnDisk( mrDn ) );
@@ -373,7 +373,7 @@ public class MetaComparatorHandlerIT ext
         testAddComparatorToEnabledSchema();
 
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         ModificationItem[] mods = new ModificationItem[1];
         Attribute attr = new BasicAttribute( "m-fqcn", BooleanComparator.class.getName() );
@@ -397,7 +397,7 @@ public class MetaComparatorHandlerIT ext
         testAddComparatorToEnabledSchema();
 
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         Attributes mods = new BasicAttributes( true );
         mods.put( "m-fqcn", BooleanComparator.class.getName() );
@@ -421,11 +421,11 @@ public class MetaComparatorHandlerIT ext
     public void testRenameComparator() throws Exception
     {
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         testAddComparatorToEnabledSchema();
 
         DN newdn = getComparatorContainer( "apachemeta" );
-        newdn.add( "m-oid" + "=" + NEW_OID );
+        newdn = newdn.add( "m-oid" + "=" + NEW_OID );
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
         assertFalse( "old comparator OID should be removed from the registry after being renamed", schemaManager
@@ -451,12 +451,12 @@ public class MetaComparatorHandlerIT ext
     public void testRenameComparatorWhenInUse() throws Exception
     {
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         testAddComparatorToEnabledSchema();
         schemaManager.getMatchingRuleRegistry().register( new DummyMR() );
 
         DN newdn = getComparatorContainer( "apachemeta" );
-        newdn.add( "m-oid" + "=" + NEW_OID );
+        newdn = newdn.add( "m-oid" + "=" + NEW_OID );
 
         try
         {
@@ -485,10 +485,10 @@ public class MetaComparatorHandlerIT ext
         testAddComparatorToEnabledSchema();
 
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN newdn = getComparatorContainer( "apache" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
 
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
@@ -509,10 +509,10 @@ public class MetaComparatorHandlerIT ext
         testAddComparatorToEnabledSchema();
 
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN newdn = getComparatorContainer( "apache" );
-        newdn.add( "m-oid" + "=" + NEW_OID );
+        newdn = newdn.add( "m-oid" + "=" + NEW_OID );
 
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
@@ -539,10 +539,10 @@ public class MetaComparatorHandlerIT ext
         schemaManager.getMatchingRuleRegistry().register( new DummyMR() );
 
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN newdn = getComparatorContainer( "apache" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
 
         try
         {
@@ -569,10 +569,10 @@ public class MetaComparatorHandlerIT ext
         schemaManager.getMatchingRuleRegistry().register( new DummyMR() );
 
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN newdn = getComparatorContainer( "apache" );
-        newdn.add( "m-oid" + "=" + NEW_OID );
+        newdn = newdn.add( "m-oid" + "=" + NEW_OID );
 
         try
         {
@@ -602,10 +602,10 @@ public class MetaComparatorHandlerIT ext
         testAddComparatorToEnabledSchema();
 
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN top = new DN();
-        top.add( "m-oid" + "=" + OID );
+        top = top.add( "m-oid" + "=" + OID );
 
         try
         {
@@ -629,10 +629,10 @@ public class MetaComparatorHandlerIT ext
         testAddComparatorToEnabledSchema();
 
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN newdn = new DN( "ou=normalizers,cn=apachemeta" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
 
         try
         {
@@ -656,11 +656,11 @@ public class MetaComparatorHandlerIT ext
         testAddComparatorToEnabledSchema();
 
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         // nis is inactive by default
         DN newdn = getComparatorContainer( "nis" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
 
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
@@ -676,13 +676,13 @@ public class MetaComparatorHandlerIT ext
 
         // nis is inactive by default
         DN dn = getComparatorContainer( "nis" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         assertFalse( "comparator OID should NOT be present when added to disabled nis schema", schemaManager
             .getComparatorRegistry().contains( OID ) );
 
         DN newdn = getComparatorContainer( "apachemeta" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
 
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 

Modified: directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaMatchingRuleHandlerIT.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaMatchingRuleHandlerIT.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaMatchingRuleHandlerIT.java (original)
+++ directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaMatchingRuleHandlerIT.java Fri Jul 30 20:44:57 2010
@@ -87,7 +87,7 @@ public class MetaMatchingRuleHandlerIT e
             "m-description: A test comparator" );
         
         DN dn = getComparatorContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         
         // Addition
         getSchemaContext( service ).createSubcontext( DN.toName( dn ), attrs );
@@ -116,7 +116,7 @@ public class MetaMatchingRuleHandlerIT e
             "m-description", DESCRIPTION0 );
 
         DN dn = getMatchingRuleContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         
         // Pre-checks
         assertFalse( isOnDisk( dn ) );
@@ -146,7 +146,7 @@ public class MetaMatchingRuleHandlerIT e
             "m-description", DESCRIPTION0 );
         
         DN dn = getMatchingRuleContainer( "nis" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         getSchemaContext( service ).createSubcontext( DN.toName( dn ), attrs );
         
         assertFalse( "adding new matchingRule to disabled schema should not register it into the registries", 
@@ -169,7 +169,7 @@ public class MetaMatchingRuleHandlerIT e
             "m-description", DESCRIPTION0 );
         
         DN dn = getMatchingRuleContainer( "notloaded" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         
         try
         {
@@ -194,7 +194,7 @@ public class MetaMatchingRuleHandlerIT e
     public void testDeleteMatchingRuleFromEnabledSchema() throws Exception
     {
         DN dn = getMatchingRuleContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         testAddMatchingRuleToEnabledSchema();
         
         assertTrue( "matchingRule should be removed from the registry after being deleted", 
@@ -223,7 +223,7 @@ public class MetaMatchingRuleHandlerIT e
     public void testDeleteMatchingRuleFromDisabledSchema() throws Exception
     {
         DN dn = getMatchingRuleContainer( "nis" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         testAddMatchingRuleToDisabledSchema();
         
         assertFalse( "matchingRule should be removed from the registry after being deleted", 
@@ -252,7 +252,7 @@ public class MetaMatchingRuleHandlerIT e
         assertEquals( mr.getSyntax().getOid(), SchemaConstants.INTEGER_SYNTAX );
 
         DN dn = getMatchingRuleContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         
         ModificationItem[] mods = new ModificationItem[2];
         Attribute attr = new BasicAttribute( "m-description", DESCRIPTION1 );
@@ -284,7 +284,7 @@ public class MetaMatchingRuleHandlerIT e
         assertEquals( mr.getSyntax().getOid(), SchemaConstants.INTEGER_SYNTAX );
 
         DN dn = getMatchingRuleContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         
         Attributes mods = new BasicAttributes( true );
         mods.put( "m-description", DESCRIPTION1 );
@@ -311,11 +311,11 @@ public class MetaMatchingRuleHandlerIT e
     public void testRenameMatchingRule() throws Exception
     {
         DN dn = getMatchingRuleContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         testAddMatchingRuleToEnabledSchema();
         
         DN newdn = getMatchingRuleContainer( "apachemeta" );
-        newdn.add( "m-oid" + "=" + NEW_OID );
+        newdn = newdn.add( "m-oid" + "=" + NEW_OID );
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
         assertFalse( "old matchingRule OID should be removed from the registry after being renamed", 
@@ -344,10 +344,10 @@ public class MetaMatchingRuleHandlerIT e
         testAddMatchingRuleToEnabledSchema();
         
         DN dn = getMatchingRuleContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN newdn = getMatchingRuleContainer( "apache" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
         
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
@@ -366,10 +366,10 @@ public class MetaMatchingRuleHandlerIT e
         testAddMatchingRuleToEnabledSchema();
         
         DN dn = getMatchingRuleContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN newdn = getMatchingRuleContainer( "apache" );
-        newdn.add( "m-oid" + "=" + NEW_OID );
+        newdn = newdn.add( "m-oid" + "=" + NEW_OID );
         
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
@@ -420,7 +420,7 @@ public class MetaMatchingRuleHandlerIT e
 //        dn.add( "m-oid" + "=" + OID );
 //
 //        DN newdn = getSyntaxContainer( "apache" );
-//        newdn.add( "m-oid" + "=" + OID );
+//        newdn = newdn.add( "m-oid" + "=" + OID );
 //        
 //        try
 //        {
@@ -446,7 +446,7 @@ public class MetaMatchingRuleHandlerIT e
 //        dn.add( "m-oid" + "=" + OID );s
 //
 //        DN newdn = getSyntaxContainer( "apache" );
-//        newdn.add( "m-oid" + "=" + NEW_OID );
+//        newdn = newdn.add( "m-oid" + "=" + NEW_OID );
 //        
 //        try
 //        {
@@ -480,7 +480,7 @@ public class MetaMatchingRuleHandlerIT e
 //        addDependeeMatchingRule();
 //        
 //        DN newdn = getSyntaxContainer( "apachemeta" );
-//        newdn.add( "m-oid" + "=" + NEW_OID );
+//        newdn = newdn.add( "m-oid" + "=" + NEW_OID );
 //        
 //        try
 //        {
@@ -509,10 +509,10 @@ public class MetaMatchingRuleHandlerIT e
         testAddMatchingRuleToEnabledSchema();
         
         DN dn = getMatchingRuleContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN top = new DN();
-        top.add( "m-oid" + "=" + OID );
+        top = top.add( "m-oid" + "=" + OID );
         
         try
         {
@@ -536,10 +536,10 @@ public class MetaMatchingRuleHandlerIT e
         testAddMatchingRuleToEnabledSchema();
         
         DN dn = getMatchingRuleContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN newdn = new DN( "ou=comparators,cn=apachemeta" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
         
         try
         {
@@ -563,11 +563,11 @@ public class MetaMatchingRuleHandlerIT e
         testAddMatchingRuleToEnabledSchema();
         
         DN dn = getMatchingRuleContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         // nis is inactive by default
         DN newdn = getMatchingRuleContainer( "nis" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
         
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
@@ -584,13 +584,13 @@ public class MetaMatchingRuleHandlerIT e
         
         // nis is inactive by default
         DN dn = getMatchingRuleContainer( "nis" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         assertFalse( "matchingRule OID should NOT be present when added to disabled nis schema", 
             schemaManager.getMatchingRuleRegistry().contains( OID ) );
 
         DN newdn = getMatchingRuleContainer( "apachemeta" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
         
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 

Modified: directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaNormalizerHandlerIT.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaNormalizerHandlerIT.java?rev=980948&r1=980947&r2=980948&view=diff
==============================================================================
--- directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaNormalizerHandlerIT.java (original)
+++ directory/apacheds/trunk/core-integ/src/test/java/org/apache/directory/server/core/schema/MetaNormalizerHandlerIT.java Fri Jul 30 20:44:57 2010
@@ -93,7 +93,7 @@ public class MetaNormalizerHandlerIT ext
             "m-description: A test normalizer" );
         
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         getSchemaContext( service ).createSubcontext( DN.toName( dn ), attrs );
         
         assertTrue( schemaManager.getNormalizerRegistry().contains( OID ) );
@@ -117,7 +117,7 @@ public class MetaNormalizerHandlerIT ext
         
         // nis is by default inactive
         DN dn = getNormalizerContainer( "nis" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         getSchemaContext( service ).createSubcontext( DN.toName( dn ), attrs );
         
         assertFalse( "adding new normalizer to disabled schema should not register it into the registries", 
@@ -139,7 +139,7 @@ public class MetaNormalizerHandlerIT ext
         
         // nis is by default inactive
         DN dn = getNormalizerContainer( "notloaded" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         try
         {
@@ -178,7 +178,7 @@ public class MetaNormalizerHandlerIT ext
             "m-description: A test normalizer" );
         
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         getSchemaContext( service ).createSubcontext( DN.toName( dn ), attrs );
         
         assertTrue( schemaManager.getNormalizerRegistry().contains( OID ) );
@@ -210,7 +210,7 @@ public class MetaNormalizerHandlerIT ext
             "m-description: A test normalizer" );
         
         DN dn = getNormalizerContainer( "nis" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         getSchemaContext( service ).createSubcontext( DN.toName( dn ), attrs );
         
         assertFalse( schemaManager.getNormalizerRegistry().contains( OID ) );
@@ -222,7 +222,7 @@ public class MetaNormalizerHandlerIT ext
     public void testDeleteNormalizerFromEnabledSchema() throws Exception
     {
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         testAddNormalizerToEnabledSchema();
         
         assertTrue( "normalizer should be removed from the registry after being deleted", 
@@ -251,7 +251,7 @@ public class MetaNormalizerHandlerIT ext
     public void testDeleteNormalizerFromDisabledSchema() throws Exception
     {
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         testAddNormalizerToEnabledSchema();
 
         assertTrue( "normalizer should be removed from the registry after being deleted", 
@@ -281,11 +281,11 @@ public class MetaNormalizerHandlerIT ext
     public void testRenameNormalizer() throws Exception
     {
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         testAddNormalizerToEnabledSchema();
         
         DN newdn = getNormalizerContainer( "apachemeta" );
-        newdn.add( "m-oid" + "=" + NEW_OID );
+        newdn = newdn.add( "m-oid" + "=" + NEW_OID );
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
         assertFalse( "old normalizer OID should be removed from the registry after being renamed", 
@@ -314,10 +314,10 @@ public class MetaNormalizerHandlerIT ext
         testAddNormalizerToEnabledSchema();
         
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN newdn = getNormalizerContainer( "apache" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
         
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
@@ -339,10 +339,10 @@ public class MetaNormalizerHandlerIT ext
         testAddNormalizerToEnabledSchema();
         
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN newdn = getNormalizerContainer( "apache" );
-        newdn.add( "m-oid" + "=" + NEW_OID );
+        newdn = newdn.add( "m-oid" + "=" + NEW_OID );
         
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
@@ -367,7 +367,7 @@ public class MetaNormalizerHandlerIT ext
         testAddNormalizerToEnabledSchema();
         
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         
         ModificationItem[] mods = new ModificationItem[1];
         Attribute attr = new BasicAttribute( "m-fqcn", DeepTrimNormalizer.class.getName() );
@@ -392,7 +392,7 @@ public class MetaNormalizerHandlerIT ext
         testAddNormalizerToEnabledSchema();
         
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         
         Attributes mods = new BasicAttributes( true );
         mods.put( "m-fqcn", DeepTrimNormalizer.class.getName() );
@@ -416,7 +416,7 @@ public class MetaNormalizerHandlerIT ext
     public void testDeleteNormalizerWhenInUse() throws Exception
     {
         DN nDn = getNormalizerContainer( "apachemeta" );
-        nDn.add( "m-oid" + "=" + OID );
+        nDn = nDn.add( "m-oid" + "=" + OID );
 
         // Create a new Normalizer
         testAddNormalizerToEnabledSchema();
@@ -433,7 +433,7 @@ public class MetaNormalizerHandlerIT ext
             "m-description: test" );
 
         DN mrDn = getMatchingRuleContainer( "apachemeta" );
-        mrDn.add( "m-oid" + "=" + OID );
+        mrDn = mrDn.add( "m-oid" + "=" + OID );
 
         // Pre-checks
         assertFalse( isOnDisk( mrDn ) );
@@ -468,10 +468,10 @@ public class MetaNormalizerHandlerIT ext
         schemaManager.getMatchingRuleRegistry().register( new DummyMR() );
         
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN newdn = getNormalizerContainer( "apache" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
         
         try
         {
@@ -497,10 +497,10 @@ public class MetaNormalizerHandlerIT ext
         schemaManager.getMatchingRuleRegistry().register( new DummyMR() );
         
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN newdn = getNormalizerContainer( "apache" );
-        newdn.add( "m-oid" + "=" + NEW_OID );
+        newdn = newdn.add( "m-oid" + "=" + NEW_OID );
         
         try
         {
@@ -523,12 +523,12 @@ public class MetaNormalizerHandlerIT ext
     public void testRenameNormalizerWhenInUse() throws Exception
     {
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
         testAddNormalizerToEnabledSchema();
         schemaManager.getMatchingRuleRegistry().register( new DummyMR() );
         
         DN newdn = getNormalizerContainer( "apachemeta" );
-        newdn.add( "m-oid" + "=" + NEW_OID );
+        newdn = newdn.add( "m-oid" + "=" + NEW_OID );
         
         try
         {
@@ -558,10 +558,10 @@ public class MetaNormalizerHandlerIT ext
         testAddNormalizerToEnabledSchema();
         
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN top = new DN();
-        top.add( "m-oid" + "=" + OID );
+        top = top.add( "m-oid" + "=" + OID );
         
         try
         {
@@ -585,10 +585,10 @@ public class MetaNormalizerHandlerIT ext
         testAddNormalizerToEnabledSchema();
         
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         DN newdn = new DN( "ou=comparators,cn=apachemeta" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
         
         try
         {
@@ -612,11 +612,11 @@ public class MetaNormalizerHandlerIT ext
         testAddNormalizerToEnabledSchema();
         
         DN dn = getNormalizerContainer( "apachemeta" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         // nis is inactive by default
         DN newdn = getNormalizerContainer( "nis" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
         
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 
@@ -633,13 +633,13 @@ public class MetaNormalizerHandlerIT ext
         
         // nis is inactive by default
         DN dn = getNormalizerContainer( "nis" );
-        dn.add( "m-oid" + "=" + OID );
+        dn = dn.add( "m-oid" + "=" + OID );
 
         assertFalse( "normalizer OID should NOT be present when added to disabled nis schema", 
             schemaManager.getNormalizerRegistry().contains( OID ) );
 
         DN newdn = getNormalizerContainer( "apachemeta" );
-        newdn.add( "m-oid" + "=" + OID );
+        newdn = newdn.add( "m-oid" + "=" + OID );
         
         getSchemaContext( service ).rename( DN.toName( dn ), DN.toName( newdn ) );
 



Mime
View raw message