Skip to content

Commit b36dcc2

Browse files
committed
Fix NPE
1 parent 4fba550 commit b36dcc2

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

SequenceAnalysis/api-src/org/labkey/api/sequenceanalysis/pipeline/PipelineStepProvider.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,11 @@ public interface PipelineStepProvider<StepType extends PipelineStep>
8383
*/
8484
public ToolParameterDescriptor getParameterByName(String name);
8585

86+
default boolean hasParameter(String name)
87+
{
88+
return getParameterByName(name) != null;
89+
}
90+
8691
/**
8792
* Creates the JSON object sent to the client that is used to build the client UI
8893
*/

singlecell/src/org/labkey/singlecell/analysis/AbstractSingleCellHandler.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ public void init(JobContext ctx, List<SequenceOutputFile> inputFiles, List<Recor
184184

185185
if (step.requiresHashing(ctx))
186186
{
187-
String methods = step.getProvider().getParameterByName("methods").extractValue(ctx.getJob(), step.getProvider(), step.getStepIdx(), String.class);
187+
String methods = step.getProvider().hasParameter("methods") ? step.getProvider().getParameterByName("methods").extractValue(ctx.getJob(), step.getProvider(), step.getStepIdx(), String.class) : null;
188188
if (methods != null && methods.contains(CellHashingService.CALLING_METHOD.demuxem.name()))
189189
{
190190
doH5Caching = true;

0 commit comments

Comments
 (0)