fix(timetable): initialize default objects and arrays if undefined

nusmodifications · Aug 4, 2023 · d28ce08 · d28ce08
1 parent 4ce23c1
commit d28ce08
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/website/src/reducers/timetables.ts b/website/src/reducers/timetables.ts
@@ -289,8 +289,11 @@ function timetables(
         draft.lessons[semester] = semTimetable(draft.lessons[semester], action);
         draft.colors[semester] = semColors(state.colors[semester], action);
         draft.hidden[semester] = semHiddenModules(state.hidden[semester], action);
+        if (!draft.customisedModules) {
+          draft.customisedModules = {};
+        }
         draft.customisedModules[semester] = customisedModules(
-          state.customisedModules[semester],
+          state.customisedModules?.[semester],
           action,
         );
       });

diff --git a/website/src/views/timetable/TimetableContent.tsx b/website/src/views/timetable/TimetableContent.tsx
@@ -517,7 +517,7 @@ function mapStateToProps(state: StoreState, ownProps: OwnProps) {
     modules,
     activeLesson: state.app.activeLesson,
     customiseModule: state.app.customiseModule,
-    customisedModules: state.timetables.customisedModules[semester],
+    customisedModules: state.timetables.customisedModules?.[semester] ?? [],
     timetableOrientation: state.theme.timetableOrientation,
     showTitle: state.theme.showTitle,
     hiddenInTimetable,