Skip to:

  1. Skip to Jira Navigation
  2. Skip to Side Navigation
  3. Skip to Main Content

Welcome to our new JIRA instance! We migrated all OpenMRS ID accounts and issues to this new cloud instance. Please use your_openmrsid@id.openmrs.org as your e-mail to sign in.

concepts used in person_attribute are saved with their uuid and no more as id

Description

After migration from 2.40.0 to 2.42.0 we notice that concept are now saved as uuid and no more with their id.
In image:

If we register a patient with this kind of configuration:

{ "id": "countryOfOrigin", "type": "person attribute", "uuid": "9895c7d0-f4eb-4f0b-8413-86f293700f5c", "answerConceptSetUuid": "165657AAAAAAAAAAAAAAAAAAAAAAAAAAAAAA", "validation": { "required": false } }, { "id": "civilStatus", "type": "person attribute", "uuid": "c126fdda-b6a9-4742-b990-beb9ab26661a", "answerConceptSetUuid": "1054AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA", "validation": { "required": false } },

the civilStatus and countryOfOrigin values will be saved with the concept uuid and not with the concept_id

It’s a direct consequence from https://openmrs.atlassian.net/browse/RESTWS-901 and if we rollback this commit
A proposition could be to rollback this commit

Attachments

1

discovered while testing

Activity

Show:

Frédéric Deniger March 15, 2024 at 5:35 PM

Yes it could be that. But didn’t take time to check this

Ryan McCauley March 15, 2024 at 5:28 PM

Yes the PR looks good to me, approved. Regarding the original issue, I might have to look a bit more into it to understand why it caused the above issue. Before my changes was the second call to set the property overriding the first and in your scenario this is the behaviour we want?

Frédéric Deniger March 15, 2024 at 3:27 PM

your feedback are welcome slightly smiling face

Details

Assignee

Reporter

Priority

Created March 15, 2024 at 3:16 PM
Updated October 1, 2024 at 8:54 PM

Flag notifications