NHSDigital / DataDictionaryPublication

Apache License 2.0
7 stars 1 forks source link

Duplicate Items #489

Open AngelaFaulding opened 4 months ago

AngelaFaulding commented 4 months ago

I created an Attribute Terminology for "PERSON STATED EYE COLOUR THE SAME INDICATOR" and I think I must have clicked twice on Submit Terminology as my machine is slow this morning. There are now two Attribute Terminologies. This seems strange as I would expect that you shouldn't be able to create an item with the same name in a folder. If we are doing a lot of authoring this could happen. I obviously can delete one of the terminologies but want to check this first. Should we be prevented form creating the same item twice?

image

I didn't release this until I tried to create a Term and got an error message:

image

pjmonks commented 4 months ago

You are correct, this should not be possible. I would have expected Mauro to validate that another terminology of the same label exists, but due to the timing of both creation requests perhaps this was not possible.

There is a simple fix we can perform on the UI to prevent this from happening which I have created an issue for. It is low priority, but we will get to it:

For now I would suggest deleting the duplicate and continue adding terms to the remaining terminology.

pjmonks commented 4 months ago

This has been fixed and release to the test environment.

AngelaFaulding commented 1 month ago

@KatePalmerLilley - we need to test this.

KatePalmerLilley commented 1 month ago

@AngelaFaulding @pjmonks I've tested what is described in https://github.com/MauroDataMapper/mdm-ui/issues/885, and yes the "Submit Terminology" button is disabled as soon as it is clicked, so I believe this particular issue to be fixed.

However, I was able to create a duplicate attribute terminology if I changed the capitalisation, e.g.:

image

Can we discuss.

KatePalmerLilley commented 1 month ago

@angela this is ready for testing and I have raised a new ticket for the new duplicate issue described above #595