Merge pull request #297 from NIAEFEUP/release/3.1.1

fix: app crashing sometimes after refresh course unit hashes
NIAEFEUP · Sep 9, 2024 · 5cb200a · 5cb200a
2 parents 5ca88d4 + 086c120
commit 5cb200a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/components/planner/sidebar/CoursesController/ClassItem.tsx b/src/components/planner/sidebar/CoursesController/ClassItem.tsx
@@ -41,7 +41,7 @@ const ClassItem = ({ course_id, classInfo, displayed, checked, preview, onSelect
         // retrieve class with the picked class id of the course option
         const pickedClass = pickedCourse.classes.find(c => c.id === course_option.picked_class_id);
 
-        classes.push(pickedClass);
+        if (pickedClass) classes.push(pickedClass);
       }
     }
 

diff --git a/src/utils/index.ts b/src/utils/index.ts
@@ -316,7 +316,7 @@ const getAllPickedSlots = (selected_courses: PickedCourses, option: Option) => {
     if (!course.picked_class_id) return []
     const courseInfo = selected_courses.find((selected_course) => selected_course.id === course.course_id)
     const classInfo = courseInfo.classes.find((classInfo) => classInfo.id === course.picked_class_id)
-    if( !classInfo ) return [];
+    if (!classInfo) return [];
     return classInfo.slots
   })
 }