nus-cs2113-AY2223S2 / pe-dev-response

0 stars 0 forks source link

Unhandled exception when using wrong format for -r switch #561

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Index out of bounds exception when input wrong format for -r switch. Even if the correct format is stated in the UG, it would be good to still notify users of where the issue arise from.

Steps to reproduce: add -e Career Fair -st 14:00 -sd 2023/02/10 -ed 2023/02/11 -r 3

Expected output: Incorrect date format to be raised.

Actual output:

add -e Career Fair -st 14:00 -sd 2023/02/10 -ed 2023/02/11 -r 3
____________________________________________________________
(since no specific ending time information is given, ending date is omitted)
____________________________________________________________
Index 1 out of bounds for length 1
____________________________________________________________

[original: nus-cs2113-AY2223S2/pe-interim#743] [original labels: severity.High type.FeatureFlaw]

hrithie commented 1 year ago

Team's Response

The team agrees that the error message produced is incorrect. However, this flaw is purely cosmetic and does not affect the usage of the command/application.

Duplicate status (if any):

--