Minor code simplification.

Matthew Swift

15.49.2013 c06598e95731538caf6025e486098cd3338787d0

Minor code simplification.

 opendj3/opendj-rest2ldap/src/main/java/org/forgerock/opendj/rest2ldap/JSONConstantAttributeMapper.java

@@ -15,6 +15,8 @@
 */
package org.forgerock.opendj.rest2ldap;

import static org.forgerock.opendj.ldap.Filter.alwaysFalse;
import static org.forgerock.opendj.ldap.Filter.alwaysTrue;
import static org.forgerock.opendj.rest2ldap.Utils.toFilter;
import static org.forgerock.opendj.rest2ldap.Utils.toLowerCase;

@@ -51,9 +53,9 @@
        final Filter filter;
        final JsonValue subValue = value.get(jsonAttribute);
        if (subValue == null) {
            filter = toFilter(false);
            filter = alwaysFalse();
        } else if (type == FilterType.PRESENT) {
            filter = toFilter(true);
            filter = alwaysTrue();
        } else if (value.isString() && valueAssertion instanceof String) {
            final String v1 = toLowerCase(value.asString());
            final String v2 = toLowerCase((String) valueAssertion);
@@ -78,7 +80,7 @@
            filter = compare(c, type, v1, v2);
        } else {
            // This attribute mapper is a candidate but it does not match.
            filter = toFilter(false);
            filter = alwaysFalse();
        }
        h.handleResult(filter);
    }
@@ -114,7 +116,7 @@
            filter = toFilter(v1.compareTo(v2) <= 0);
            break;
        default:
            filter = toFilter(false); // Not supported.
            filter = alwaysFalse(); // Not supported.
            break;
        }
        return filter;

 opendj3/opendj-rest2ldap/src/main/java/org/forgerock/opendj/rest2ldap/ObjectAttributeMapper.java

@@ -15,8 +15,8 @@
 */
package org.forgerock.opendj.rest2ldap;

import static org.forgerock.opendj.ldap.Filter.alwaysFalse;
import static org.forgerock.opendj.rest2ldap.Utils.accumulate;
import static org.forgerock.opendj.rest2ldap.Utils.toFilter;
import static org.forgerock.opendj.rest2ldap.Utils.toLowerCase;
import static org.forgerock.opendj.rest2ldap.Utils.transform;

@@ -102,7 +102,7 @@
            // Either the filter targeted the entire object (i.e. it was "/"), or it targeted
            // an unrecognized attribute within the object. Either way, the filter will
            // never match.
            h.handleResult(toFilter(false));
            h.handleResult(alwaysFalse());
        }
    }


 opendj3/opendj-rest2ldap/src/main/java/org/forgerock/opendj/rest2ldap/SimpleAttributeMapper.java

@@ -19,6 +19,7 @@
import static java.util.Collections.emptySet;
import static java.util.Collections.singleton;
import static java.util.Collections.singletonList;
import static org.forgerock.opendj.ldap.Filter.alwaysFalse;
import static org.forgerock.opendj.ldap.Functions.fixedFunction;
import static org.forgerock.opendj.rest2ldap.Utils.base64ToByteString;
import static org.forgerock.opendj.rest2ldap.Utils.byteStringToBase64;
@@ -183,7 +184,7 @@
            h.handleResult(toFilter(c, type, ldapAttributeName.toString(), valueAssertion));
        } else {
            // This attribute mapper does not support partial filtering.
            h.handleResult(toFilter(false));
            h.handleResult(alwaysFalse());
        }
    }


 opendj3/opendj-rest2ldap/src/main/java/org/forgerock/opendj/rest2ldap/Utils.java

@@ -17,6 +17,7 @@

import static javax.xml.bind.DatatypeConverter.parseDateTime;
import static javax.xml.bind.DatatypeConverter.printDateTime;
import static org.forgerock.opendj.ldap.Filter.alwaysFalse;
import static org.forgerock.opendj.ldap.Functions.byteStringToBoolean;
import static org.forgerock.opendj.ldap.Functions.byteStringToGeneralizedTime;
import static org.forgerock.opendj.ldap.Functions.byteStringToLong;
@@ -249,7 +250,7 @@
            break;
        case EXTENDED:
        default:
            filter = toFilter(false); // Not supported.
            filter = alwaysFalse(); // Not supported.
            break;
        }
        return filter;