You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by se...@apache.org on 2017/03/20 14:54:37 UTC

svn commit: r1787790 - /directory/shared/trunk/ldap/model/src/main/java/org/apache/directory/api/ldap/model/entry/AbstractValue.java

Author: semancik
Date: Mon Mar 20 14:54:37 2017
New Revision: 1787790

URL: http://svn.apache.org/viewvc?rev=1787790&view=rev
Log:
Another fix to tolerate servers without proper syntax definition.

Modified:
    directory/shared/trunk/ldap/model/src/main/java/org/apache/directory/api/ldap/model/entry/AbstractValue.java

Modified: directory/shared/trunk/ldap/model/src/main/java/org/apache/directory/api/ldap/model/entry/AbstractValue.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/ldap/model/src/main/java/org/apache/directory/api/ldap/model/entry/AbstractValue.java?rev=1787790&r1=1787789&r2=1787790&view=diff
==============================================================================
--- directory/shared/trunk/ldap/model/src/main/java/org/apache/directory/api/ldap/model/entry/AbstractValue.java (original)
+++ directory/shared/trunk/ldap/model/src/main/java/org/apache/directory/api/ldap/model/entry/AbstractValue.java Mon Mar 20 14:54:37 2017
@@ -162,7 +162,12 @@ public abstract class AbstractValue<T> i
             {
                 if ( upValue != null )
                 {
-                    boolean isHR = attributeType.getSyntax().isHumanReadable();
+                    boolean isHR = true;
+                    // Some broken LDAP servers do not have proper syntax definitions
+                    if ( attributeType.getSyntax() != null )
+                    {
+                        isHR = attributeType.getSyntax().isHumanReadable();
+                    }
                     
 
                     if ( isHR != isHumanReadable() )