Minor clean-up of ref property mapper

Guy Paddock

27.20.2017 5060dad8106471069963df5b75ada0ceadefcb97

Minor clean-up of ref property mapper

Minor tweaks to Javadocs and a little formatting (this class needs a lot more).

 opendj-rest2ldap/src/main/java/org/forgerock/opendj/rest2ldap/ReferencePropertyMapper.java

@@ -82,6 +82,7 @@
                            final String baseDnTemplate, final AttributeDescription primaryKey,
                            final PropertyMapper mapper) {
        super(ldapAttributeName);

        this.schema = schema;
        this.baseDnTemplate = DnTemplate.compile(baseDnTemplate);
        this.primaryKey = primaryKey;
@@ -89,12 +90,10 @@
    }

    /**
     * Sets the filter which should be used when searching for referenced LDAP
     * entries. The default is {@code (objectClass=*)}.
     * Sets the filter which should be used when searching for referenced LDAP entries.
     *
     * @param filter
     *            The filter which should be used when searching for referenced
     *            LDAP entries.
     *          The filter which should be used when searching for referenced LDAP entries.
     * @return This property mapper.
     */
    public ReferencePropertyMapper searchFilter(final Filter filter) {
@@ -104,11 +103,10 @@

    /**
     * Sets the filter which should be used when searching for referenced LDAP
     * entries. The default is {@code (objectClass=*)}.
     * entries.
     *
     * @param filter
     *            The filter which should be used when searching for referenced
     *            LDAP entries.
     *          The filter which should be used when searching for referenced LDAP entries.
     * @return This property mapper.
     */
    public ReferencePropertyMapper searchFilter(final String filter) {
@@ -116,8 +114,8 @@
    }

    /**
     * Sets the search scope which should be used when searching for referenced
     * LDAP entries. The default is {@link SearchScope#WHOLE_SUBTREE}.
     * Sets the search scope which should be used when searching for referenced LDAP entries.
     * The default is {@link SearchScope#WHOLE_SUBTREE}.
     *
     * @param scope
     *            The search scope which should be used when searching for
@@ -142,9 +140,9 @@
        return mapper.getLdapFilter(context, resource, path, subPath, type, operator, valueAssertion)
                .thenAsync(new AsyncFunction<Filter, Filter, ResourceException>() {
                    @Override
                    public Promise<Filter, ResourceException> apply(final Filter result) {
                    public Promise<Filter, ResourceException> apply(final Filter filter) {
                        // Search for all referenced entries and construct a filter.
                        final SearchRequest request = createSearchRequest(context, result);
                        final SearchRequest request = createSearchRequest(context, filter);
                        final List<Filter> subFilters = new LinkedList<>();

                        return connectionFrom(context).searchAsync(request, new SearchResultHandler() {
@@ -325,8 +323,9 @@
        }
    }

    private SearchRequest createSearchRequest(final Context context, final Filter result) {
        final Filter searchFilter = filter != null ? Filter.and(filter, result) : result;
    private SearchRequest createSearchRequest(final Context context, final Filter filter) {
        final Filter searchFilter = this.filter != null ? Filter.and(this.filter, filter) : filter;

        return newSearchRequest(baseDnTemplate.format(context), scope, searchFilter, "1.1");
    }