deeyonn
commented
1 year ago Team's Response
Thank you for the report. As mentioned in 3.10 of the UG, we already have a warning that if the file contains invalid data or has invalid format, the content will be erased and an empty addressbook will be loaded. This is a warning for the user before the user tries to edit the file directly.
Duplicate status (if any):
--
Steps:
1) Type add n/John Doe p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 c/3.50/4.00 g/male u/Nanyang Polytechnic gd/05-2024 m/Computer Science ji/173296 jt/Software Engineer Intern t/rejected t/KIV into command box 2) Type export 3) Edit json file by removing the p/ 4) Type checkout (FILE_NAME) into command box
Expected: The app should throw a error that the field of p/ is missing or wrong. Actual: The app deletes all the data in that Json file and replaces with sample data.
This is an issue as a user might have accidently edited a field in the Json file, and all the data is removed without informing the user, or giving the user chance to edit the file again. This will affect the user experience greatly if the user already has a lot of data stored. Thus an error thrown would be better.
[original: nus-cs2103-AY2223S1/pe-interim#2415] [original labels: severity.Medium type.FunctionalityBug]