search

nus-cs2113-AY2021S2 / pe-dev-response

0 stars 0 forks source link

incomplete delete feature #1168

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Overview of the bug: As a user I added a patient to my list. I then proceeded to load one of the patient and recorded the symptoms, diagnosis, prescription. I added records to this patient. Now although I delete the patient from my list of patients, I am still able to view and retrieve the deleted patient's records. As a user this is hard to understand.

Commands to recreate this bug:

  1. add S4956795B
  2. load S4956795B
  3. delete /p S4956795B
  4. current

In my opinion, this feature does not solve the stated problem of the intended user which is to delete a patient from the entire database. Hence the delete patient feature is incomplete and don't fit well with the product

image.png

[original: nus-cs2113-AY2021S2/pe-interim#1168] [original labels: severity.Medium type.FeatureFlaw]

fsgmhoward commented 3 years ago

Team's Response

This is a valid concern. We did not purge the data out of memory when we delete the patient.

Duplicate status (if any):

Duplicate of #246