fix bug where sections with no meeting time cannot be deselected

recalapp · Apr 6, 2016 · 11e6d75 · 11e6d75
1 parent fff14ab
commit 11e6d75
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/course_selection/static/js/models/ScheduleManager.ts b/course_selection/static/js/models/ScheduleManager.ts
@@ -199,7 +199,12 @@ class ScheduleManager {
                 }
             } else {
                 if (Utils.isInList(section.id, sectionIds)) {
-                    this.enrollSection(section);
+                    // don't enroll if the section
+                    // 1. doesn't having meeting times
+                    // 2. isn't the only section available for that type
+                    if (this._isTheOnlySectionOfType(course, section) || section.has_meetings) {
+                        this.enrollSection(section);
+                    }
                 }
             }
         }