Merge pull request #779 from sandboxnu/course-crash-bug

Course crash bug
sandboxnu · Dec 3, 2024 · d3d7d58 · d3d7d58
2 parents b690d5d + 38fe9ee
commit d3d7d58
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/packages/frontend/utils/plan/getAllCoursesInMajor.ts b/packages/frontend/utils/plan/getAllCoursesInMajor.ts
@@ -22,7 +22,7 @@ export const getAllCoursesInMajor = (
   const majorRequirements = major.requirementSections.reduce(
     (courses: IRequiredCourse[], section: Section) => {
       const requiredCourses: IRequiredCourse[] = [];
-      getRequiredCourses(section.requirements, requiredCourses);
+      getRequiredCourses(section.requirements ?? [], requiredCourses);
       return courses.concat(requiredCourses);
     },
     []
@@ -54,9 +54,19 @@ const getRequiredCourses = (
     } else if (requirement.type === "COURSE") {
       requiredCourses.push(requirement);
     } else if (requirement.type === "SECTION") {
-      getRequiredCourses(requirement.requirements, requiredCourses);
+      getRequiredCourses(requirement.requirements ?? [], requiredCourses);
+    } else if (Array.isArray(requirement)) {
+      getRequiredCourses(extractRequirements(requirement), requiredCourses);
     } else {
-      getRequiredCourses(requirement.courses, requiredCourses);
+      getRequiredCourses(requirement.courses ?? [], requiredCourses);
     }
   }
 };
+
+const extractRequirements = (requirements: Requirement2[]): Requirement2[] => {
+  const extracted: Requirement2[] = [];
+  for (const value of requirements) {
+    extracted.push(value);
+  }
+  return extracted;
+};