search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Addalloc command as given in UG examples doesn't work when allocating student with same id to group #5824

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

Steps to reproduce
addstudent -n Jonas Chow -i E0123456
addstudent -n Jonas Chow -i E0123457
addgroup -g S01A
addalloc -g S01A -n Jonas Chow -i E0123457

Expected result
Allocate S01A to Jonas Chow with id E0123457

Actual result
Invalid command format! addalloc: Adds a student to an existing group. Parameters: -g (-n | -i ) Example: addalloc -g T01A -n Jonas Chow

Screenshots

Screen Shot 2021-11-12 at 4.20.52 PM.png

[original: nus-cs2103-AY2122S1/pe-interim#5797] [original labels: severity.Medium type.DocumentationBug]

nbvannhi commented 2 years ago

Team's Response

As stated in the UG, the format of addalloc command only allows either the student name -n or NUSNET ID -i to be specified. Thus addalloc -g S01A -n Jonas Chow -i E0123457 is seen as invalid since both fields are specified. It seems like the tester misunderstood the command format. The command worked as expected.

image.png

Moreover, the title

Addalloc command as given in UG examples doesn't work when allocating student with same id to group

doesn't seem to match with the description since there is no such command included in the UG.

image.png

Duplicate status (if any):

--