search

pos0414 / pe

0 stars 0 forks source link

Help command decrement section wrong #8

Open pos0414 opened 2 years ago

pos0414 commented 2 years ago

The command format (parameters) for decrement session is wrong. should it exclude the expiry date? Just like what is shown in UG. image.png

nus-se-bot commented 2 years ago

Team's Response

Duplicate as it pertains to the usage message for the decrement command

The 'Original' Bug

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

Error msg misleading for decrement

In the UG it says decrement command only takes name and quanitty, but program error message says it should take expiry date also, just like shown below. command: decrement i/chicken q/1 e/2020-02-29 image.png

image.png

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

Their Response to the 'Original' Bug

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

Severity is low as this only appears in rare situations and causes minor inconvenience. A user would have seen the usage format in the UG and used the correct format.

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 not duplicate as it is from a different command, and they have different natures.

This issue is from help, which displays all information about command formats. The claimed "original" issue is from decrement's error message.

Therefore, it is neither the exact same bug, nor multiple buggy behavior that is caused by the same defect and cannot be fixed independently. It can be easily fixed by updating help command's string and error message string separately. Thus it does not fall under any of the categories mentioned in the module website. Therefore, it is not a duplicate issue.

image.png

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: I believe it should be severity.Medium instead.

This issue causes occasional inconvenience to users who wish to use help command and refer to decrement command's section.

Using the help command means the user was not sure of how to use the specific command. Thus, if it displays false information like this, it definitely causes big inconvenience because the user will not know what is the correct format, unless they refer to UG again to search through the long pages to find the command.

Thus, it should be severity.Medium.