ADD Feature does not account for adding in drug allergies containing prefix

[nus-se-script]

Steps to reproduce:

• As per the UG, I have inputted add ic/S1974944F n/John Doe p/91234567 e/johndoe@email.com g/M b/11-11-1990 d/Para p/cetamol | Penicillin i/Infectious Diseases, which reads p/ as a prefix.

Expected:

• I expected to get a new patient with drug allergy Para p/cetamol.

Actual:

• However, I got an error message instead.

Screenshots:

This is problematic as there are medicine that have similar drug names that uses the same prefix. You can consider supporting this for better convenience and usability for your users. One way you can do is by using a more specific prefix, or by having a warning rather than blocking it.

---

_{[original: nus-cs2103-AY2324S2/pe-interim#5071] [original labels: severity.Low type.FeatureFlaw]}

[jmsandiegoo]

Team's Response

Although it might seem like an issue, /p is a reserved keyword with respect to the command format as stated in the UG. The drug allergy can contain reserved prefix values but need to make sure that it is not preceded by the space at the start as that would be considered a prefix parameter according to the command format. Perhaps an escape character functionality can be implemented in the future releases to allow spaces beforehand. Thus, this can be considered an enhancement and can be seen as NotInScope

Duplicate status (if any):

--