Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Patient ID sequence should not go backwards (greater to lesser) #485

Closed
mnorbeck opened this issue Jun 1, 2016 · 4 comments
Closed

Patient ID sequence should not go backwards (greater to lesser) #485

mnorbeck opened this issue Jun 1, 2016 · 4 comments
Labels
🐛bug issue/pull request that documents/fixes a bug

Comments

@mnorbeck
Copy link

mnorbeck commented Jun 1, 2016

It was observed and confirmed that at the Tebow hospital, the patient ID sequence was set at a number less than the greatest patient ID in the system. There were 2000+ patients in the database currently, but the sequence was set at something around 1400. It was not causing an issue, since the system prevented creation of patients with duplicate IDs. However, in consult with @jkleinsc it was agreed that this behavior should be corrected.

@tangollama tangollama added 🐛bug issue/pull request that documents/fixes a bug Patient help wanted indicates that an issue is open for contributions labels Jun 26, 2016
@gnowoel
Copy link
Contributor

gnowoel commented Sep 24, 2016

The suggested next patient ID is saved in the sequence model. However, if we use a custom ID in the form, the model won't update accordingly. For example, entering P00010 for a suggested P00005, it would next time suggest P00006 rather than P00011. Guess this is part of the reason why the sequence comes slower than the greatest patient ID.

A related problem happens when we cancel a new patient form. Because it's cancelled, the suggested ID is not used, but the number keeps getting incremented regardless. The sequence just goes too fast this time.

Not sure if these are root cause of what @mnorbeck encountered, but the situation shouldn't appear again once the sequence cached up with the greatest patient ID.

@jkleinsc jkleinsc removed the help wanted indicates that an issue is open for contributions label Oct 5, 2016
@mdarmanin
Copy link
Contributor

Is this assigned? I don't see anyone assigned yet.

@jkleinsc
Copy link
Member

@mdarmani @gnowoel is working on this.

@jkleinsc
Copy link
Member

I am going to close this as #667 addressed this issue. #572 is closing the loop on making sure dups can't be entered.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐛bug issue/pull request that documents/fixes a bug
Projects
None yet
Development

No branches or pull requests

5 participants