nus-tic4002-AY2122S2 / pe-dev-response

0 stars 0 forks source link

create group has some issus when try to create a group contains'g/' #211

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

create g/c is okay, will create a group named c

image.png

create g/ g/ will show

image.png

but actually create g/g/ can create the group

image.png

And when create a group, actually can have space between the prefix create g/ d

image.png

Anyway normally no user will create group like this may just need care about some user needed for special case


[original: nus-tic4002-AY2122s2/pe-interim#205] [original labels: severity.Low type.FunctionalityBug]

IcyNoPeople commented 2 years ago

Team's Response

This is working correctly as intended. As we mentioned in the user guide, under the feature section: If a parameter is expected only once in the command but you specified it multiple times, only the last occurrence of the parameter will be taken.

In the case of create g/ g/, it is treated as the user has specified the parameter multiple time and will take the last occurrence and which in this case, has no input and as such, display the correct message.

For the case of create g/g/, it is treated as g/ to be the input and is able to create the group.

Duplicate status (if any):

--