nusCS2113-AY1920S1 / pe-dev-response

0 stars 0 forks source link

Trailing spaces after song name not handled in new command #1071

Open nus-pe-bot opened 5 years ago

nus-pe-bot commented 5 years ago

Screen Shot 2019-11-15 at 4.49.25 PM.png

As seen in the image above, trailing whitespaces are not handled by the application. This results in a syntactically correct command being handled as a errorneous command


[original: nusCS2113-AY1920S1/pe-interim#1071]

jwyf commented 5 years ago

Team's Response

This issue is a duplicate of issue #602. As with issue #602, this issue is to be rejected as it is the intended behaviour of Ducats, supported the User Guide. Below is the same explanation given to reject issue #602.

The User Guide did not mention that Ducats’ commands can support extra spaces between its parameters. In fact, for each command, the User Guide gives a specific Format for the users to follow. To address the command specific to the tester’s screenshot, please refer to page 4 of the User Guide.

For convenience, the screenshot of the relevant extract of the User Guide is below: image.png

As can be seen, the Format clearly shows that for the new command, there is only 1 space between each of its parameters.

For details on the Formats of Ducats, please refer to page 3 of the User Guide. For convenience again, the screenshot of the relevant extract of the User Guide is below: image.png

From the above extract, coupled with the provided Formats for each and every Ducats command, it should be clear to the user that Ducat’s commands cannot support extra spacing between parameters.

Therefore, the fact that Ducats commands does not support extra spaces between its parameters is the intended behaviour.

Duplicate status (if any):

Duplicate of #602