Editing Json file

[lyndonlim27]

Editing the json file allows patients to have the same id.

Suggestion: It would be good to have a validity checker whenever the app is launched.

[nus-pe-script]

Team's Response

This is undefined behaviour which occurs when json file is directly edited, which is not supported in the current version.

The 'Original' Bug

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

App cannot be launched after editing the id of a patient to that of another patient

after editing the id of a patient to that of another patient, the application can no longer be launched. This problem is fixed after changing the id to one that does not currently exist.

---

_{[original: nus-cs2103-AY2223S2/pe-interim#3267] [original labels: type.FunctionalityBug severity.Medium]}

Their Response to the 'Original' Bug

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

We have not provided for editing of the data file directly to be a feature in this milestone. Modifying the data file directly when neither the UG nor DG provides for such functionality can be seen as extreme behaviour seeking to corrupt the app data. This is a less important use case than developing other core functionality in the app. Potential future improvement.

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)

• [ ] I disagree

Reason for disagreement: [replace this with your explanation]

---

## :question: Issue response Team chose [`response.NotInScope`] - [x] I disagree **Reason for disagreement:** As shown in the screenshot below, there is no warning in UG that states the json file cannot be edited. This should still be in scope as many teams including mine have handled invalid json file that was edited directly.  The team mentioned potential future improvement but this is not mentioned in the DG planned enhancement section 

---