-
Notifications
You must be signed in to change notification settings - Fork 498
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
Fix history with PATCH in STU3 #2177
Fix history with PATCH in STU3 #2177
Conversation
resource.FullUrlElement = new FhirUri(_urlResolver.ResolveResourceWrapperUrl(r.Resource, true)); | ||
resource.Request = new Bundle.RequestComponent | ||
if (hasVerb) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we throw an error if hasVerb is false, or will be it caught in the switch case below every time?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Switch statement wouldn't throw exception because it's out parameter and it must be filled with something and for enums it's probably first thing in enum, which is Http.Get.
I'm reluctant to throw, since I'm not sure in which cases we don't have method, so I'll just put creation of response and request under if statement.
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Description
In STU3 PATCH doesn't exist in BundleVerb which leads to null ref because request is empty.
So substitute it to PUT.
Related issues
Addresses https://microsofthealth.visualstudio.com/Health/_workitems/edit/84461/
Testing
Manually and updated unit tests.
FHIR Team Checklist
Semver Change (docs)
Patch