ruiyigan / pe

0 stars 0 forks source link

Alternate path required for DeleteCommand #18

Open ruiyigan opened 1 year ago

ruiyigan commented 1 year ago

If the user types in an invalid index, the exception is thrown before I do removeMemberinProjects(p) based on your code base.

Hence, there should be an alternative path to show the exception being returned when invalid too.

Screenshot 2023-04-14 at 15.26.36.png

Screenshot 2023-04-14 at 15.28.53.png

nus-se-script commented 1 year ago

Team's Response

This DocumentationBug is rejected, as there is no issue with the shown sequence diagram. It occurs after the creation of the DeleteCommand. Any issues with an invalid index would be thrown by ParserUtil before any DeleteCommand objects are created. Hence, this issue is invalid.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]