nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Editing a student's address, phone, email without name does not work and throws an irrelevant error. #2565

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Editing address alone:

image.png

Editing address and name:

image.png

Editing phone alone:

image.png

Editing phone and name:

image.png

Editing email alone:
image.png

Editing email and name:

image.png

Editing a student's address, phone, email requires the name to be edited as well. Makes it inconvenient for the user.


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

NoraYUuu commented 3 years ago

Team's Response

This is an unintended behaviour as users might want to edit other fields of a student without having to include the name prefix in the command as well. This would only be an occasional hindrance as users are still able to edit any of the four fields of a student successfully - just that the name prefix has to be included in the command.

For instance, if a user wanted to edit the address field of a student named Jason Tan, the following commands may be used:

edit 1 a/221 Baker Street n/a, edit 1 n/Jason Tan

This would update the student's address successfully. Similarly, the other fields of the student can be updated as well by using their respective prefixes.

Duplicate status (if any):

--