REPORT-899 ReportLoader should not set parameters to required=false b…

…y default (#246)
openmrs · Aug 9, 2023 · f5f3cfc · f5f3cfc
1 parent 237d85c
commit f5f3cfc
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/api/src/main/java/org/openmrs/module/reporting/config/ReportLoader.java b/api/src/main/java/org/openmrs/module/reporting/config/ReportLoader.java
@@ -129,7 +129,7 @@ public static List<Parameter> constructParameters(List<ParameterDescriptor> para
                     p.setLabel(parameterDescriptor.getLabel());
                     p.setType(parameterType);
                     p.setDefaultValue(getParameterValue(parameterType, parameterDescriptor.getValue()));
-                    p.setRequired(BooleanUtils.isTrue(parameterDescriptor.getRequired()));
+                    p.setRequired(BooleanUtils.isNotFalse(parameterDescriptor.getRequired()));
                     parameters.add(p);
                 } catch (Exception e) {
                     throw new RuntimeException("Unable to configure parameter " + parameterDescriptor.getKey(), e);