nus-cs2113-AY2223S2 / pe-dev-response

0 stars 0 forks source link

Bug arise from other commands after incorrect input #1363

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

The list command seems to show a index out of bounds error as well after the user enters a wrong format for the -r switch.

Steps to reproduce:

  1. add -e Career Fair -st 14:00 -sd 2023/02/10 -ed 2023/02/11 -r 3 - This causes the index out of bounds error being displayed to the user. Afterwards, the list command shows this bug as well.

  2. list

Expected output: List of items to be shown

Actual output:

list
____________________________________________________________
   > 1.[E] Career Fair (2023/02/10 14:00 to 2023/02/11 16:00 not recurring)
   > 2.[E] Career Fair (2023/02/10 14:00 not recurring)
   > 3.[E] Career Fair (2023/02/10 14:00 not recurring)
   > 4.[E] Career Fair (2023/02/10 14:00 not recurring)
   > 5.[E] Career Fair (2023/02/10 14:00 not recurring)
   > 6.[E] Career Fair (2023/02/10 14:00 not recurring)
____________________________________________________________
Index 1 out of bounds for length 1
____________________________________________________________

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

Zemdalk commented 1 year ago

Team's Response

We should have checked the correctness of recurring event format and prevent it from adding new events in event list. Thank you for reporting this bug!

However, as the bug does not result in any functionality issues nor cause any issues in the overall program, it should be classified as severity.Low.

Duplicate status (if any):

--