rexkoh425 / pe

0 stars 0 forks source link

The error message does not match the error #4

Open rexkoh425 opened 2 weeks ago

rexkoh425 commented 2 weeks ago

image.png

command is 1)add-expense ChiCha a/ 17 d/ 2024-07-09 c/ FnB 2)history f/ 1985-01-01 /t 2000-01-11

the dates are of correct format however the error message says the date format is wrong , the thing that is wrong are invalid parameters.

soc-se-bot commented 2 weeks ago

Team's Response

No details provided by team.

The 'Original' Bug

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

wrong error message for add-income

The amount that was specified in the command is correct according to the User Docs, it is the date command that is wrong as it does not have a space after d/. However the error message specifies that it is an error with the amount image.png


[original: nus-cs2113-AY2425S1/pe-interim#204] [original labels: severity.Medium type.FeatureFlaw]

Their Response to the 'Original' Bug

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

Thank you for the report, we have specified in the Document that the keyword must be followed by a space as it is expected to read all the string after a keyword until the end of the input string/ next keyword found. We've also printed out the whole value read so that the user can notice that the command went wrong but yes, the message may sound confusing! However, we have lowered the severity to Low as we do not believe this bug to be very misleading to the user given that we have specified in the user guide that a space must follow all keywords, and should only cause a minor inconvenience to users who may have wrongly input the command.

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: [replace this with your explanation]


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