nus-cs2103-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Bug detected in Edit person. #2594

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

When the edit is executed after a find command, the index is wrongly matched.

pre-condition: the 3rd person is called null in the person list

command : find null - edit 3 n/non

image.png


[original: nus-cs2103-AY1920S2/pe-interim#2592]

terencemah commented 4 years ago

Team's Response

The command is working as expected. To edit the person with the name null, the index 1 should have been entered instead; index 3 is invalid as there is only 1 person on the currently displayed person list. It is specified in the relevant user guide section that the index entered should be based on the currently displayed list, and not the overall person list.

Screenshot 2020-04-18 at 6.30.15 AM.png

Duplicate status (if any):

--