diff --git a/org.hl7.fhir.r4/src/main/java/org/hl7/fhir/r4/fhirpath/ExpressionNode.java b/org.hl7.fhir.r4/src/main/java/org/hl7/fhir/r4/fhirpath/ExpressionNode.java
index 66977c737..a2bf898b7 100644
--- a/org.hl7.fhir.r4/src/main/java/org/hl7/fhir/r4/fhirpath/ExpressionNode.java
+++ b/org.hl7.fhir.r4/src/main/java/org/hl7/fhir/r4/fhirpath/ExpressionNode.java
@@ -50,7 +50,7 @@ public class ExpressionNode {
Empty, Not, Exists, SubsetOf, SupersetOf, IsDistinct, Distinct, Count, Where, Select, All, Repeat, Aggregate,
Item /* implicit from name[] */, As, Is, Single, First, Last, Tail, Skip, Take, Union, Combine, Intersect, Exclude,
Iif, Upper, Lower, ToChars, IndexOf, Substring, StartsWith, EndsWith, Matches, MatchesFull, ReplaceMatches,
- Contains, Replace, Length, Children, Descendants, MemberOf, Trace, Check, Today, Now, Resolve, Extension, AllFalse,
+ Contains, Replace, Length, Children, Descendants, MemberOf, Trace, DefineVariable, Check, Today, Now, Resolve, Extension, AllFalse,
AnyFalse, AllTrue, AnyTrue, HasValue, OfType, Type, ConvertsToBoolean, ConvertsToInteger, ConvertsToString,
ConvertsToDecimal, ConvertsToQuantity, ConvertsToDateTime, ConvertsToDate, ConvertsToTime, ToBoolean, ToInteger,
ToString, ToDecimal, ToQuantity, ToDateTime, ToTime, ConformsTo, Round, Sqrt, Abs, Ceiling, Exp, Floor, Ln, Log,
@@ -60,7 +60,7 @@ public class ExpressionNode {
Encode, Decode, Escape, Unescape, Trim, Split, Join, LowBoundary, HighBoundary, Precision,
// Local extensions to FHIRPath
- HtmlChecks1, HtmlChecks2, AliasAs, Alias, Comparable;
+ HtmlChecks1, HtmlChecks2, Comparable;
public static Function fromCode(String name) {
if (name.equals("empty"))
@@ -151,6 +151,8 @@ public class ExpressionNode {
return Function.MemberOf;
if (name.equals("trace"))
return Function.Trace;
+ if (name.equals("defineVariable"))
+ return Function.DefineVariable;
if (name.equals("check"))
return Function.Check;
if (name.equals("today"))
@@ -171,10 +173,6 @@ public class ExpressionNode {
return Function.AnyTrue;
if (name.equals("hasValue"))
return Function.HasValue;
- if (name.equals("alias"))
- return Function.Alias;
- if (name.equals("aliasAs"))
- return Function.AliasAs;
if (name.equals("htmlChecks"))
return Function.HtmlChecks1;
if (name.equals("htmlchecks"))
@@ -353,6 +351,8 @@ public class ExpressionNode {
return "memberOf";
case Trace:
return "trace";
+ case DefineVariable :
+ return "defineVariable";
case Check:
return "check";
case Today:
@@ -373,10 +373,6 @@ public class ExpressionNode {
return "anyTrue";
case HasValue:
return "hasValue";
- case Alias:
- return "alias";
- case AliasAs:
- return "aliasAs";
case Encode:
return "encode";
case Decode:
diff --git a/org.hl7.fhir.r4/src/main/java/org/hl7/fhir/r4/fhirpath/FHIRPathEngine.java b/org.hl7.fhir.r4/src/main/java/org/hl7/fhir/r4/fhirpath/FHIRPathEngine.java
index c91f19dae..a23cb6c79 100644
--- a/org.hl7.fhir.r4/src/main/java/org/hl7/fhir/r4/fhirpath/FHIRPathEngine.java
+++ b/org.hl7.fhir.r4/src/main/java/org/hl7/fhir/r4/fhirpath/FHIRPathEngine.java
@@ -139,6 +139,7 @@ public class FHIRPathEngine {
/**
* A constant reference - e.g. a reference to a name that must be resolved in
* context. The % will be removed from the constant name before this is invoked.
+ * Variables created with defineVariable will not be processed by resolveConstant (or resolveConstantType)
*
* This will also be called if the host invokes the FluentPath engine with a
* context of null
@@ -604,7 +605,7 @@ public class FHIRPathEngine {
}
log = new StringBuilder();
return execute(new ExecutionContext(null, base != null && base.isResource() ? base : null,
- base != null && base.isResource() ? base : null, base, null, base), list, ExpressionNode, true);
+ base != null && base.isResource() ? base : null, base, base), list, ExpressionNode, true);
}
/**
@@ -623,7 +624,7 @@ public class FHIRPathEngine {
}
log = new StringBuilder();
return execute(
- new ExecutionContext(null, base.isResource() ? base : null, base.isResource() ? base : null, base, null, base),
+ new ExecutionContext(null, base.isResource() ? base : null, base.isResource() ? base : null, base, base),
list, exp, true);
}
@@ -642,7 +643,7 @@ public class FHIRPathEngine {
list.add(base);
}
log = new StringBuilder();
- return execute(new ExecutionContext(appContext, focusResource, rootResource, base, null, base), list,
+ return execute(new ExecutionContext(appContext, focusResource, rootResource, base, base), list,
ExpressionNode, true);
}
@@ -661,7 +662,7 @@ public class FHIRPathEngine {
list.add(base);
}
log = new StringBuilder();
- return execute(new ExecutionContext(appContext, focusResource, rootResource, base, null, base), list,
+ return execute(new ExecutionContext(appContext, focusResource, rootResource, base, base), list,
expressionNode, true);
}
@@ -681,7 +682,7 @@ public class FHIRPathEngine {
list.add(base);
}
log = new StringBuilder();
- return execute(new ExecutionContext(appContext, focusResource, rootResource, base, null, base), list, exp, true);
+ return execute(new ExecutionContext(appContext, focusResource, rootResource, base, base), list, exp, true);
}
/**
@@ -848,16 +849,14 @@ public class FHIRPathEngine {
private Base context;
private Base thisItem;
private List total;
- private Map aliases;
private int index;
+ private Map> definedVariables;
- public ExecutionContext(Object appInfo, Base resource, Base rootResource, Base context, Map aliases,
- Base thisItem) {
+ public ExecutionContext(Object appInfo, Base resource, Base rootResource, Base context, Base thisItem) {
this.appInfo = appInfo;
this.context = context;
this.focusResource = resource;
this.rootResource = rootResource;
- this.aliases = aliases;
this.thisItem = thisItem;
this.index = 0;
}
@@ -886,26 +885,34 @@ public class FHIRPathEngine {
return new IntegerType(index);
}
- public void addAlias(String name, List focus) throws FHIRException {
- if (aliases == null) {
- aliases = new HashMap();
- } else {
- aliases = new HashMap(aliases); // clone it, since it's going to change
- }
- if (focus.size() > 1) {
- throw makeException(null, I18nConstants.FHIRPATH_ALIAS_COLLECTION);
- }
- aliases.put(name, focus.size() == 0 ? null : focus.get(0));
- }
-
- public Base getAlias(String name) {
- return aliases == null ? null : aliases.get(name);
- }
-
public ExecutionContext setIndex(int i) {
index = i;
return this;
}
+
+ public boolean hasDefinedVariable(String name) {
+ return definedVariables != null && definedVariables.containsKey(name);
+ }
+
+ public List getDefinedVariable(String name) {
+ return definedVariables == null ? makeNull() : definedVariables.get(name);
+ }
+
+ public void setDefinedVariable(String name, List value) {
+ if (isSystemVariable(name))
+ throw new PathEngineException(worker.formatMessage(I18nConstants.FHIRPATH_REDEFINE_VARIABLE, name), I18nConstants.FHIRPATH_REDEFINE_VARIABLE);
+
+ if (definedVariables == null) {
+ definedVariables = new HashMap>();
+ } else {
+ if (definedVariables.containsKey(name)) {
+ // Can't do this, so throw an error
+ throw new PathEngineException(worker.formatMessage(I18nConstants.FHIRPATH_REDEFINE_VARIABLE, name), I18nConstants.FHIRPATH_REDEFINE_VARIABLE);
+ }
+ }
+
+ definedVariables.put(name, value);
+ }
}
private class ExecutionTypeContext {
@@ -914,6 +921,7 @@ public class FHIRPathEngine {
private TypeDetails context;
private TypeDetails thisItem;
private TypeDetails total;
+ private Map definedVariables;
public ExecutionTypeContext(Object appInfo, String resource, TypeDetails context, TypeDetails thisItem) {
super();
@@ -932,6 +940,29 @@ public class FHIRPathEngine {
return thisItem;
}
+ public boolean hasDefinedVariable(String name) {
+ return definedVariables != null && definedVariables.containsKey(name);
+ }
+
+ public TypeDetails getDefinedVariable(String name) {
+ return definedVariables == null ? null : definedVariables.get(name);
+ }
+
+ public void setDefinedVariable(String name, TypeDetails value) {
+ if (isSystemVariable(name))
+ throw new PathEngineException("Redefine of variable "+name, I18nConstants.FHIRPATH_REDEFINE_VARIABLE);
+
+ if (definedVariables == null) {
+ definedVariables = new HashMap();
+ } else {
+ if (definedVariables.containsKey(name)) {
+ // Can't do this, so throw an error
+ throw new PathEngineException("Redefine of variable "+name, I18nConstants.FHIRPATH_REDEFINE_VARIABLE);
+ }
+ }
+
+ definedVariables.put(name, value);
+ }
}
private ExpressionNode parseExpression(FHIRLexer lexer, boolean proximal) throws FHIRLexerException {
@@ -1333,6 +1364,8 @@ public class FHIRPathEngine {
return checkParamCount(lexer, location, exp, 1);
case Trace:
return checkParamCount(lexer, location, exp, 1, 2);
+ case DefineVariable:
+ return checkParamCount(lexer, location, exp, 1, 2);
case Check:
return checkParamCount(lexer, location, exp, 2);
case Today:
@@ -1353,10 +1386,6 @@ public class FHIRPathEngine {
return checkParamCount(lexer, location, exp, 0);
case HasValue:
return checkParamCount(lexer, location, exp, 0);
- case Alias:
- return checkParamCount(lexer, location, exp, 1);
- case AliasAs:
- return checkParamCount(lexer, location, exp, 1);
case Encode:
return checkParamCount(lexer, location, exp, 1);
case Decode:
@@ -1442,9 +1471,10 @@ public class FHIRPathEngine {
return false;
}
- private List execute(ExecutionContext context, List focus, ExpressionNode exp, boolean atEntry)
+ private List execute(ExecutionContext inContext, List focus, ExpressionNode exp, boolean atEntry)
throws FHIRException {
// System.out.println("Evaluate {'"+exp.toString()+"'} on "+focus.toString());
+ ExecutionContext context = contextForParameter(inContext);
List work = new ArrayList();
switch (exp.getKind()) {
case Unary:
@@ -1488,6 +1518,7 @@ public class FHIRPathEngine {
ExpressionNode next = exp.getOpNext();
ExpressionNode last = exp;
while (next != null) {
+ context = contextForParameter(inContext);
List work2 = preOperate(work, last.getOperation(), exp);
if (work2 != null) {
work = work2;
@@ -1551,9 +1582,10 @@ public class FHIRPathEngine {
return new TypeDetails(CollectionStatus.SINGLETON, exp.getName());
}
- private TypeDetails executeType(ExecutionTypeContext context, TypeDetails focus, ExpressionNode exp, boolean atEntry)
+ private TypeDetails executeType(ExecutionTypeContext inContext, TypeDetails focus, ExpressionNode exp, boolean atEntry)
throws PathEngineException, DefinitionException {
- TypeDetails result = new TypeDetails(null);
+ ExecutionTypeContext context = contextForParameter(inContext);
+ TypeDetails result = new TypeDetails(null);
switch (exp.getKind()) {
case Name:
if (atEntry && exp.getName().equals("$this")) {
@@ -1597,6 +1629,7 @@ public class FHIRPathEngine {
ExpressionNode next = exp.getOpNext();
ExpressionNode last = exp;
while (next != null) {
+ context = contextForParameter(inContext);
TypeDetails work;
if (last.getOperation() == Operation.Is || last.getOperation() == Operation.As) {
work = executeTypeName(context, focus, next, atEntry);
@@ -1621,6 +1654,10 @@ public class FHIRPathEngine {
}
FHIRConstant c = (FHIRConstant) constant;
if (c.getValue().startsWith("%")) {
+ String varName = c.getValue().substring(1);
+ if (context.hasDefinedVariable(varName)) {
+ return context.getDefinedVariable(varName);
+ }
return resolveConstant(context, c.getValue(), beforeContext, expr, explicitConstant);
} else if (c.getValue().startsWith("@")) {
return new ArrayList(Arrays.asList(processDateConstant(context.appInfo, c.getValue().substring(1), expr)));
@@ -1691,6 +1728,22 @@ public class FHIRPathEngine {
}
}
+ static boolean isSystemVariable(String name){
+ if (name.equals("sct"))
+ return true;
+ if (name.equals("loinc"))
+ return true;
+ if (name.equals("ucum"))
+ return true;
+ if (name.equals("resource"))
+ return true;
+ if (name.equals("rootResource"))
+ return true;
+ if (name.equals("context"))
+ return true;
+ return false;
+ }
+
private List resolveConstant(ExecutionContext context, String s, boolean beforeContext, ExpressionNode expr, boolean explicitConstant)
throws PathEngineException {
if (s.equals("%sct")) {
@@ -3134,8 +3187,14 @@ public class FHIRPathEngine {
} else if (s.startsWith("%`ext-")) {
return new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String);
} else if (hostServices == null) {
+ String varName = s.substring(1);
+ if (context.hasDefinedVariable(varName))
+ return context.getDefinedVariable(varName);
throw makeException(expr, I18nConstants.FHIRPATH_UNKNOWN_CONSTANT, s);
} else {
+ String varName = s.substring(1);
+ if (context.hasDefinedVariable(varName))
+ return context.getDefinedVariable(varName);
return hostServices.resolveConstantType(this, context.appInfo, s, explicitConstant);
}
}
@@ -3432,6 +3491,25 @@ public class FHIRPathEngine {
new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String));
return focus;
}
+ case DefineVariable : {
+ checkParamTypes(exp, exp.getFunction().toCode(), paramTypes, new TypeDetails(CollectionStatus.UNORDERED, TypeDetails.FP_String));
+ // set the type of the variable
+ // Actually evaluate the value of the first parameter (to get the name of the variable if possible)
+ // and if have that, set it into the context
+ ExpressionNode p = exp.getParameters().get(0);
+ if (p.getKind() == Kind.Constant && p.getConstant() != null) {
+ String varName = exp.getParameters().get(0).getConstant().primitiveValue();
+ if (varName != null) {
+ if (paramTypes.size() > 1)
+ context.setDefinedVariable(varName, paramTypes.get(1));
+ else
+ context.setDefinedVariable(varName, focus);
+ }
+ } else {
+ // this variable is not a constant, so we can't analyze what name it could have
+ }
+ return focus;
+ }
case Check: {
checkParamTypes(exp, exp.getFunction().toCode(), paramTypes,
new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String));
@@ -3466,14 +3544,6 @@ public class FHIRPathEngine {
return new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_Boolean);
case Comparable:
return new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_Boolean);
- case Alias:
- checkParamTypes(exp, exp.getFunction().toCode(), paramTypes,
- new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String));
- return anything(CollectionStatus.SINGLETON);
- case AliasAs:
- checkParamTypes(exp, exp.getFunction().toCode(), paramTypes,
- new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String));
- return focus;
case Encode:
checkParamTypes(exp, exp.getFunction().toCode(), paramTypes,
new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String));
@@ -3623,7 +3693,7 @@ public class FHIRPathEngine {
|| exp.getFunction() == Function.All || exp.getFunction() == Function.Select
|| exp.getFunction() == Function.Repeat || exp.getFunction() == Function.Aggregate;
case 1:
- return exp.getFunction() == Function.Trace;
+ return exp.getFunction() == Function.Trace || exp.getFunction() == Function.DefineVariable;
default:
return false;
}
@@ -3831,6 +3901,8 @@ public class FHIRPathEngine {
return funcMemberOf(context, focus, exp);
case Trace:
return funcTrace(context, focus, exp);
+ case DefineVariable:
+ return funcDefineVariable(context, focus, exp);
case Check:
return funcCheck(context, focus, exp);
case Today:
@@ -3851,8 +3923,6 @@ public class FHIRPathEngine {
return funcAllTrue(context, focus, exp);
case HasValue:
return funcHasValue(context, focus, exp);
- case AliasAs:
- return funcAliasAs(context, focus, exp);
case Encode:
return funcEncode(context, focus, exp);
case Decode:
@@ -3867,8 +3937,6 @@ public class FHIRPathEngine {
return funcSplit(context, focus, exp);
case Join:
return funcJoin(context, focus, exp);
- case Alias:
- return funcAlias(context, focus, exp);
case HtmlChecks1:
return funcHtmlChecks1(context, focus, exp);
case HtmlChecks2:
@@ -4417,24 +4485,6 @@ public class FHIRPathEngine {
return result;
}
- private List funcAliasAs(ExecutionContext context, List focus, ExpressionNode exp) throws FHIRException {
- List nl = execute(context, focus, exp.getParameters().get(0), true);
- String name = nl.get(0).primitiveValue();
- context.addAlias(name, focus);
- return focus;
- }
-
- private List funcAlias(ExecutionContext context, List focus, ExpressionNode exp) throws FHIRException {
- List nl = execute(context, focus, exp.getParameters().get(0), true);
- String name = nl.get(0).primitiveValue();
- List res = new ArrayList();
- Base b = context.getAlias(name);
- if (b != null) {
- res.add(b);
- }
- return res;
- }
-
private List funcHtmlChecks1(ExecutionContext context, List focus, ExpressionNode exp)
throws FHIRException {
// todo: actually check the HTML
@@ -4585,12 +4635,50 @@ public class FHIRPathEngine {
}
private ExecutionContext changeThis(ExecutionContext context, Base newThis) {
- return new ExecutionContext(context.appInfo, context.focusResource, context.rootResource, context.context,
- context.aliases, newThis);
+ ExecutionContext newContext = new ExecutionContext(context.appInfo, context.focusResource, context.rootResource, context.context,
+ newThis);
+ // append all of the defined variables from the context into the new context
+ if (context.definedVariables != null) {
+ for (String s : context.definedVariables.keySet()) {
+ newContext.setDefinedVariable(s, context.definedVariables.get(s));
+ }
+ }
+ return newContext;
+ }
+
+ private ExecutionContext contextForParameter(ExecutionContext context) {
+ ExecutionContext newContext = new ExecutionContext(context.appInfo, context.focusResource, context.rootResource, context.context, context.thisItem);
+ newContext.total = context.total;
+ newContext.index = context.index;
+ // append all of the defined variables from the context into the new context
+ if (context.definedVariables != null) {
+ for (String s : context.definedVariables.keySet()) {
+ newContext.setDefinedVariable(s, context.definedVariables.get(s));
+ }
+ }
+ return newContext;
}
private ExecutionTypeContext changeThis(ExecutionTypeContext context, TypeDetails newThis) {
- return new ExecutionTypeContext(context.appInfo, context.resource, context.context, newThis);
+ ExecutionTypeContext newContext = new ExecutionTypeContext(context.appInfo, context.resource, context.context, newThis);
+ // append all of the defined variables from the context into the new context
+ if (context.definedVariables != null) {
+ for (String s : context.definedVariables.keySet()) {
+ newContext.setDefinedVariable(s, context.definedVariables.get(s));
+ }
+ }
+ return newContext;
+ }
+
+ private ExecutionTypeContext contextForParameter(ExecutionTypeContext context) {
+ ExecutionTypeContext newContext = new ExecutionTypeContext(context.appInfo, context.resource, context.context, context.thisItem);
+ // append all of the defined variables from the context into the new context
+ if (context.definedVariables != null) {
+ for (String s : context.definedVariables.keySet()) {
+ newContext.setDefinedVariable(s, context.definedVariables.get(s));
+ }
+ }
+ return newContext;
}
private List funcNow(ExecutionContext context, List focus, ExpressionNode exp) {
@@ -5417,6 +5505,20 @@ public class FHIRPathEngine {
return focus;
}
+ private List funcDefineVariable(ExecutionContext context, List focus, ExpressionNode exp) throws FHIRException {
+ List nl = execute(context, focus, exp.getParameters().get(0), true);
+ String name = nl.get(0).primitiveValue();
+ List value;
+ if (exp.getParameters().size() == 2) {
+ value = execute(context, focus, exp.getParameters().get(1), true);
+ } else {
+ value = focus;
+ }
+ // stash the variable into the context
+ context.setDefinedVariable(name, value);
+ return focus;
+ }
+
private List funcCheck(ExecutionContext context, List focus, ExpressionNode expr) throws FHIRException {
List n1 = execute(context, focus, expr.getParameters().get(0), true);
if (!convertToBoolean(n1)) {
diff --git a/org.hl7.fhir.r4b/src/main/java/org/hl7/fhir/r4b/fhirpath/ExpressionNode.java b/org.hl7.fhir.r4b/src/main/java/org/hl7/fhir/r4b/fhirpath/ExpressionNode.java
index d79e026ce..c9731541a 100644
--- a/org.hl7.fhir.r4b/src/main/java/org/hl7/fhir/r4b/fhirpath/ExpressionNode.java
+++ b/org.hl7.fhir.r4b/src/main/java/org/hl7/fhir/r4b/fhirpath/ExpressionNode.java
@@ -50,7 +50,7 @@ public class ExpressionNode {
Empty, Not, Exists, SubsetOf, SupersetOf, IsDistinct, Distinct, Count, Where, Select, All, Repeat, Aggregate,
Item /* implicit from name[] */, As, Is, Single, First, Last, Tail, Skip, Take, Union, Combine, Intersect, Exclude,
Iif, Upper, Lower, ToChars, IndexOf, Substring, StartsWith, EndsWith, Matches, MatchesFull, ReplaceMatches,
- Contains, Replace, Length, Children, Descendants, MemberOf, Trace, Check, Today, Now, Resolve, Extension, AllFalse,
+ Contains, Replace, Length, Children, Descendants, MemberOf, Trace, DefineVariable, Check, Today, Now, Resolve, Extension, AllFalse,
AnyFalse, AllTrue, AnyTrue, HasValue, OfType, Type, ConvertsToBoolean, ConvertsToInteger, ConvertsToString,
ConvertsToDecimal, ConvertsToQuantity, ConvertsToDateTime, ConvertsToDate, ConvertsToTime, ToBoolean, ToInteger,
ToString, ToDecimal, ToQuantity, ToDateTime, ToTime, ConformsTo, Round, Sqrt, Abs, Ceiling, Exp, Floor, Ln, Log,
@@ -60,7 +60,7 @@ public class ExpressionNode {
Encode, Decode, Escape, Unescape, Trim, Split, Join, LowBoundary, HighBoundary, Precision,
// Local extensions to FHIRPath
- HtmlChecks1, HtmlChecks2, AliasAs, Alias, Comparable;
+ HtmlChecks1, HtmlChecks2, Comparable;
public static Function fromCode(String name) {
if (name.equals("empty"))
@@ -151,6 +151,8 @@ public class ExpressionNode {
return Function.MemberOf;
if (name.equals("trace"))
return Function.Trace;
+ if (name.equals("defineVariable"))
+ return Function.DefineVariable;
if (name.equals("check"))
return Function.Check;
if (name.equals("today"))
@@ -171,10 +173,6 @@ public class ExpressionNode {
return Function.AnyTrue;
if (name.equals("hasValue"))
return Function.HasValue;
- if (name.equals("alias"))
- return Function.Alias;
- if (name.equals("aliasAs"))
- return Function.AliasAs;
if (name.equals("htmlChecks"))
return Function.HtmlChecks1;
if (name.equals("htmlchecks"))
@@ -353,6 +351,8 @@ public class ExpressionNode {
return "memberOf";
case Trace:
return "trace";
+ case DefineVariable :
+ return "defineVariable";
case Check:
return "check";
case Today:
@@ -373,10 +373,6 @@ public class ExpressionNode {
return "anyTrue";
case HasValue:
return "hasValue";
- case Alias:
- return "alias";
- case AliasAs:
- return "aliasAs";
case Encode:
return "encode";
case Decode:
diff --git a/org.hl7.fhir.r4b/src/main/java/org/hl7/fhir/r4b/fhirpath/FHIRPathEngine.java b/org.hl7.fhir.r4b/src/main/java/org/hl7/fhir/r4b/fhirpath/FHIRPathEngine.java
index 81fe330f8..15c9b4c7a 100644
--- a/org.hl7.fhir.r4b/src/main/java/org/hl7/fhir/r4b/fhirpath/FHIRPathEngine.java
+++ b/org.hl7.fhir.r4b/src/main/java/org/hl7/fhir/r4b/fhirpath/FHIRPathEngine.java
@@ -140,6 +140,7 @@ public class FHIRPathEngine {
/**
* A constant reference - e.g. a reference to a name that must be resolved in
* context. The % will be removed from the constant name before this is invoked.
+ * Variables created with defineVariable will not be processed by resolveConstant (or resolveConstantType)
*
* This will also be called if the host invokes the FluentPath engine with a
* context of null
@@ -605,7 +606,7 @@ public class FHIRPathEngine {
}
log = new StringBuilder();
return execute(new ExecutionContext(null, base != null && base.isResource() ? base : null,
- base != null && base.isResource() ? base : null, base, null, base), list, ExpressionNode, true);
+ base != null && base.isResource() ? base : null, base, base), list, ExpressionNode, true);
}
/**
@@ -624,7 +625,7 @@ public class FHIRPathEngine {
}
log = new StringBuilder();
return execute(
- new ExecutionContext(null, base.isResource() ? base : null, base.isResource() ? base : null, base, null, base),
+ new ExecutionContext(null, base.isResource() ? base : null, base.isResource() ? base : null, base, base),
list, exp, true);
}
@@ -643,7 +644,7 @@ public class FHIRPathEngine {
list.add(base);
}
log = new StringBuilder();
- return execute(new ExecutionContext(appContext, focusResource, rootResource, base, null, base), list,
+ return execute(new ExecutionContext(appContext, focusResource, rootResource, base, base), list,
ExpressionNode, true);
}
@@ -662,7 +663,7 @@ public class FHIRPathEngine {
list.add(base);
}
log = new StringBuilder();
- return execute(new ExecutionContext(appContext, focusResource, rootResource, base, null, base), list,
+ return execute(new ExecutionContext(appContext, focusResource, rootResource, base, base), list,
expressionNode, true);
}
@@ -682,7 +683,7 @@ public class FHIRPathEngine {
list.add(base);
}
log = new StringBuilder();
- return execute(new ExecutionContext(appContext, focusResource, rootResource, base, null, base), list, exp, true);
+ return execute(new ExecutionContext(appContext, focusResource, rootResource, base, base), list, exp, true);
}
/**
@@ -849,16 +850,14 @@ public class FHIRPathEngine {
private Base context;
private Base thisItem;
private List total;
- private Map aliases;
private int index;
+ private Map> definedVariables;
- public ExecutionContext(Object appInfo, Base resource, Base rootResource, Base context, Map aliases,
- Base thisItem) {
+ public ExecutionContext(Object appInfo, Base resource, Base rootResource, Base context, Base thisItem) {
this.appInfo = appInfo;
this.context = context;
this.focusResource = resource;
this.rootResource = rootResource;
- this.aliases = aliases;
this.thisItem = thisItem;
this.index = 0;
}
@@ -887,26 +886,34 @@ public class FHIRPathEngine {
return new IntegerType(index);
}
- public void addAlias(String name, List focus) throws FHIRException {
- if (aliases == null) {
- aliases = new HashMap();
- } else {
- aliases = new HashMap(aliases); // clone it, since it's going to change
- }
- if (focus.size() > 1) {
- throw makeException(null, I18nConstants.FHIRPATH_ALIAS_COLLECTION);
- }
- aliases.put(name, focus.size() == 0 ? null : focus.get(0));
- }
-
- public Base getAlias(String name) {
- return aliases == null ? null : aliases.get(name);
- }
-
public ExecutionContext setIndex(int i) {
index = i;
return this;
}
+
+ public boolean hasDefinedVariable(String name) {
+ return definedVariables != null && definedVariables.containsKey(name);
+ }
+
+ public List getDefinedVariable(String name) {
+ return definedVariables == null ? makeNull() : definedVariables.get(name);
+ }
+
+ public void setDefinedVariable(String name, List value) {
+ if (isSystemVariable(name))
+ throw new PathEngineException(worker.formatMessage(I18nConstants.FHIRPATH_REDEFINE_VARIABLE, name), I18nConstants.FHIRPATH_REDEFINE_VARIABLE);
+
+ if (definedVariables == null) {
+ definedVariables = new HashMap>();
+ } else {
+ if (definedVariables.containsKey(name)) {
+ // Can't do this, so throw an error
+ throw new PathEngineException(worker.formatMessage(I18nConstants.FHIRPATH_REDEFINE_VARIABLE, name), I18nConstants.FHIRPATH_REDEFINE_VARIABLE);
+ }
+ }
+
+ definedVariables.put(name, value);
+ }
}
private class ExecutionTypeContext {
@@ -915,6 +922,7 @@ public class FHIRPathEngine {
private TypeDetails context;
private TypeDetails thisItem;
private TypeDetails total;
+ private Map definedVariables;
public ExecutionTypeContext(Object appInfo, String resource, TypeDetails context, TypeDetails thisItem) {
super();
@@ -933,6 +941,29 @@ public class FHIRPathEngine {
return thisItem;
}
+ public boolean hasDefinedVariable(String name) {
+ return definedVariables != null && definedVariables.containsKey(name);
+ }
+
+ public TypeDetails getDefinedVariable(String name) {
+ return definedVariables == null ? null : definedVariables.get(name);
+ }
+
+ public void setDefinedVariable(String name, TypeDetails value) {
+ if (isSystemVariable(name))
+ throw new PathEngineException("Redefine of variable "+name, I18nConstants.FHIRPATH_REDEFINE_VARIABLE);
+
+ if (definedVariables == null) {
+ definedVariables = new HashMap();
+ } else {
+ if (definedVariables.containsKey(name)) {
+ // Can't do this, so throw an error
+ throw new PathEngineException("Redefine of variable "+name, I18nConstants.FHIRPATH_REDEFINE_VARIABLE);
+ }
+ }
+
+ definedVariables.put(name, value);
+ }
}
private ExpressionNode parseExpression(FHIRLexer lexer, boolean proximal) throws FHIRLexerException {
@@ -1333,6 +1364,8 @@ public class FHIRPathEngine {
return checkParamCount(lexer, location, exp, 1);
case Trace:
return checkParamCount(lexer, location, exp, 1, 2);
+ case DefineVariable:
+ return checkParamCount(lexer, location, exp, 1, 2);
case Check:
return checkParamCount(lexer, location, exp, 2);
case Today:
@@ -1353,10 +1386,6 @@ public class FHIRPathEngine {
return checkParamCount(lexer, location, exp, 0);
case HasValue:
return checkParamCount(lexer, location, exp, 0);
- case Alias:
- return checkParamCount(lexer, location, exp, 1);
- case AliasAs:
- return checkParamCount(lexer, location, exp, 1);
case Encode:
return checkParamCount(lexer, location, exp, 1);
case Decode:
@@ -1442,9 +1471,10 @@ public class FHIRPathEngine {
return false;
}
- private List execute(ExecutionContext context, List focus, ExpressionNode exp, boolean atEntry)
+ private List execute(ExecutionContext inContext, List focus, ExpressionNode exp, boolean atEntry)
throws FHIRException {
// System.out.println("Evaluate {'"+exp.toString()+"'} on "+focus.toString());
+ ExecutionContext context = contextForParameter(inContext);
List work = new ArrayList();
switch (exp.getKind()) {
case Unary:
@@ -1488,6 +1518,7 @@ public class FHIRPathEngine {
ExpressionNode next = exp.getOpNext();
ExpressionNode last = exp;
while (next != null) {
+ context = contextForParameter(inContext);
List work2 = preOperate(work, last.getOperation(), exp);
if (work2 != null) {
work = work2;
@@ -1551,8 +1582,9 @@ public class FHIRPathEngine {
return new TypeDetails(CollectionStatus.SINGLETON, exp.getName());
}
- private TypeDetails executeType(ExecutionTypeContext context, TypeDetails focus, ExpressionNode exp, boolean atEntry)
+ private TypeDetails executeType(ExecutionTypeContext inContext, TypeDetails focus, ExpressionNode exp, boolean atEntry)
throws PathEngineException, DefinitionException {
+ ExecutionTypeContext context = contextForParameter(inContext);
TypeDetails result = new TypeDetails(null);
switch (exp.getKind()) {
case Name:
@@ -1597,6 +1629,7 @@ public class FHIRPathEngine {
ExpressionNode next = exp.getOpNext();
ExpressionNode last = exp;
while (next != null) {
+ context = contextForParameter(inContext);
TypeDetails work;
if (last.getOperation() == Operation.Is || last.getOperation() == Operation.As) {
work = executeTypeName(context, focus, next, atEntry);
@@ -1622,6 +1655,10 @@ public class FHIRPathEngine {
}
FHIRConstant c = (FHIRConstant) constant;
if (c.getValue().startsWith("%")) {
+ String varName = c.getValue().substring(1);
+ if (context.hasDefinedVariable(varName)) {
+ return context.getDefinedVariable(varName);
+ }
return resolveConstant(context, c.getValue(), beforeContext, expr, explicitConstant);
} else if (c.getValue().startsWith("@")) {
return new ArrayList(Arrays.asList(processDateConstant(context.appInfo, c.getValue().substring(1), expr)));
@@ -1692,6 +1729,22 @@ public class FHIRPathEngine {
}
}
+ static boolean isSystemVariable(String name){
+ if (name.equals("sct"))
+ return true;
+ if (name.equals("loinc"))
+ return true;
+ if (name.equals("ucum"))
+ return true;
+ if (name.equals("resource"))
+ return true;
+ if (name.equals("rootResource"))
+ return true;
+ if (name.equals("context"))
+ return true;
+ return false;
+ }
+
private List resolveConstant(ExecutionContext context, String s, boolean beforeContext, ExpressionNode expr, boolean explicitConstant)
throws PathEngineException {
if (s.equals("%sct")) {
@@ -3136,8 +3189,14 @@ public class FHIRPathEngine {
} else if (s.startsWith("%`ext-")) {
return new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String);
} else if (hostServices == null) {
+ String varName = s.substring(1);
+ if (context.hasDefinedVariable(varName))
+ return context.getDefinedVariable(varName);
throw makeException(expr, I18nConstants.FHIRPATH_UNKNOWN_CONSTANT, s);
} else {
+ String varName = s.substring(1);
+ if (context.hasDefinedVariable(varName))
+ return context.getDefinedVariable(varName);
return hostServices.resolveConstantType(this, context.appInfo, s, explicitConstant);
}
}
@@ -3434,6 +3493,25 @@ public class FHIRPathEngine {
new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String));
return focus;
}
+ case DefineVariable : {
+ checkParamTypes(exp, exp.getFunction().toCode(), paramTypes, new TypeDetails(CollectionStatus.UNORDERED, TypeDetails.FP_String));
+ // set the type of the variable
+ // Actually evaluate the value of the first parameter (to get the name of the variable if possible)
+ // and if have that, set it into the context
+ ExpressionNode p = exp.getParameters().get(0);
+ if (p.getKind() == Kind.Constant && p.getConstant() != null) {
+ String varName = exp.getParameters().get(0).getConstant().primitiveValue();
+ if (varName != null) {
+ if (paramTypes.size() > 1)
+ context.setDefinedVariable(varName, paramTypes.get(1));
+ else
+ context.setDefinedVariable(varName, focus);
+ }
+ } else {
+ // this variable is not a constant, so we can't analyze what name it could have
+ }
+ return focus;
+ }
case Check: {
checkParamTypes(exp, exp.getFunction().toCode(), paramTypes,
new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String));
@@ -3468,14 +3546,6 @@ public class FHIRPathEngine {
return new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_Boolean);
case Comparable:
return new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_Boolean);
- case Alias:
- checkParamTypes(exp, exp.getFunction().toCode(), paramTypes,
- new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String));
- return anything(CollectionStatus.SINGLETON);
- case AliasAs:
- checkParamTypes(exp, exp.getFunction().toCode(), paramTypes,
- new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String));
- return focus;
case Encode:
checkParamTypes(exp, exp.getFunction().toCode(), paramTypes,
new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String));
@@ -3625,7 +3695,7 @@ public class FHIRPathEngine {
|| exp.getFunction() == Function.All || exp.getFunction() == Function.Select
|| exp.getFunction() == Function.Repeat || exp.getFunction() == Function.Aggregate;
case 1:
- return exp.getFunction() == Function.Trace;
+ return exp.getFunction() == Function.Trace || exp.getFunction() == Function.DefineVariable;
default:
return false;
}
@@ -3833,6 +3903,8 @@ public class FHIRPathEngine {
return funcMemberOf(context, focus, exp);
case Trace:
return funcTrace(context, focus, exp);
+ case DefineVariable:
+ return funcDefineVariable(context, focus, exp);
case Check:
return funcCheck(context, focus, exp);
case Today:
@@ -3853,8 +3925,6 @@ public class FHIRPathEngine {
return funcAllTrue(context, focus, exp);
case HasValue:
return funcHasValue(context, focus, exp);
- case AliasAs:
- return funcAliasAs(context, focus, exp);
case Encode:
return funcEncode(context, focus, exp);
case Decode:
@@ -3869,8 +3939,6 @@ public class FHIRPathEngine {
return funcSplit(context, focus, exp);
case Join:
return funcJoin(context, focus, exp);
- case Alias:
- return funcAlias(context, focus, exp);
case HtmlChecks1:
return funcHtmlChecks1(context, focus, exp);
case HtmlChecks2:
@@ -4427,24 +4495,6 @@ public class FHIRPathEngine {
return result;
}
- private List funcAliasAs(ExecutionContext context, List focus, ExpressionNode exp) throws FHIRException {
- List nl = execute(context, focus, exp.getParameters().get(0), true);
- String name = nl.get(0).primitiveValue();
- context.addAlias(name, focus);
- return focus;
- }
-
- private List funcAlias(ExecutionContext context, List focus, ExpressionNode exp) throws FHIRException {
- List nl = execute(context, focus, exp.getParameters().get(0), true);
- String name = nl.get(0).primitiveValue();
- List res = new ArrayList();
- Base b = context.getAlias(name);
- if (b != null) {
- res.add(b);
- }
- return res;
- }
-
private List funcHtmlChecks1(ExecutionContext context, List focus, ExpressionNode exp)
throws FHIRException {
// todo: actually check the HTML
@@ -4587,12 +4637,50 @@ public class FHIRPathEngine {
}
private ExecutionContext changeThis(ExecutionContext context, Base newThis) {
- return new ExecutionContext(context.appInfo, context.focusResource, context.rootResource, context.context,
- context.aliases, newThis);
+ ExecutionContext newContext = new ExecutionContext(context.appInfo, context.focusResource, context.rootResource, context.context,
+ newThis);
+ // append all of the defined variables from the context into the new context
+ if (context.definedVariables != null) {
+ for (String s : context.definedVariables.keySet()) {
+ newContext.setDefinedVariable(s, context.definedVariables.get(s));
+ }
+ }
+ return newContext;
+ }
+
+ private ExecutionContext contextForParameter(ExecutionContext context) {
+ ExecutionContext newContext = new ExecutionContext(context.appInfo, context.focusResource, context.rootResource, context.context, context.thisItem);
+ newContext.total = context.total;
+ newContext.index = context.index;
+ // append all of the defined variables from the context into the new context
+ if (context.definedVariables != null) {
+ for (String s : context.definedVariables.keySet()) {
+ newContext.setDefinedVariable(s, context.definedVariables.get(s));
+ }
+ }
+ return newContext;
}
private ExecutionTypeContext changeThis(ExecutionTypeContext context, TypeDetails newThis) {
- return new ExecutionTypeContext(context.appInfo, context.resource, context.context, newThis);
+ ExecutionTypeContext newContext = new ExecutionTypeContext(context.appInfo, context.resource, context.context, newThis);
+ // append all of the defined variables from the context into the new context
+ if (context.definedVariables != null) {
+ for (String s : context.definedVariables.keySet()) {
+ newContext.setDefinedVariable(s, context.definedVariables.get(s));
+ }
+ }
+ return newContext;
+ }
+
+ private ExecutionTypeContext contextForParameter(ExecutionTypeContext context) {
+ ExecutionTypeContext newContext = new ExecutionTypeContext(context.appInfo, context.resource, context.context, context.thisItem);
+ // append all of the defined variables from the context into the new context
+ if (context.definedVariables != null) {
+ for (String s : context.definedVariables.keySet()) {
+ newContext.setDefinedVariable(s, context.definedVariables.get(s));
+ }
+ }
+ return newContext;
}
private List funcNow(ExecutionContext context, List focus, ExpressionNode exp) {
@@ -5420,6 +5508,20 @@ public class FHIRPathEngine {
return focus;
}
+ private List funcDefineVariable(ExecutionContext context, List focus, ExpressionNode exp) throws FHIRException {
+ List nl = execute(context, focus, exp.getParameters().get(0), true);
+ String name = nl.get(0).primitiveValue();
+ List value;
+ if (exp.getParameters().size() == 2) {
+ value = execute(context, focus, exp.getParameters().get(1), true);
+ } else {
+ value = focus;
+ }
+ // stash the variable into the context
+ context.setDefinedVariable(name, value);
+ return focus;
+ }
+
private List funcCheck(ExecutionContext context, List focus, ExpressionNode expr) throws FHIRException {
List n1 = execute(context, focus, expr.getParameters().get(0), true);
if (!convertToBoolean(n1)) {
diff --git a/org.hl7.fhir.r5/src/main/java/org/hl7/fhir/r5/fhirpath/ExpressionNode.java b/org.hl7.fhir.r5/src/main/java/org/hl7/fhir/r5/fhirpath/ExpressionNode.java
index 1571c7407..2b5a33033 100644
--- a/org.hl7.fhir.r5/src/main/java/org/hl7/fhir/r5/fhirpath/ExpressionNode.java
+++ b/org.hl7.fhir.r5/src/main/java/org/hl7/fhir/r5/fhirpath/ExpressionNode.java
@@ -59,7 +59,7 @@ public class ExpressionNode {
Encode, Decode, Escape, Unescape, Trim, Split, Join, LowBoundary, HighBoundary, Precision,
// Local extensions to FHIRPath
- HtmlChecks1, HtmlChecks2, AliasAs, Alias, Comparable, hasTemplateIdOf;
+ HtmlChecks1, HtmlChecks2, Comparable, hasTemplateIdOf;
public static Function fromCode(String name) {
if (name.equals("empty")) return Function.Empty;
@@ -117,8 +117,6 @@ public class ExpressionNode {
if (name.equals("allTrue")) return Function.AllTrue;
if (name.equals("anyTrue")) return Function.AnyTrue;
if (name.equals("hasValue")) return Function.HasValue;
- if (name.equals("alias")) return Function.Alias;
- if (name.equals("aliasAs")) return Function.AliasAs;
if (name.equals("htmlChecks")) return Function.HtmlChecks1;
if (name.equals("htmlchecks")) return Function.HtmlChecks1; // support change of care from R3
if (name.equals("htmlChecks2")) return Function.HtmlChecks2;
@@ -223,8 +221,6 @@ public class ExpressionNode {
case AllTrue : return "allTrue";
case AnyTrue : return "anyTrue";
case HasValue : return "hasValue";
- case Alias : return "alias";
- case AliasAs : return "aliasAs";
case Encode : return "encode";
case Decode : return "decode";
case Escape : return "escape";
diff --git a/org.hl7.fhir.r5/src/main/java/org/hl7/fhir/r5/fhirpath/FHIRPathEngine.java b/org.hl7.fhir.r5/src/main/java/org/hl7/fhir/r5/fhirpath/FHIRPathEngine.java
index 28506b2d3..7630b8f06 100644
--- a/org.hl7.fhir.r5/src/main/java/org/hl7/fhir/r5/fhirpath/FHIRPathEngine.java
+++ b/org.hl7.fhir.r5/src/main/java/org/hl7/fhir/r5/fhirpath/FHIRPathEngine.java
@@ -770,7 +770,7 @@ public class FHIRPathEngine {
list.add(base);
}
log = new StringBuilder();
- return execute(new ExecutionContext(null, base != null && base.isResource() ? base : null, base != null && base.isResource() ? base : null, base, null, base), list, ExpressionNode, true);
+ return execute(new ExecutionContext(null, base != null && base.isResource() ? base : null, base != null && base.isResource() ? base : null, base, base), list, ExpressionNode, true);
}
/**
@@ -789,7 +789,7 @@ public class FHIRPathEngine {
list.add(base);
}
log = new StringBuilder();
- return execute(new ExecutionContext(null, base.isResource() ? base : null, base.isResource() ? base : null, base, null, base), list, exp, true);
+ return execute(new ExecutionContext(null, base.isResource() ? base : null, base.isResource() ? base : null, base, base), list, exp, true);
}
/**
@@ -807,7 +807,7 @@ public class FHIRPathEngine {
list.add(base);
}
log = new StringBuilder();
- return execute(new ExecutionContext(appContext, focusResource, rootResource, base, null, base), list, ExpressionNode, true);
+ return execute(new ExecutionContext(appContext, focusResource, rootResource, base, base), list, ExpressionNode, true);
}
/**
@@ -825,7 +825,7 @@ public class FHIRPathEngine {
list.add(base);
}
log = new StringBuilder();
- return execute(new ExecutionContext(appContext, focusResource, rootResource, base, null, base), list, expressionNode, true);
+ return execute(new ExecutionContext(appContext, focusResource, rootResource, base, base), list, expressionNode, true);
}
/**
@@ -844,7 +844,7 @@ public class FHIRPathEngine {
list.add(base);
}
log = new StringBuilder();
- return execute(new ExecutionContext(appContext, focusResource, rootResource, base, null, base), list, exp, true);
+ return execute(new ExecutionContext(appContext, focusResource, rootResource, base, base), list, exp, true);
}
/**
@@ -1009,16 +1009,14 @@ public class FHIRPathEngine {
private Base context;
private Base thisItem;
private List total;
- private Map aliases;
private int index;
private Map> definedVariables;
- public ExecutionContext(Object appInfo, Base resource, Base rootResource, Base context, Map aliases, Base thisItem) {
+ public ExecutionContext(Object appInfo, Base resource, Base rootResource, Base context, Base thisItem) {
this.appInfo = appInfo;
this.context = context;
this.focusResource = resource;
this.rootResource = rootResource;
- this.aliases = aliases;
this.thisItem = thisItem;
this.index = 0;
}
@@ -1042,20 +1040,6 @@ public class FHIRPathEngine {
return new IntegerType(index);
}
- public void addAlias(String name, List focus) throws FHIRException {
- if (aliases == null) {
- aliases = new HashMap();
- } else {
- aliases = new HashMap(aliases); // clone it, since it's going to change
- }
- if (focus.size() > 1) {
- throw makeException(null, I18nConstants.FHIRPATH_ALIAS_COLLECTION);
- }
- aliases.put(name, focus.size() == 0 ? null : focus.get(0));
- }
- public Base getAlias(String name) {
- return aliases == null ? null : aliases.get(name);
- }
public ExecutionContext setIndex(int i) {
index = i;
return this;
@@ -1064,9 +1048,11 @@ public class FHIRPathEngine {
public boolean hasDefinedVariable(String name) {
return definedVariables != null && definedVariables.containsKey(name);
}
+
public List getDefinedVariable(String name) {
return definedVariables == null ? makeNull() : definedVariables.get(name);
}
+
public void setDefinedVariable(String name, List value) {
if (isSystemVariable(name))
throw new PathEngineException(worker.formatMessage(I18nConstants.FHIRPATH_REDEFINE_VARIABLE, name), I18nConstants.FHIRPATH_REDEFINE_VARIABLE);
@@ -1110,9 +1096,11 @@ public class FHIRPathEngine {
public boolean hasDefinedVariable(String name) {
return definedVariables != null && definedVariables.containsKey(name);
}
+
public TypeDetails getDefinedVariable(String name) {
return definedVariables == null ? null : definedVariables.get(name);
}
+
public void setDefinedVariable(String name, TypeDetails value) {
if (isSystemVariable(name))
throw new PathEngineException("Redefine of variable "+name, I18nConstants.FHIRPATH_REDEFINE_VARIABLE);
@@ -1478,8 +1466,6 @@ public class FHIRPathEngine {
case AllTrue: return checkParamCount(lexer, location, exp, 0);
case AnyTrue: return checkParamCount(lexer, location, exp, 0);
case HasValue: return checkParamCount(lexer, location, exp, 0);
- case Alias: return checkParamCount(lexer, location, exp, 1);
- case AliasAs: return checkParamCount(lexer, location, exp, 1);
case Encode: return checkParamCount(lexer, location, exp, 1);
case Decode: return checkParamCount(lexer, location, exp, 1);
case Escape: return checkParamCount(lexer, location, exp, 1);
@@ -1552,7 +1538,7 @@ public class FHIRPathEngine {
}
break;
case Function:
- List work2 = evaluateFunction("aliasAs".equals(exp.getName()) ? inContext : context, focus, exp);
+ List work2 = evaluateFunction(context, focus, exp);
work.addAll(work2);
break;
case Constant:
@@ -3216,12 +3202,14 @@ public class FHIRPathEngine {
} else if (s.startsWith("%`ext-")) {
return new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String);
} else if (hostServices == null) {
+ String varName = s.substring(1);
+ if (context.hasDefinedVariable(varName))
+ return context.getDefinedVariable(varName);
throw makeException(expr, I18nConstants.FHIRPATH_UNKNOWN_CONSTANT, s);
} else {
String varName = s.substring(1);
- if (context.hasDefinedVariable(varName)) {
+ if (context.hasDefinedVariable(varName))
return context.getDefinedVariable(varName);
- }
TypeDetails v = hostServices.resolveConstantType(this, context.appInfo, s, explicitConstant);
if (v == null) {
throw makeException(expr, I18nConstants.FHIRPATH_UNKNOWN_CONSTANT, s);
@@ -3635,12 +3623,6 @@ public class FHIRPathEngine {
return new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_Boolean);
case Comparable :
return new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_Boolean);
- case Alias :
- checkParamTypes(exp, exp.getFunction().toCode(), paramTypes, new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String));
- return anything(CollectionStatus.SINGLETON);
- case AliasAs :
- checkParamTypes(exp, exp.getFunction().toCode(), paramTypes, new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String));
- return focus;
case Encode:
checkParamTypes(exp, exp.getFunction().toCode(), paramTypes, new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String));
return new TypeDetails(CollectionStatus.SINGLETON, TypeDetails.FP_String);
@@ -3988,7 +3970,6 @@ public class FHIRPathEngine {
case AnyTrue: return funcAnyTrue(context, focus, exp);
case AllTrue: return funcAllTrue(context, focus, exp);
case HasValue : return funcHasValue(context, focus, exp);
- case AliasAs : return funcAliasAs(context, focus, exp);
case Encode : return funcEncode(context, focus, exp);
case Decode : return funcDecode(context, focus, exp);
case Escape : return funcEscape(context, focus, exp);
@@ -3996,7 +3977,6 @@ public class FHIRPathEngine {
case Trim : return funcTrim(context, focus, exp);
case Split : return funcSplit(context, focus, exp);
case Join : return funcJoin(context, focus, exp);
- case Alias : return funcAlias(context, focus, exp);
case HtmlChecks1 : return funcHtmlChecks1(context, focus, exp);
case HtmlChecks2 : return funcHtmlChecks2(context, focus, exp);
case Comparable : return funcComparable(context, focus, exp);
@@ -4567,24 +4547,6 @@ public class FHIRPathEngine {
return result;
}
- private List funcAliasAs(ExecutionContext context, List focus, ExpressionNode exp) throws FHIRException {
- List nl = execute(context, focus, exp.getParameters().get(0), true);
- String name = nl.get(0).primitiveValue();
- context.addAlias(name, focus);
- return focus;
- }
-
- private List funcAlias(ExecutionContext context, List focus, ExpressionNode exp) throws FHIRException {
- List nl = execute(context, focus, exp.getParameters().get(0), true);
- String name = nl.get(0).primitiveValue();
- List res = new ArrayList();
- Base b = context.getAlias(name);
- if (b != null) {
- res.add(b);
- }
- return res;
- }
-
private List funcHtmlChecks1(ExecutionContext context, List focus, ExpressionNode exp) throws FHIRException {
// todo: actually check the HTML
if (focus.size() != 1) {
@@ -4743,7 +4705,7 @@ public class FHIRPathEngine {
private ExecutionContext changeThis(ExecutionContext context, Base newThis) {
- ExecutionContext newContext = new ExecutionContext(context.appInfo, context.focusResource, context.rootResource, context.context, context.aliases, newThis);
+ ExecutionContext newContext = new ExecutionContext(context.appInfo, context.focusResource, context.rootResource, context.context, newThis);
// append all of the defined variables from the context into the new context
if (context.definedVariables != null) {
for (String s : context.definedVariables.keySet()) {
@@ -4754,7 +4716,7 @@ public class FHIRPathEngine {
}
private ExecutionContext contextForParameter(ExecutionContext context) {
- ExecutionContext newContext = new ExecutionContext(context.appInfo, context.focusResource, context.rootResource, context.context, context.aliases, context.thisItem);
+ ExecutionContext newContext = new ExecutionContext(context.appInfo, context.focusResource, context.rootResource, context.context, context.thisItem);
newContext.total = context.total;
newContext.index = context.index;
// append all of the defined variables from the context into the new context