Document the behavior of accepting a command parameter multiple times

se-edu / addressbook-level3

:ab::three: Address Book sample application (Level 3)

https://se-education.org/addressbook-level3

MIT License

28 stars 430 forks source link

Document the behavior of accepting a command parameter multiple times #58

Closed damithc closed 3 years ago

damithc commented 3 years ago

Currently, if a command contains the same parameter e.g., p/ multiple times, the last value will be taken. Rationale: this allows the user to rectify erroneous parameter values on the fly, without having to backspace.

Suggested:

Document this behavior as it can be misinterpreted as a bug
When this happens, the command output can contain a friendly warning that multiple values were found in the command, in case the repetition was unintentional

damithc commented 3 years ago

Fixed by 063450a5c7a220895d5f5c19a1422607cea92df7