jiangqx0225 / pe

0 stars 0 forks source link

after adding the current non existing profile back to list, the record disappears #13

Open jiangqx0225 opened 3 years ago

jiangqx0225 commented 3 years ago

No details provided. image.png

nus-pe-bot commented 3 years ago

Team's Response

The patient has no medical records because this is a new patient that was just added in. The issue lies in a bug in the delete feature, which is a duplicate of issue #246.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

After deleting patient while still in retrieve mode, I can still add records to that patient. [This issues has a few highly severe bugs inside]

First, I am accessing a patient's NRIC and in "retrieve" mode.

Then I tried deleting the same patient. Afterwards, I tried retrieve and I am able to retrieve the patient that I deleted.

I even tried adding new records and no error message was thrown. Then I check the list and the NRIC is "deleted" despite me being able to add new records to that particular patient.

error5.PNG Image pic continue to below image.png


[original: nus-cs2113-AY2021S2/pe-interim#246] [original labels: severity.High type.FunctionalityBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

This is a valid issue that we did not purge the data out of memory when we do deletion.

Since it is unlikely for the GP to run into this issue (e.g. it is expected to load a new patient or close the program after deleting a patient), the severity of this issue should be lowered. The GP would have to trigger this issue purposely to face this problem.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your reason]


:question: Issue type

Team chose [type.FunctionalityBug] Originally [type.FeatureFlaw]

Reason for disagreement: [replace this with your reason]


:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your reason]