Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NULL contextValue in RepositoryRetrieveProvider:retrieve #268

Closed
wants to merge 9 commits into from

Conversation

dkayiwa
Copy link

@dkayiwa dkayiwa commented Mar 10, 2023

This pull request is for helping me find out why contextValue is always NULL in the RepositoryRetrieveProvider:retrieve() method callback.

@dkayiwa
Copy link
Author

dkayiwa commented Mar 14, 2023

Thanks @brynrhodes and team for the great work on this. I confirm that after replacing ANCContactDataElements library with the one that you updated, i am able to see the PatientId coming through appropriately in the retrieve, with the expected contextValue.

But now the output careplan has changed from 3a28c95 to this: https://pastebin.com/Lf1jDRXn

I did not expect the output care plan to be different because i have not changed the data bundle. The only thing that changed is the ANCContactDataElements library.

Did you expect your changes to affect the output careplan?

dkayiwa referenced this pull request Mar 28, 2023
* add check method overload

* method overload test

* update test

* verify signature before and after translation

* slim fixture

* consider argument size in function overload check
@dkayiwa
Copy link
Author

dkayiwa commented Mar 29, 2023

@c-schuler this is the pull request having the test to reproduce the java.lang.IllegalArgumentException: Found multiple versions / instances of library FHIRHelpers. error.

@JPercival
Copy link
Contributor

Did this get resolved for you?

@dkayiwa
Copy link
Author

dkayiwa commented Aug 1, 2023

Did this get resolved for you?

Yes it did.

@JPercival JPercival closed this Aug 3, 2023
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.

None yet

2 participants