Fixes #5127 mapped uuids not saving in registry #5139
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Made it so the uuids for mapped resources such as vitals are being populated in the uuid registry. Users will need to run the sql_upgrade.php for any existing installation in order for the vitals to get their registry populated.
|
bonus: the populateAllMissingUuids function is actually called by an OpenEMR service every 4 hours, so should take care of the missing uuid's silently without the user even knowing it :) |
bradymiller
approved these changes
Apr 12, 2022
adunsulag
added a commit
to DiscoverAndChange/openemr
that referenced
this pull request
Apr 15, 2022
Made it so the uuids for mapped resources such as vitals are being populated in the uuid registry. Users will need to run the sql_upgrade.php for any existing installation in order for the vitals to get their registry populated.
adunsulag
added a commit
that referenced
this pull request
Apr 16, 2022
* Fixes #5095 Patient,Goals,Immunization (#5097) Not sure what happened but it looks like provider_uuid got stomped on in CarePlan and Immunization. Adding that in to our uuid mappings fixes the problem with the resources returning 500. For Patient it looks like the patient/* context was broken in the Patient resource endpoint when we refactored the parameter name for documentation purposes. I've updated the $id parameter in the patient route to be $uuid. * Openemr fhir single patient api #5122 (#5129) * Fix SurgeryService query. FHIR searchAll was relying on a username column that doesn't exist. Not sure if some code got stomped on or how this ever passed our inferno test suite. * Fix Provenance queries. Again not sure if code got stomped on but DiagnosticReports and DocumentReference and Immunization Provenance were not working with single patient api. * FHIR document how medication works inside request Added comments that explain how medication is embedded via the code property inside of MedicationRequest. * Fixes #5125,#5123 FHIR ids,Provenance Reworked the FHIR ids for Provenance,CarePlan, and Goals. FHIR R4.0.1 changed the logical id allowed fields (or at least inferno started validating them) so we had to adjust how we handle these resources. I use a different surrogate key string which is valid in 4.0.1. I also handle the prior key data so we can still read and return the old format so people's data doesn't break. I also fix the issue where clinical notes,procedures and immunization weren't returning provenance resources * Fix style issues. * Fix Care Plan unit tests. We had too many fixture interdependencies now that the careplan also joins on the forms table. Since I was testing the surrogate key logic I stopped the getOne test and just focused on the surrogateKey. * Fixes #5137 Vitals save data absent value (#5138) Make it so we can save a 0 value for columns in vitals to signify the data is missing. * Fixes #5127 mapped uuids not saving in registry (#5139) Made it so the uuids for mapped resources such as vitals are being populated in the uuid registry. Users will need to run the sql_upgrade.php for any existing installation in order for the vitals to get their registry populated.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #5127 Made it so the uuids for mapped resources such as vitals are being populated in the uuid registry. Users will need to run the sql_upgrade.php for any existing installation in order for any new vitals to be retrievable via FHIR for the single resource GET operation.