FHIR-46548: where clause on alias #1748

This commit is contained in:
oliveregger 2024-09-18 23:07:39 +02:00
parent 0c7e6ce9ac
commit d69a8e3e43
2 changed files with 60 additions and 14 deletions

View File

@ -1075,19 +1075,16 @@ public class StructureMapUtilities {
if (lexer.hasToken("where")) { if (lexer.hasToken("where")) {
lexer.take(); lexer.take();
ExpressionNode node = fpe.parse(lexer); ExpressionNode node = fpe.parse(lexer);
source.setUserData(MAP_WHERE_EXPRESSION, node);
source.setCondition(node.toString()); source.setCondition(node.toString());
} }
if (lexer.hasToken("check")) { if (lexer.hasToken("check")) {
lexer.take(); lexer.take();
ExpressionNode node = fpe.parse(lexer); ExpressionNode node = fpe.parse(lexer);
source.setUserData(MAP_WHERE_CHECK, node);
source.setCheck(node.toString()); source.setCheck(node.toString());
} }
if (lexer.hasToken("log")) { if (lexer.hasToken("log")) {
lexer.take(); lexer.take();
ExpressionNode node = fpe.parse(lexer); ExpressionNode node = fpe.parse(lexer);
source.setUserData(MAP_WHERE_CHECK, node);
source.setLogMessage(node.toString()); source.setLogMessage(node.toString());
} }
} }
@ -1601,6 +1598,7 @@ public class StructureMapUtilities {
ExpressionNode expr = (ExpressionNode) src.getUserData(MAP_SEARCH_EXPRESSION); ExpressionNode expr = (ExpressionNode) src.getUserData(MAP_SEARCH_EXPRESSION);
if (expr == null) { if (expr == null) {
expr = fpe.parse(src.getElement()); expr = fpe.parse(src.getElement());
patchVariablesInExpression(expr, vars);
src.setUserData(MAP_SEARCH_EXPRESSION, expr); src.setUserData(MAP_SEARCH_EXPRESSION, expr);
} }
String search = fpe.evaluateToString(vars, null, null, new StringType(), expr); // string is a holder of nothing to ensure that variables are processed correctly String search = fpe.evaluateToString(vars, null, null, new StringType(), expr); // string is a holder of nothing to ensure that variables are processed correctly
@ -1629,17 +1627,21 @@ public class StructureMapUtilities {
} }
items.removeAll(remove); items.removeAll(remove);
} }
if (src.hasCondition()) { if (src.hasCondition()) {
ExpressionNode expr = (ExpressionNode) src.getUserData(MAP_WHERE_EXPRESSION); ExpressionNode expr = (ExpressionNode) src.getUserData(MAP_WHERE_EXPRESSION);
if (expr == null) { if (expr == null) {
expr = fpe.parse(src.getCondition()); expr = fpe.parse(src.getCondition());
// fpe.check(context.appInfo, ??, ??, expr) patchVariablesInExpression(expr, vars);
src.setUserData(MAP_WHERE_EXPRESSION, expr); src.setUserData(MAP_WHERE_EXPRESSION, expr);
} }
List<Base> remove = new ArrayList<Base>(); List<Base> remove = new ArrayList<Base>();
for (Base item : items) { for (Base item : items) {
if (!fpe.evaluateToBoolean(vars, null, null, item, expr)) { Variables varsForSource = vars.copy();
if (src.hasVariable()) {
varsForSource.add(VariableMode.INPUT, src.getVariable(), item);
}
if (!fpe.evaluateToBoolean(varsForSource, null, null, item, expr)) {
log(indent + " condition [" + src.getCondition() + "] for " + item.toString() + " : false"); log(indent + " condition [" + src.getCondition() + "] for " + item.toString() + " : false");
remove.add(item); remove.add(item);
} else } else
@ -1652,12 +1654,15 @@ public class StructureMapUtilities {
ExpressionNode expr = (ExpressionNode) src.getUserData(MAP_WHERE_CHECK); ExpressionNode expr = (ExpressionNode) src.getUserData(MAP_WHERE_CHECK);
if (expr == null) { if (expr == null) {
expr = fpe.parse(src.getCheck()); expr = fpe.parse(src.getCheck());
// fpe.check(context.appInfo, ??, ??, expr) patchVariablesInExpression(expr, vars);
src.setUserData(MAP_WHERE_CHECK, expr); src.setUserData(MAP_WHERE_CHECK, expr);
} }
List<Base> remove = new ArrayList<Base>();
for (Base item : items) { for (Base item : items) {
if (!fpe.evaluateToBoolean(vars, null, null, item, expr)) Variables varsForSource = vars.copy();
if (src.hasVariable()) {
varsForSource.add(VariableMode.INPUT, src.getVariable(), item);
}
if (!fpe.evaluateToBoolean(varsForSource, null, null, item, expr))
throw new FHIRException("Rule \"" + ruleId + "\": Check condition failed"); throw new FHIRException("Rule \"" + ruleId + "\": Check condition failed");
} }
} }
@ -1666,16 +1671,21 @@ public class StructureMapUtilities {
ExpressionNode expr = (ExpressionNode) src.getUserData(MAP_WHERE_LOG); ExpressionNode expr = (ExpressionNode) src.getUserData(MAP_WHERE_LOG);
if (expr == null) { if (expr == null) {
expr = fpe.parse(src.getLogMessage()); expr = fpe.parse(src.getLogMessage());
// fpe.check(context.appInfo, ??, ??, expr) patchVariablesInExpression(expr, vars);
src.setUserData(MAP_WHERE_LOG, expr); src.setUserData(MAP_WHERE_LOG, expr);
} }
CommaSeparatedStringBuilder b = new CommaSeparatedStringBuilder(); CommaSeparatedStringBuilder b = new CommaSeparatedStringBuilder();
for (Base item : items) for (Base item : items) {
b.appendIfNotNull(fpe.evaluateToString(vars, null, null, item, expr)); Variables varsForSource = vars.copy();
if (src.hasVariable()) {
varsForSource.add(VariableMode.INPUT, src.getVariable(), item);
}
b.appendIfNotNull(fpe.evaluateToString(varsForSource, null, null, item, expr));
}
if (b.length() > 0) if (b.length() > 0)
services.log(b.toString()); services.log(b.toString());
} }
if (src.hasListMode() && !items.isEmpty()) { if (src.hasListMode() && !items.isEmpty()) {
switch (src.getListMode()) { switch (src.getListMode()) {
@ -1753,6 +1763,29 @@ public class StructureMapUtilities {
if (tgt.hasVariable() && v != null) if (tgt.hasVariable() && v != null)
vars.add(VariableMode.OUTPUT, tgt.getVariable(), v); vars.add(VariableMode.OUTPUT, tgt.getVariable(), v);
} }
public static void patchVariablesInExpression(ExpressionNode node, Variables vars) {
if (node.isProximal() && node.getKind() == ExpressionNode.Kind.Name && node.getName() !=null && node.getName().length()>0 && !node.getName().startsWith("%")) {
// Check if this name is in the variables
if (vars.get(VariableMode.INPUT, node.getName())!=null)
node.setName("%" + node.getName());
}
// walk into children
var next = node.getOpNext();
if (next != null)
patchVariablesInExpression(next, vars);
var grp = node.getGroup();
if (grp != null)
patchVariablesInExpression(grp, vars);
var inner = node.getInner();
if (inner != null)
patchVariablesInExpression(inner, vars);
if (node.parameterCount() > 0) {
for(ExpressionNode p : node.getParameters()) {
patchVariablesInExpression(p, vars);
}
}
}
private Base runTransform(String rulePath, TransformContext context, StructureMap map, StructureMapGroupComponent group, StructureMapGroupRuleTargetComponent tgt, Variables vars, Base dest, String element, String srcVar, boolean root) throws FHIRException { private Base runTransform(String rulePath, TransformContext context, StructureMap map, StructureMapGroupComponent group, StructureMapGroupRuleTargetComponent tgt, Variables vars, Base dest, String element, String srcVar, boolean root) throws FHIRException {
try { try {
@ -1793,6 +1826,7 @@ public class StructureMapUtilities {
ExpressionNode expr = (ExpressionNode) tgt.getUserData(MAP_EXPRESSION); ExpressionNode expr = (ExpressionNode) tgt.getUserData(MAP_EXPRESSION);
if (expr == null) { if (expr == null) {
expr = fpe.parse(getParamStringNoNull(vars, tgt.getParameter().get(tgt.getParameter().size() - 1), tgt.toString())); expr = fpe.parse(getParamStringNoNull(vars, tgt.getParameter().get(tgt.getParameter().size() - 1), tgt.toString()));
patchVariablesInExpression(expr, vars);
tgt.setUserData(MAP_EXPRESSION, expr); tgt.setUserData(MAP_EXPRESSION, expr);
} }
List<Base> v = fpe.evaluate(vars, null, null, tgt.getParameter().size() == 2 ? getParam(vars, tgt.getParameter().get(0)) : new BooleanType(false), expr); List<Base> v = fpe.evaluate(vars, null, null, tgt.getParameter().size() == 2 ? getParam(vars, tgt.getParameter().get(0)) : new BooleanType(false), expr);
@ -2520,7 +2554,6 @@ public class StructureMapUtilities {
ExpressionNode expr = (ExpressionNode) tgt.getUserData(MAP_EXPRESSION); ExpressionNode expr = (ExpressionNode) tgt.getUserData(MAP_EXPRESSION);
if (expr == null) { if (expr == null) {
expr = fpe.parse(getParamString(vars, tgt.getParameter().get(tgt.getParameter().size() - 1))); expr = fpe.parse(getParamString(vars, tgt.getParameter().get(tgt.getParameter().size() - 1)));
tgt.setUserData(MAP_WHERE_EXPRESSION, expr);
} }
return fpe.check(vars, null, expr); return fpe.check(vars, null, expr);
case TRANSLATE: case TRANSLATE:

View File

@ -71,6 +71,19 @@ public class StructureMapUtilitiesTest implements ITransformerServices {
assertEquals("2023-10-26", fp.evaluateToString(target, "birthDate")); assertEquals("2023-10-26", fp.evaluateToString(target, "birthDate"));
assertEquals("2023-09-20T13:19:13.502Z", fp.evaluateToString(target, "deceased")); assertEquals("2023-09-20T13:19:13.502Z", fp.evaluateToString(target, "deceased"));
} }
@Test
public void testWhereClause() throws IOException, FHIRException {
StructureMapUtilities scu = new StructureMapUtilities(context, this);
scu.setDebug(true);
String fileMap = TestingUtilities.loadTestResource("r5", "structure-mapping", "whereclause.map");
Element source = Manager.parseSingle(context, TestingUtilities.loadTestResourceStream("r4", "examples", "capabilitystatement-example.json"), FhirFormat.JSON);
StructureMap structureMap = scu.parse(fileMap, "whereclause");
Element target = Manager.build(context, scu.getTargetType(structureMap));
scu.transform(null, source, structureMap, target);
FHIRPathEngine fp = new FHIRPathEngine(context);
assertEquals("true", fp.evaluateToString(target, "rest.resource.interaction.where(code='create').exists()"));
}
private void assertSerializeDeserialize(StructureMap structureMap) { private void assertSerializeDeserialize(StructureMap structureMap) {
Assertions.assertEquals("syntax", structureMap.getName()); Assertions.assertEquals("syntax", structureMap.getName());