[PE-D] Entering a comma as an ingredient name is allowed, even though the application specified otherwise.
Closed this issue · 1 comments
Command to replicate bug:
addR n/salad i/, c/40 img/images/healthy1.jpg instr/Cook pasta. Serve immediately. t/fast t/easy
Command with bug:
- add recipe command
- edit recipe command
If the user enters this command, a new recipe without ingredients will be added. The ingredient section shows “ , “. However, the error handling of the application specifies that ingredients should only contain alphanumeric characters so I do not think that the above command should allow users to add a new recipe since a comma is not an alphanumeric character.
Screenshot showing the requirements of ingredient names:
Labels: severity.Medium
type.FunctionalityBug
original: jonasngs/ped#3
Defensive programming in addRecipe and addFridge (check ingredientString from parseIngredient is not length 0) (cait)
Update ingredientParser (olivia) check if ingredient is just a bunch of commas