search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

"Add" command takes in empty input for "drug_allergy" field #3478

Open nus-pe-bot opened 4 months ago

nus-pe-bot commented 4 months ago

The "add" command does not check for valid alphanumeric strings when taking in the drug allergy field, potentially redundantly storing drug allergies which are not meaningful.

Steps to reproduce:

  1. Type "add ic/S9974944F n/John Doe p/91234567 e/johndoe@email.com g/M b/11-11- 1990 d/ | | | | i/Infectious Diseases"

  2. Press "Enter"

The result is as follows (index 27):

Screenshot 2024-04-19 164234.png

[original: nus-cs2103-AY2324S2/pe-interim#4390] [original labels: severity.Low type.FeatureFlaw]

jmsandiegoo commented 4 months ago

Team's Response

Duplicate of #5429

Duplicate status (if any):

Duplicate of #5249