Better error handling

bbimber · bbimber · commit 13ea5d6df4a6 · 2021-02-17T10:13:31.000-08:00
diff --git a/SequenceAnalysis/src/org/labkey/sequenceanalysis/SequenceAnalysisController.java b/SequenceAnalysis/src/org/labkey/sequenceanalysis/SequenceAnalysisController.java
@@ -4053,6 +4053,11 @@ public ApiResponse execute(RunSequenceHandlerForm form, BindException errors) th
                 errors.reject(ERROR_MSG, "Unable to parse JSON params: " + e.getMessage());
                 return null;
             }
+            catch (IllegalArgumentException e)
+            {
+                errors.reject(ERROR_MSG, e.getMessage());
+                return null;
+            }
         }
 
         protected PipelineJob createOutputJob(RunSequenceHandlerForm form, Container targetContainer, String jobName, PipeRoot pr1, SequenceOutputHandler handler, List<SequenceOutputFile> inputs, JSONObject json) throws IOException, PipelineJobException

Original file line number	Diff line number	Diff line change
`@@ -4053,6 +4053,11 @@ public ApiResponse execute(RunSequenceHandlerForm form, BindException errors) th`
`4053`	`4053`	`errors.reject(ERROR_MSG, "Unable to parse JSON params: " + e.getMessage());`
`4054`	`4054`	`return null;`
`4055`	`4055`	`}`
	`4056`	`+ catch (IllegalArgumentException e)`
	`4057`	`+ {`
	`4058`	`+ errors.reject(ERROR_MSG, e.getMessage());`
	`4059`	`+ return null;`
	`4060`	`+ }`
`4056`	`4061`	`}`
`4057`	`4062`
`4058`	`4063`	`protected PipelineJob createOutputJob(RunSequenceHandlerForm form, Container targetContainer, String jobName, PipeRoot pr1, SequenceOutputHandler handler, List<SequenceOutputFile> inputs, JSONObject json) throws IOException, PipelineJobException`