nus-pe-bot
commented
2 years ago Team's Response
There is a good reason why I made it such that the app completely does not run when file is corrupted, the reason is to prevent the corrupted file from being deleted and overwritten.
In such a case, I want the user to back up that data file and first before proceeding. And thus the issue is that there should have been a message telling the user of how to back up the data file and delete it afterwards.
The 'Original' Bug
[The team marked this bug as a duplicate of the following bug]
Editing the save file crashes program
Although it tells me that my file is corrupted, no extra info or instructions is given.
Unless I myself go delete the file (or to uncorrupt the file), I cannot run the app anymore
[original: nus-cs2113-AY2122S1/pe-interim#1375] [original labels: severity.Medium type.FunctionalityBug]
Their Response to the 'Original' Bug
[This is the team's response to the above 'original' bug]
Indeed, there should be a message to tell user to delete the corrupted file in the data folder. Moreover, the text data file is not supposed to be edited manually.
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 type
Team chose [type.FunctionalityBug]
Originally [type.FeatureFlaw]
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
:question: Issue severity
Team chose [severity.Low]
Originally [severity.Medium]
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
If the text file becomes corrupted, the user is only given this information and the application does not give him the option to delete it or overwrite it to continue using the application, it simply ceases to work. This would make the application unusable past this point, unless the user understands how to uncorrupt the save file, which they might not. The user guide does not provide useful troubleshooting information to the user for this either.
Even after clearing both the csv file (excel) and the text file completely, the same error message still arises.