Skip to content

example and document for range in sheet #1811 - #44

Open
oliveregger wants to merge 2 commits into
FHIR:masterfrom
ahdis:oe_liquid_sheet_range
Open

example and document for range in sheet #1811#44
oliveregger wants to merge 2 commits into
FHIR:masterfrom
ahdis:oe_liquid_sheet_range

Conversation

@oliveregger

Copy link
Copy Markdown
Contributor

see hapifhir/org.hl7.fhir.core#1881 (other PR's for 1881 need to be done first, otherwise the ci builds fails due to missing fucntionality)

grahamegrieve
grahamegrieve previously approved these changes Jan 15, 2025

@grahamegrieve grahamegrieve left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@lmckenzi

Copy link
Copy Markdown
Collaborator

@oliveregger Is this ok to merge? (The referenced HAPI PR is marked as 'closed')

@oliveregger

Copy link
Copy Markdown
Contributor Author

i need to check, it has been a long time ...

@oliveregger

Copy link
Copy Markdown
Contributor Author

It would be ok to merge, howewer the ig itself is currently building:

Execute Test Data Factory 'LiquidDemo'. Log in LiquidDemo.log (00:00.074 / 00:10.653, 529Mb)
Error running test factory 'LiquidDemo': Error @1, 9: The function "lookup" requires between 4 and 4 parameters
Publishing Content Failed: org.hl7.fhir.r5.fhirpath.FHIRLexer$FHIRLexerException: Error @1, 9: The function "lookup" requires between 4 and 4 parameters (00:00.017 / 00:10.671, 533Mb)
(00:00.000 / 00:10.671, 533Mb)
Use -? to get command line help (00:00.000 / 00:10.672, 533Mb)
(00:00.000 / 00:10.672, 533Mb)
Stack Dump (for debugging): (00:00.000 / 00:10.672, 533Mb)
org.hl7.fhir.exceptions.FHIRException: org.hl7.fhir.r5.fhirpath.FHIRLexer$FHIRLexerException: Error @1, 9: The function "lookup" requires between 4 and 4 parameters
at org.hl7.fhir.r5.testfactory.TestDataFactory.executeLiquid(TestDataFactory.java:424)
at org.hl7.fhir.r5.testfactory.TestDataFactory.execute(TestDataFactory.java:239)
at org.hl7.fhir.igtools.publisher.PublisherIGLoader.processFactories(PublisherIGLoader.java:4269)
at org.hl7.fhir.igtools.publisher.PublisherIGLoader.load(PublisherIGLoader.java:2036)
at org.hl7.fhir.igtools.publisher.Publisher.createIg(Publisher.java:386)
at org.hl7.fhir.igtools.publisher.Publisher.execute(Publisher.java:232)
at org.hl7.fhir.igtools.publisher.Publisher.main(Publisher.java:1579)

and this will persist with the upate above, it looks like another PR is need that the liquid example can use the lookup information, i will have another look.

@oliveregger

Copy link
Copy Markdown
Contributor Author

if it the example is adapted to the new syntax the following exception will appear:

org.hl7.fhir.exceptions.FHIRException: java.lang.NullPointerException: Cannot invoke "java.util.Map.get(Object)" because the return value of "org.hl7.fhir.r5.liquid.BaseTableWrapper.getTables()" is null
at org.hl7.fhir.r5.testfactory.TestDataFactory.executeLiquid(TestDataFactory.java:424)
at org.hl7.fhir.r5.testfactory.TestDataFactory.execute(TestDataFactory.java:239)
at org.hl7.fhir.igtools.publisher.PublisherIGLoader.processFactories(PublisherIGLoader.java:4269)
at org.hl7.fhir.igtools.publisher.PublisherIGLoader.load(PublisherIGLoader.java:2036)
at org.hl7.fhir.igtools.publisher.Publisher.createIg(Publisher.java:386)
at org.hl7.fhir.igtools.publisher.Publisher.execute(Publisher.java:232)
at org.hl7.fhir.igtools.publisher.Publisher.main(Publisher.java:1579)
Caused by: java.lang.NullPointerException: Cannot invoke "java.util.Map.get(Object)" because the return value of "org.hl7.fhir.r5.liquid.BaseTableWrapper.getTables()" is null
at org.hl7.fhir.r5.testfactory.TestDataFactory$TableLookupFunction.execute(TestDataFactory.java:185)
at org.hl7.fhir.r5.fhirpath.BaseHostServices.executeFunction(BaseHostServices.java:40)
at org.hl7.fhir.r5.liquid.LiquidEngine.executeFunction(LiquidEngine.java:977)
at org.hl7.fhir.r5.fhirpath.FHIRPathEngine.evaluateFunction(FHIRPathEngine.java:4225)
at org.hl7.fhir.r5.fhirpath.FHIRPathEngine.execute(FHIRPathEngine.java:1552)
at org.hl7.fhir.r5.fhirpath.FHIRPathEngine.evaluate(FHIRPathEngine.java:827)
at org.hl7.fhir.r5.liquid.LiquidEngine$LiquidStatement.evaluate(LiquidEngine.java:341)
at org.hl7.fhir.r5.liquid.LiquidEngine.evaluate(LiquidEngine.java:239)
at org.hl7.fhir.r5.testfactory.TestDataFactory.runInstance(TestDataFactory.java:442)
at org.hl7.fhir.r5.testfactory.TestDataFactory.runBundle(TestDataFactory.java:456)
at org.hl7.fhir.r5.testfactory.TestDataFactory.executeLiquid(TestDataFactory.java:412)

@oliveregger
oliveregger force-pushed the oe_liquid_sheet_range branch from 96ad7f4 to d48e79f Compare January 6, 2026 14:17
oliveregger added a commit to ahdis/org.hl7.fhir.core that referenced this pull request Jan 6, 2026
oliveregger added a commit to ahdis/org.hl7.fhir.core that referenced this pull request Jan 6, 2026
@oliveregger

Copy link
Copy Markdown
Contributor Author

@lmckenzi if hapifhir/org.hl7.fhir.core#2274 is merged it should work with this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants