search

AY2324S2-CS2113T-T09-1 / tp

0 stars 6 forks source link

[PE-D][Tester B] 'take' command rejecting valid numerical listIndex #198

Closed soc-pe-bot closed 7 months ago

soc-pe-bot commented 7 months ago

'take -l listIndex' command only appears to work for listIndex value 1. I have listed this as high severity as this would cause problems for a user tracking their medications, which for a medication app, is medically very undesirable.

Evidence of list:

image.png

Evidence of take -l 1:

image.png

Evidence of ability to view listIndex 2, 3 and 4:

image.png

image.png

'take -l listIndex' bug for listIndex values other than 1:

image.png

image.png

image.png

Labels: type.FunctionalityBug severity.High original: LWachtel1/ped#6

T0nyLin commented 7 months ago

@wenenhoe I believe this is due to misunderstanding that the "take" index is referencing the number from MedMgr, when it is actually the index from DailyMgr list. So need to be clearer on the UG and error message, and to ask user to see "list -t today" instead.