svn commit: r1331856 - /oodt/trunk/workflow/src/main/java/org/apache/oodt/cas/workflow/structs/Workflow.java

[ma...@apache.org]

Author: mattmann
Date: Sun Apr 29 00:10:04 2012
New Revision: 1331856

URL: http://svn.apache.org/viewvc?rev=1331856&view=rev
Log:
- can't return a new list of combined pre- and post- conditions. Need to return preConditions ref, as was expected semantics by callers before. OODT-215 and OODT-310 WIP.

Modified:
    oodt/trunk/workflow/src/main/java/org/apache/oodt/cas/workflow/structs/Workflow.java

Modified: oodt/trunk/workflow/src/main/java/org/apache/oodt/cas/workflow/structs/Workflow.java
URL: http://svn.apache.org/viewvc/oodt/trunk/workflow/src/main/java/org/apache/oodt/cas/workflow/structs/Workflow.java?rev=1331856&r1=1331855&r2=1331856&view=diff
==============================================================================
--- oodt/trunk/workflow/src/main/java/org/apache/oodt/cas/workflow/structs/Workflow.java (original)
+++ oodt/trunk/workflow/src/main/java/org/apache/oodt/cas/workflow/structs/Workflow.java Sun Apr 29 00:10:04 2012
@@ -148,16 +148,15 @@ public class Workflow {
 
   /**
    * Deprecated. Currently, this will return a list
-   * of all pre- and post- {@link WorkflowCondition}s.
+   * of all pre- {@link WorkflowCondition}s. The list
+   * is mutable, and can change the inner portion of the
+   * class. 
    * 
-   * @return All pre-and-post conditions.
+   * @return All pre- conditions.
    */
   @Deprecated
   public List<WorkflowCondition> getConditions() {
-    List<WorkflowCondition> allConds = new Vector<WorkflowCondition>();
-    allConds.addAll(preConditions);
-    allConds.addAll(this.postConditions);
-    return allConds;
+    return this.preConditions;
   }
 
   /**