AY2021S1-CS2113-T13-1 / tp

NotUS is a quick and simple, CLI-based, note-taking application for keyboard-inclined users. NotUS can also assist in planning timetables to highlight possible clashes. Visit our website below.
https://ay2021s1-cs2113-t13-1.github.io/tp/
0 stars 5 forks source link

Issue with sorting of timetable after list-e. #187

Closed brandonywl closed 3 years ago

brandonywl commented 3 years ago

No picture present. Timetable was only sorted after list-e. Perhaps implement a sort function on timetable and call it after each AddEventCommand.

brandonywl commented 3 years ago

Timetable should not be mutated in this fashion. I suspect that this is due to timetable and ListEventCommand or how StorageManager calls/saves the events.

brandonywl commented 3 years ago

Identified the issue to be when events are loaded. Only some of them are sorted or shifted in index.

brandonywl commented 3 years ago

Identified it to be loading non-recurring event first, then recurring event. Thus order will be shifted around and all recurring events will be placed last.