search

yhtingg / pe

0 stars 0 forks source link

No error catching for numbers that start with 0 #2

Open yhtingg opened 4 years ago

yhtingg commented 4 years ago

image.png

image.png

nus-pe-bot commented 4 years ago

Team's Response

This is due to the user editing the configuration file, which is not a supported use case stated in the current User Guide. Hence, this bug is out of scope.

In the future, only if we decide to support relocation of data files, we will clarify that the paths should be readable/writable by the user, but this does not apply to the current version.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Seems like the team did not understand the issue I posted and marked it as a duplicate.

image.png

From the screenshot, it can be seen that when the validations of certain fields are not being met, such an informative error message would appear. Since no error message (other than the AccessDeniedException) was displayed to the Result Box, I assumed that there was no error catching for numbers starting with 0 as the meal was still added. However, I thought that such numbers should not be considered valid.

:question: Issue type

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

Reason for disagreement: [replace this with your reason]