nusCS2113-AY1920S1 / pe-dev-response

0 stars 0 forks source link

Copy function does not work #1087

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Screen Shot 2019-11-15 at 5.01.57 PM.png Screen Shot 2019-11-15 at 5.07.01 PM.png

As seen in the above two images, bars 2 to 4 were grouped under timesplit, but copy function does not recognize them. This is replicable by assigning bars 2 to 4 to a group name, then trying to assign the same bars to another group name, then calling the copy function for the first group name


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

Samuel787 commented 4 years ago

Team's Response

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 group command, there can only be 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

However, I do agree that when such an input is received by Ducats, it shouldn't allow the group to be created. It should have shown an exception message instead.

Since the group command works perfectly fine for any correct input format as specified in the user guide and only lacks in the fact that it doesn't show an exception message when extra spaces are inserted between the parameters, this should be a MEDIUM severity bug.

Duplicate status (if any):

--