search

Piyotato / pe

0 stars 0 forks source link

Single invalid name entry in JSON file causes entire data file to be wiped #12

Open Piyotato opened 1 year ago

Piyotato commented 1 year ago

No details provided by bug reporter.

soc-se-bot commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Editing meeting times in JSON file to clash results in entire data being wiped on load

Steps to Reproduce:

  1. Add two meetings with different times
  2. Edit the data file such that they have the same time
  3. Start up the application

Although you welcome users to try edit the file, small mistakes such as this (even if the timings overlap by 1 minute) will cause the entirety of the data to be wiped. A warning would be more appropriate.

[original: nus-cs2103-AY2223S1/pe-interim#4833] [original labels: severity.Medium type.FunctionalityBug]

Their Response to the 'Original' Bug

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

Thank you for the bug report.

Referring to the section below given in the UG:

image.png

we believe that there was sufficient warning given to users on cautiously editing the data file.

Also, we feel that this issue correlates to the feature where the entire data file would be cleared on corruption. As suggested, it would be more appropriate for us to add a warning message. But, for now we would like to consider it as a feature flaw.

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)

Reason for disagreement: [replace this with your explanation]

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]

:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]