search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Inappropriate error message displayed #379

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

Command to reproduce: "filterTxn 1 amts/". The error here is invalid prefix of amts/, but the error message is saying:

image.png

Nothing is wrong with the index (1 is a non-zero unsigned integer), prefix then is the issue.

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

SeeYangZhi commented 1 week ago

Team's Response

Thank you for your feedback!

Upon reviewing the issue, it appears there may have been a misunderstanding regarding the usage of the filterTxn command. As outlined in the User Guide, the filterTxn command only accepts specific parameters.

As there is no parameter for amts/, the only accepted plaintext input would be INDEX. Therefore, the tester’s command “filterTxn 1 amts/” should be read as:

In this case, the INDEX value is not a valid non-zero unsigned integer, which is precisely why the error message is triggered. This behavior aligns with the expected functionality as documented.

Since the issue arises from a misunderstanding of the User Guide and not a flaw in the app, we will mark this report as response.Rejected.

We encourage the tester to review the relevant section of the User Guide for further clarification, and we’re happy to provide additional assistance if needed.

Duplicate status (if any):

--