Make all of the plugin result objects immutable, and create a default SUCCE...

neil_a_wilson

10.23.2006 bb40b9a02cf40af0349d8e3576f51133d264d4b1

 opends/src/server/org/opends/server/api/plugin/PreOperationPluginResult.java

@@ -46,17 +46,26 @@



  /**
   * A pre-operation plugin result instance that indicates all
   * processing was successful.
   */
  public static final PreOperationPluginResult SUCCESS =
       new PreOperationPluginResult();



  // Indicates whether any further pre-operation plugins should be
  // invoked for this operation.
  private boolean continuePluginProcessing;
  private final boolean continuePluginProcessing;

  // Indicates whether the pre-operation plugin terminated the client
  // connection.
  private boolean connectionTerminated;
  private final boolean connectionTerminated;

  // Indicates whether the server should immediately send the response
  // from this plugin to the client with no further processing.
  private boolean sendResponseImmediately;
  private final boolean sendResponseImmediately;



@@ -68,13 +77,9 @@
   * and that the post-operation plugin processing should not be
   * skipped.
   */
  public PreOperationPluginResult()
  private PreOperationPluginResult()
  {
    assert debugConstructor(CLASS_NAME);

    this.connectionTerminated     = false;
    this.continuePluginProcessing = true;
    this.sendResponseImmediately  = false;
    this(false, true, false);
  }


@@ -128,24 +133,6 @@


  /**
   * Specifies whether the post-response plugin terminated the client
   * connection.
   *
   * @param  connectionTerminated  Specifies whether the post-response
   *                               plugin terminated the client
   *                               connection.
   */
  public void setConnectionTerminated(boolean connectionTerminated)
  {
    assert debugEnter(CLASS_NAME, "setConnectionTerminated",
                      String.valueOf(connectionTerminated));

    this.connectionTerminated = connectionTerminated;
  }



  /**
   * Indicates whether any further post-response plugins should be
   * invoked for this operation.
   *
@@ -163,25 +150,6 @@


  /**
   * Specifies whether any further post-response plugins should be
   * invoked for this operation.
   *
   * @param  continuePluginProcessing  Specifies whether any further
   *                                   post-response plugins should be
   *                                   invoked for this operation.
   */
  public void setContinuePluginProcessing(
                   boolean continuePluginProcessing)
  {
    assert debugEnter(CLASS_NAME, "setContinuePluginProcessing",
                      String.valueOf(continuePluginProcessing));

    this.continuePluginProcessing = continuePluginProcessing;
  }



  /**
   * Indicates whether the server should send the response set by this
   * plugin to the client immediately with no further processing on
   * the operation.
@@ -201,28 +169,6 @@


  /**
   * Specifies whether the server should send the response set by this
   * plugin to the client immediately with no further processing on
   * the operation.
   *
   * @param  sendResponseImmediately  Indicates whether the server
   *                                  should send the response set by
   *                                  this plugin to the client
   *                                  immediately with no further
   *                                  processing on the operation.
   */
  public void setSendResponseImmediately(
                   boolean sendResponseImmediately)
  {
    assert debugEnter(CLASS_NAME, "setSendResponseImmediately",
                      String.valueOf(sendResponseImmediately));

    this.sendResponseImmediately = sendResponseImmediately;
  }



  /**
   * Retrieves a string representation of this post-response plugin
   * result.
   *