AY2021S1-CS2103T-W10-2/tp

[PE-D] Entering a comma as an ingredient name is allowed, even though the application specified otherwise.

Closed this issue · 1 comments

bug4.PNG

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:

bug5.PNG


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