STeng618 / pe

0 stars 0 forks source link

Specificity of error message #3

Open STeng618 opened 5 months ago

STeng618 commented 5 months ago

Create a valid calorie item with description chicken ; rice ; Screenshot 2024-04-19 at 4.35.15 PM.png

Upon relaunch, the application flags the entry as invalid with the error message "Please ensure that a positive integer value is given", which is confusing for the user.

Screenshot 2024-04-19 at 4.34.32 PM.png

nus-se-bot commented 5 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Error is not purged

Add a calorie item with description chicken;

Quit the application.

Launch the application, an error is flagged.

Quit the application again.

Launch the application again, the same error is flagged.

Implication: users have to manually remove the corrupted entry from the data file


[original: nus-cs2113-AY2324S2/pe-interim#1853] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

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

Hi there, thank you for the bug report!

We agree with you that this error was not handled, however we feel that the severity of this bug is low because the semi colon is hardly used. Furthermore, we feel that entering semi colons for calories description occurs only in very rare occasions.

image.png

Thank you!

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: This bug is about the error message instructing the user to "ensure that a positive integer value is given", even though the user did not provide any input to the application in the first place and thus could be confused .

The original bug is about the application not purging corrupted entries in data files, relying on the user to manually remove those entries. Else the same error resurfaces every time the application is relaunched.

Even if the functionalities to purge corrupted entries automatically (after one execution run of the application) are in place, the bug with the error message will still be present whenever there are corrupted entries in the data files. Fixing the "original" bug will not fix the confusing error message


## :question: Issue type Team chose [`type.FunctionalityBug`] Originally [`type.FeatureFlaw`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]