search

ivanderjmw / pe

0 stars 0 forks source link

[CS2113T-F11-1][TheSchwarzenegger] Error with parsing duplicate arguments #2

Open ivanderjmw opened 3 years ago

ivanderjmw commented 3 years ago

Bug

When given duplicate arguments, the program will parse as if nothing is wrong.

Steps to reproduce

  1. Enter profile menu.
  2. Delete any profile that exists using delete.
  3. Input the following: add /n Name /n Name 2 /h 182 /w 90 /e 120 /c 0

Expected output

Prompts the user to reenter because there is an error

Screenshot

Screenshot 2020-11-13 at 4.37.52 PM.png

nus-pe-bot commented 3 years ago

Team's Response

Thank you for your feedback.

However, we consider it as a feature because the application is also for user who can type fast and prefers CLI over normal GUI. Therefore, by doing this, we can help the user utilizes his/ her time by not having to go back and delete the previous wrong information (perhaps a typo) but carry on and add the correct information with the same tag.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I think it is a nice feature, but you can indicate this in your UG for future versions.

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]