AY1920S2-CS2113T-T13-3 / tp

0 stars 6 forks source link

Bug #3 dummy field but still executed #51

Open JustinnT opened 4 years ago

JustinnT commented 4 years ago

Input: addp \name John \hobby sleep Expected output: ??? Actual output: adds the patient name John but ignores the dummy field.

@thanhduc2000 Also so sorry, please explain more on this bug on what should be the expected output :). On my system it just ignores the dummy fields. I believe this is the correct behavior.

thanhduc2000 commented 4 years ago

Expected output should be like: "hobby is not a valid header, do you still want to add the item (yes/no)". Type yes if want to add and no if not

thanhduc2000 commented 4 years ago

I mean it's not technically a bug but I want to add something like that

JustinnT commented 4 years ago

I feel like it might slightly hard because what happens if there are multiple invalid headers?

Something like addp \name John \hobby sleep \favouritefood sushi \favouritebook harry potter \movie Avengers end game

Perhaps we can consider it once we are finishing with the main features of v2/3?

thanhduc2000 commented 4 years ago

Then don't have to specify name of invalid headers, just state that "there are valid headers in your command, ...". Then for later we can list out the invalid headers in v2 and 3