Bugfix to PedigreeToolParameterDescriptor

bbimber · bbimber · commit bc29d9ffa78f · 2024-12-18T15:53:02.000-08:00
diff --git a/SequenceAnalysis/api-src/org/labkey/api/sequenceanalysis/pipeline/PedigreeToolParameterDescriptor.java b/SequenceAnalysis/api-src/org/labkey/api/sequenceanalysis/pipeline/PedigreeToolParameterDescriptor.java
@@ -13,11 +13,12 @@ public PedigreeToolParameterDescriptor()
         this(true);
     }
 
-    public PedigreeToolParameterDescriptor(boolean isRequired)
+    public PedigreeToolParameterDescriptor(final boolean isRequired)
     {
-        super(null, NAME, "Pedigree Source", "This is the table used for pedigree data", "laboratory-pedigreeselectorfield", "laboratory.subjects", new JSONObject());
+        super(null, NAME, "Pedigree Source", "This is the table used for pedigree data", "laboratory-pedigreeselectorfield", "laboratory.subjects", new JSONObject(){{
+            put("allowBlank", !isRequired);
+        }});
 
-        getAdditionalExtConfig().put("allowBlank", isRequired);
         _isRequired = isRequired;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -13,11 +13,12 @@ public PedigreeToolParameterDescriptor()`
`13`	`13`	`this(true);`
`14`	`14`	`}`
`15`	`15`
`16`		`- public PedigreeToolParameterDescriptor(boolean isRequired)`
	`16`	`+ public PedigreeToolParameterDescriptor(final boolean isRequired)`
`17`	`17`	`{`
`18`		`- super(null, NAME, "Pedigree Source", "This is the table used for pedigree data", "laboratory-pedigreeselectorfield", "laboratory.subjects", new JSONObject());`
	`18`	`+ super(null, NAME, "Pedigree Source", "This is the table used for pedigree data", "laboratory-pedigreeselectorfield", "laboratory.subjects", new JSONObject(){{`
	`19`	`+ put("allowBlank", !isRequired);`
	`20`	`+ }});`
`19`	`21`
`20`		`- getAdditionalExtConfig().put("allowBlank", isRequired);`
`21`	`22`	`_isRequired = isRequired;`
`22`	`23`	`}`
`23`	`24`