search

CS2113-AY1819S1-T12-2 / HealthBook

Health Book - An address book app for health care professionals
https://cs2113-ay1819s1-t12-2.github.io/main/
MIT License
0 stars 5 forks source link

No proper error handling when the same medical history record is added #157

Closed nus-pe-script closed 5 years ago

nus-pe-script commented 5 years ago

Describe the bug A clear and concise description of what the bug is.

To Reproduce Steps to reproduce the behavior:

  1. Go to comman line and type addhist 1 hsd/10-10-2010 hsa/Alcohol hsc/Kuwait hsds/d ( if there's a person present in the list)
  2. type addhist 1 hsd/10-10-2010 hsa/Alcohol hsc/Kuwait hsds/d again
  3. The message displayed in the result panel will still be Added medical history to Person: John Doe...
  4. See error

Expected behavior The result panel should display that this record has already been added to the person's history

Screenshots If applicable, add screenshots to help explain your problem. image

Additional context Add any other context about the problem here.

Reported by: @linnnruoo Severity: Medium

[original: nusCS2113-AY1819S1/pe-1#288]

xShaivan commented 5 years ago

The duplicate check has been implemented as a command exception. Thank you for pointing out the bug! This will be found in the v1.4 release.