Section 4.3.1: Removal of fields from person not specified

[Tan-Jin-Waye]

Steps to replicate

edit -p INDEX for some valid index, with an empty field for p/, e/ or c/ e.g. edit -p 1 p/`

Expected behaviour

None provided.

Actual behaviour

Field is set to empty. This is only specified for t/ but not for other fields.

[soc-pe-bot]

Team's Response

The UG says that

• Existing values will be updated to the input values.
• When editing tags, the existing tags of the person will be removed i.e adding of tags is not cumulative.
• You can remove all the person’s tags by typing t/ without specifying any tags after it.

The first point here states the behaviour.
The third point here was mentioned specifically because 1 person can have multiple tags, so t/ will remove all tags. This is in contrast to e/, p/ and c/, where an empty input value removes the only existing email/phone/company.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

• [ ] I disagree

Reason for disagreement: [replace this with your explanation]

---