Second bugfix to support for long-string multi-value params

bbimber · bbimber · commit 778ef907826d · 2022-05-24T07:51:48.000-07:00
diff --git a/singlecell/api-src/org/labkey/api/singlecell/pipeline/AbstractSingleCellPipelineStep.java b/singlecell/api-src/org/labkey/api/singlecell/pipeline/AbstractSingleCellPipelineStep.java
@@ -371,16 +371,15 @@ else if ("sequenceanalysis-trimmingtextarea".equals(pd.getFieldXtype()))
         {
             val = val.replace("'", "\\\'");
             serializeMultiValueParam(pd, body, val);
-            return;
-
         }
         else if (pd.isMultiValue())
         {
             serializeMultiValueParam(pd, body, val);
-            return;
         }
-
-        body.add((pd.getVariableName() + " <- '" + val + "'"));
+        else
+        {
+            body.add((pd.getVariableName() + " <- '" + val + "'"));
+        }
     }
 
     private void serializeMultiValueParam(SeuratToolParameter pd, List<String> body, String val)

Original file line number	Diff line number	Diff line change
`@@ -371,16 +371,15 @@ else if ("sequenceanalysis-trimmingtextarea".equals(pd.getFieldXtype()))`
`371`	`371`	`{`
`372`	`372`	`val = val.replace("'", "\\\'");`
`373`	`373`	`serializeMultiValueParam(pd, body, val);`
`374`		`- return;`
`375`		`-`
`376`	`374`	`}`
`377`	`375`	`else if (pd.isMultiValue())`
`378`	`376`	`{`
`379`	`377`	`serializeMultiValueParam(pd, body, val);`
`380`		`- return;`
`381`	`378`	`}`
`382`		`-`
`383`		`- body.add((pd.getVariableName() + " <- '" + val + "'"));`
	`379`	`+ else`
	`380`	`+ {`
	`381`	`+ body.add((pd.getVariableName() + " <- '" + val + "'"));`
	`382`	`+ }`
`384`	`383`	`}`
`385`	`384`
`386`	`385`	`private void serializeMultiValueParam(SeuratToolParameter pd, List<String> body, String val)`