nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Adding of multiple same fields take the last field #4529

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

I believe that adding of multiple of the same fields in the add command should not be allowed and an error message should be thrown.

Steps to reproduce: add n/John Doe p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 c/3.50/4.00 g/male u/Nanyang Polytechnic gd/05-2024 m/Computer Science ji/17329 jt/Software Engineer Intern t/rejected t/KI n/Song Ivan

Actual output : Shown in the picture below Screenshot 2022-11-11 at 4.52.59 PM.png

Expected output: Error as you cannot have to of the same fields (else should at least state in the UG which field will be taken)


[original: nus-cs2103-AY2223S1/pe-interim#4471] [original labels: type.FunctionalityBug severity.Low]

maryjess commented 1 year ago

Team's Response

It is already stated in our UG that if such parameter is mentioned twice, the last one will be taken.

image.png

Duplicate status (if any):

--