search

randallnhr / pe

0 stars 0 forks source link

Editing the name in the Json file overrides the same name check and wipes the data. #8

Open randallnhr opened 1 year ago

randallnhr commented 1 year ago

image.png

image.png

Problem: When editing an existing person's name to another existing person (Cat ng to cat ng), the app wipes the entire data file although the format of the Json file is still correct.

nus-pe-bot commented 1 year ago

Team's Response

Thank you for the information.

As a data file containing duplicate persons is not a valid list allowed by the application, rejecting it is the intended behaviour as specified by the User Guide's warning below. Furthermore, the application does not delete or overwrite such a data file until new commands are entered - providing the user room to recover even if he/she has attempted to edit it without maintaining backups.

However, we understand your concerns over data integrity - implementing data recovery from a malformed file would be a useful future addition to the application.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.High`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]