[#DIRSERVER-314] Reversed operations: LdapName.getPrefix() and LdapName.getSuffix()

Directory ApacheDS

Reversed operations: LdapName.getPrefix() and LdapName.getSuffix()

Created: 10/Sep/05 11:52 AM Updated: 10/Feb/06 12:34 PM

Return to search

Component/s:

None

Affects Version/s:

None

Fix Version/s:

None

Time Tracking:

Not Specified

Resolution Date:

14/Sep/05 08:43 PM

Description

« Hide

This code shows what we did wrong:

        org.apache.ldap.common.name.LdapName name = new org.apache.ldap.common.name.LdapName( "ou=a, ou=b, ou=c" );
        System.out.println( name.getPrefix( 1 ) );
        System.out.println( name.getPrefix( 2 ) );
        System.out.println( name.getSuffix( 1 ) );
        System.out.println( name.getSuffix( 2 ) );
        System.out.println( name.get( 0 ) );
        System.out.println( name.get( 1 ) );
        System.out.println( name.get( 2 ) );

        javax.naming.ldap.LdapName newName = new javax.naming.ldap.LdapName( "ou=a, ou=b, ou=c" );
        System.out.println( newName.getPrefix( 1 ) );
        System.out.println( newName.getPrefix( 2 ) );
        System.out.println( newName.getSuffix( 1 ) );
        System.out.println( newName.getSuffix( 2 ) );
        System.out.println( newName.get( 0 ) );
        System.out.println( newName.get( 1 ) );
        System.out.println( newName.get( 2 ) );

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

Repository	Revision	Date	User	Message
ASF	#280419	Mon Sep 12 20:52:12 UTC 2005	akarasulu	changes ... o fixed problem with test case in ExceptionServiceTest that used existing g attribute in Modify operation with add op but was rejected thanks to new checks for this o fixed getPrefix() and getSuffix() issues where they were reversed as filed in the following jira issue but Trustin in DIREVE-248 here: http://issues.apache.org/jira/browse/DIREVE-248
				Files Changed
				MODIFY /directory/shared/ldap/trunk/common/src/test/org/apache/ldap/common/name/LdapNameTest.java MODIFY /directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/partition/DefaultContextPartitionNexus.java MODIFY /directory/shared/ldap/trunk/common/src/java/org/apache/ldap/common/name/LdapName.java MODIFY /directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/jndi/ServerContext.java MODIFY /directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/operational/OperationalAttributeService.java MODIFY /directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/exception/ExceptionService.java MODIFY /directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/partition/impl/btree/jdbm/JdbmContextPartition.java MODIFY /directory/apacheds/trunk/core/src/test/org/apache/ldap/server/exception/ExceptionServiceTest.java MODIFY /directory/shared/ldap/trunk/common/src/java/org/apache/ldap/common/util/NamespaceTools.java

Alex Karasulu made changes - 13/Sep/05 05:52 AM

Field	Original Value	New Value
Status	Open [ 1 ]	In Progress [ 3 ]

[ Permlink | « Hide ]

Alex Karasulu added a comment - 13/Sep/05 05:53 AM

Committed changes in revision 280419 here:

http://svn.apache.org/viewcvs.cgi?view=rev&rev=280419

Alex Karasulu made changes - 13/Sep/05 05:53 AM

Resolution		Fixed [ 1 ]
Status	In Progress [ 3 ]	Resolved [ 5 ]

[ Permlink | « Hide ]

Trustin Lee added a comment - 14/Sep/05 08:42 PM

It is shouldn't be 'fixed' but 'duplicate' because we've found ~~DIR-112~~ is the same one.

Trustin Lee made changes - 14/Sep/05 08:42 PM

Status	Resolved [ 5 ]	Reopened [ 4 ]
Resolution	Fixed [ 1 ]

Trustin Lee made changes - 14/Sep/05 08:43 PM

Status	Reopened [ 4 ]	Closed [ 6 ]
Resolution		Duplicate [ 3 ]

Alex Karasulu made changes - 10/Feb/06 12:34 PM

Key	DIREVE-248	~~DIRSERVER-314~~
Fix Version/s	0.9.3 [ 12310193 ]
Project	Directory Server [ 10516 ]	Directory ApacheDS [ 12310260 ]
Affects Version/s	0.9.2 [ 12310192 ]