prgJ / pe

0 stars 0 forks source link

Double parameter takes only the second value #4

Open prgJ opened 2 years ago

prgJ commented 2 years ago

Upon users accidentally inputting two parameters, like for example in this command with two 'round' parameters, add n/John Doe p/98765432 e/johnd@example.com j/Software Engineer r/Technical Interview r/Instagram check s/Java the following is produced: Screenshot 2022-04-16 at 2.52.54 PM.png

Only the second parameter is accepted. Such behaviour does not throw an error, nor is it specified in the user guide. However it might lead to users to falsely input data and not realize it, especially if they are not aware this can happen.

nus-pe-script commented 2 years ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

This is expected behaviour. The user guide does state that when a parameter is expected only once, but entered twice. The last occurence of the input will be taken. Screenshot of User Guide:

Screenshot 2022-04-17 at 4.07.53 PM.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]