You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by vt...@apache.org on 2005/02/19 19:29:09 UTC

svn commit: r154437 - in incubator/directory/ldap/trunk/common/src: java/org/apache/ldap/common/name/LdapName.java test/org/apache/ldap/common/name/LdapNameTest.java

Author: vtence
Date: Sat Feb 19 10:29:07 2005
New Revision: 154437

URL: http://svn.apache.org/viewcvs?view=rev&rev=154437
Log:
Fixed DIRLDAP-35 : AttributeType comparison is now case insensitive

Modified:
    incubator/directory/ldap/trunk/common/src/java/org/apache/ldap/common/name/LdapName.java
    incubator/directory/ldap/trunk/common/src/test/org/apache/ldap/common/name/LdapNameTest.java

Modified: incubator/directory/ldap/trunk/common/src/java/org/apache/ldap/common/name/LdapName.java
URL: http://svn.apache.org/viewcvs/incubator/directory/ldap/trunk/common/src/java/org/apache/ldap/common/name/LdapName.java?view=diff&r1=154436&r2=154437
==============================================================================
--- incubator/directory/ldap/trunk/common/src/java/org/apache/ldap/common/name/LdapName.java (original)
+++ incubator/directory/ldap/trunk/common/src/java/org/apache/ldap/common/name/LdapName.java Sat Feb 19 10:29:07 2005
@@ -196,7 +196,7 @@
              */
             for ( int ii = 0; ii < l_min; ii++ ) 
             {
-                int l_comparison = get( ii ).compareTo( l_dn.get( ii ) ) ;
+                int l_comparison = get( ii ).compareToIgnoreCase( l_dn.get( ii ) ) ;
                 
                 if (  l_comparison != 0 )
                 {

Modified: incubator/directory/ldap/trunk/common/src/test/org/apache/ldap/common/name/LdapNameTest.java
URL: http://svn.apache.org/viewcvs/incubator/directory/ldap/trunk/common/src/test/org/apache/ldap/common/name/LdapNameTest.java?view=diff&r1=154436&r2=154437
==============================================================================
--- incubator/directory/ldap/trunk/common/src/test/org/apache/ldap/common/name/LdapNameTest.java (original)
+++ incubator/directory/ldap/trunk/common/src/test/org/apache/ldap/common/name/LdapNameTest.java Sat Feb 19 10:29:07 2005
@@ -24,6 +24,7 @@
 import java.util.Collections ;
 
 import javax.naming.Name ;
+import javax.naming.NamingException;
 
 import junit.framework.TestCase ;
 
@@ -946,5 +947,13 @@
             m_parser.parse( "ou=People" ) ) ) ; 
         assertTrue( ! m_parser.parse( "ou=Groups" ).equals( 
             m_parser.parse( "ou=People" ) ) ) ; 
+    }
+
+   public void testAttributeTypeComparisonIsCaseInsensitive() throws NamingException
+    {
+        Name name1 = new LdapName("cn=HomeDir+cn=WorkDir");
+        Name name2 = new LdapName("cn=HomeDir+CN=WorkDir");
+
+        assertEquals( 0, name1.compareTo( name2 ) );
     }
 }