nus-tic4002-AY2122S2 / pe-dev-response

0 stars 0 forks source link

assign n/NAME g/GROUPNAME got small issue when use 'g/ g/' #216

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

A similar problem which already existed in the create group command

assign n/Bernice Yu g/ g/

image.png

assign n/David Li g/ 1 works means can have space after prefix

Again, this behavior is rare and will not affect other fucntions, just need care about special case


[original: nus-tic4002-AY2122s2/pe-interim#206] [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 assign n/Bernice Yu 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.

Duplicate status (if any):

--