search

wuzhzn / pe

0 stars 0 forks source link

Doubles are numbers #10

Open wuzhzn opened 1 year ago

wuzhzn commented 1 year ago

image.png

Doubles should be treated as numbers, or the exceptions should specify integers.

nus-pe-script 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]

Large positive numbers in the number field are not allowed

image.png

Large positive numbers in the flags that require numbers are not detected as numbers. This occurs in most of the commands that require some form of numbers. e.g. add, remove, recipe need , make, recipe favourite, recipe unfavourite

[original: nus-cs2113-AY2223S2/pe-interim#819] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

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

Downgraded to very low because incorrect error message is a typo/cosmetic issue

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 is a separate issue from the duplicate since this bug is a typo/cosmetic bug that I found would be a better error message, while the duplicate bug is due to a choice of int as the type.

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