rollback fix for 2991, return entry should go through workflow elements

jdemendi

25.17.2008 1b4603e10c65c535a04e66c9fad1c602abb854c8

 opends/src/server/org/opends/server/core/SearchOperationWrapper.java

@@ -41,7 +41,6 @@
import org.opends.server.types.SearchResultEntry;
import org.opends.server.types.SearchResultReference;
import org.opends.server.types.SearchScope;
import org.opends.server.workflowelement.WorkflowElement;


/**
@@ -55,24 +54,6 @@
  // The wrapped operation.
  private SearchOperation search;

  // The workflow element which has invoked the current operation.
  // The returned entries and returned references must be sent to that
  // workflow element.
  private WorkflowElement<?> callingWorkflowElement = null;

  /**
   * Set the calling workflow element.
   *
   * @param callingWorkflowElement  the workflow element which has invoked
   *                                the current operation
   */
  public void setCallingWorkflowElement(
      WorkflowElement<?> callingWorkflowElement
      )
  {
    this.callingWorkflowElement = callingWorkflowElement;
  }

  /**
   * Creates a new search operation based on the provided search operation.
   *
@@ -91,20 +72,7 @@
  {
    boolean result;

    // If the calling workflow element is defined then send the return
    // entry to the workflow element, otherwise send the entry to the
    // calling operation.
    // Sometimes, the calling workflow element might not be set for
    // an internal operation because the internal search is done using
    // a local backend operation instead of an operation basis.
    if (callingWorkflowElement != null)
    {
      result = callingWorkflowElement.returnEntry(entry, controls);
    }
    else
    {
      result = this.search.returnEntry(entry, controls);
    }
    result = this.search.returnEntry(entry, controls);

    return result;
  }
@@ -116,19 +84,7 @@
  {
    boolean result;

    // If the calling workflow element is not set then send the
    // reference to the calling operation, otherwise send the reference
    // to the calling workflow element.
    // an internal operation because the internal search is done using
    // a local backend operation instead of an operation basis.
    if (callingWorkflowElement != null)
    {
      result = callingWorkflowElement.returnReference(dn, reference);
    }
    else
    {
      result = this.search.returnReference(dn, reference);
    }
    result = this.search.returnReference(dn, reference);

    return result;
  }