Merge pull request #282 from SeqWare/feature/fix_inmemory_workflow_params

Fix in memory workflow params (were not being stored properly)

Author: denis-yuen

seqware-common/src/main/java/net/sourceforge/seqware/common/metadata/MetadataInMemory.java

@@ -70,9 +70,9 @@
 
 /**
  * This stores some metadata in memory as an exploration of running workflows without a running database or web service.
- * 
+ *
  * Data will only be stored while this VM is still active and cannot be accessed by other clients.
- * 
+ *
  * @author dyuen
  */
 public class MetadataInMemory implements Metadata {
@@ -84,7 +84,7 @@ public class MetadataInMemory implements Metadata {
 
     /**
      * Not really thread-safe, why does Guava not have a synchronized wrapper?
-     * 
+     *
      * @return the store
      */
     private static synchronized Table<Integer, Class, Object> getStore() {
@@ -323,7 +323,10 @@ public ReturnValue addWorkflow(String name, String version, String description,
         TreeSet<WorkflowParam> setOfDefaultParams = new TreeSet<>();
         for (Entry<String, Map<String, String>> e : hm.entrySet()) {
             WorkflowParam workflowParam = MetadataWS.convertMapToWorkflowParam(e.getValue(), workflow);
+            int nextSwAccession = getNextSwAccession();
+            workflowParam.setWorkflowParamId(nextSwAccession);
             setOfDefaultParams.add(workflowParam);
+            MetadataInMemory.getStore().put(nextSwAccession, WorkflowParam.class, workflowParam);
         }
         workflow.setWorkflowParams(setOfDefaultParams);
         return returnValue;