diff --git a/org.hl7.fhir.r5/src/main/java/org/hl7/fhir/r5/utils/sql/Validator.java b/org.hl7.fhir.r5/src/main/java/org/hl7/fhir/r5/utils/sql/Validator.java index 13c132962..b9f136fc5 100644 --- a/org.hl7.fhir.r5/src/main/java/org/hl7/fhir/r5/utils/sql/Validator.java +++ b/org.hl7.fhir.r5/src/main/java/org/hl7/fhir/r5/utils/sql/Validator.java @@ -58,21 +58,21 @@ public class Validator { private FHIRPathEngine fpe; private List prohibitedNames = new ArrayList(); private List issues = new ArrayList(); - private TrueFalseOrUnknown arrays; - private TrueFalseOrUnknown complexTypes; - private TrueFalseOrUnknown needsName; + private TrueFalseOrUnknown supportsArrays; + private TrueFalseOrUnknown supportsComplexTypes; + private TrueFalseOrUnknown supportsNeedsName; private String resourceName; private String name; - public Validator(IWorkerContext context, FHIRPathEngine fpe, List prohibitedNames, TrueFalseOrUnknown arrays, TrueFalseOrUnknown complexTypes, TrueFalseOrUnknown needsName) { + public Validator(IWorkerContext context, FHIRPathEngine fpe, List prohibitedNames, TrueFalseOrUnknown supportsArrays, TrueFalseOrUnknown supportsComplexTypes, TrueFalseOrUnknown supportsNeedsName) { super(); this.context = context; this.fpe = fpe; this.prohibitedNames = prohibitedNames; - this.arrays = arrays; - this.complexTypes = complexTypes; - this.needsName = needsName; + this.supportsArrays = supportsArrays; + this.supportsComplexTypes = supportsComplexTypes; + this.supportsNeedsName = supportsNeedsName; } public String getResourceName() { @@ -85,9 +85,9 @@ public class Validator { JsonElement nameJ = viewDefinition.get("name"); if (nameJ == null) { - if (needsName == null) { + if (supportsNeedsName == null) { hint(path, viewDefinition, "No name provided. A name is required in many contexts where a ViewDefinition is used"); - } else if (needsName == TrueFalseOrUnknown.TRUE) { + } else if (supportsNeedsName == TrueFalseOrUnknown.TRUE) { error(path, viewDefinition, "No name provided", IssueType.REQUIRED); } } else if (!(nameJ instanceof JsonString)) { @@ -339,9 +339,9 @@ public class Validator { hint(path, column, "collection is true, but the path statement(s) can only return single values for the column '"+columnName+"'"); } } else { - if (arrays == TrueFalseOrUnknown.UNKNOWN) { + if (supportsArrays == TrueFalseOrUnknown.UNKNOWN) { warning(path, expression, "The column '"+columnName+"' appears to be a collection based on it's path. Collections are not supported in all execution contexts"); - } else if (arrays == TrueFalseOrUnknown.FALSE) { + } else if (supportsArrays == TrueFalseOrUnknown.FALSE) { warning(path, expression, "The column '"+columnName+"' appears to be a collection based on it's path, but this is not allowed in the current execution context"); } if (td.getCollectionStatus() != CollectionStatus.SINGLETON) { @@ -378,9 +378,9 @@ public class Validator { String type = types.iterator().next(); boolean ok = false; if (!isSimpleType(type) && !"null".equals(type)) { - if (complexTypes == TrueFalseOrUnknown.UNKNOWN) { + if (supportsComplexTypes == TrueFalseOrUnknown.UNKNOWN) { warning(path, expression, "Column is a complex type. This is not supported in some Runners"); - } else if (complexTypes == TrueFalseOrUnknown.FALSE) { + } else if (supportsComplexTypes == TrueFalseOrUnknown.FALSE) { error(path, expression, "Column is a complex type but this is not allowed in this context", IssueType.BUSINESSRULE); } else { ok = true;