From 53fc79524f98d20d26df8efb93e25448a9fa3ed6 Mon Sep 17 00:00:00 2001
From: ievavold <ievavold@wisc.edu>
Date: Thu, 21 Mar 2019 10:28:58 -0500
Subject: [PATCH] ROENROLL-1508 fix add course to non-primary plan

---
 src/app/degree-planner/store/effects/course.effects.ts | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/app/degree-planner/store/effects/course.effects.ts b/src/app/degree-planner/store/effects/course.effects.ts
index 57b0237..06d1206 100644
--- a/src/app/degree-planner/store/effects/course.effects.ts
+++ b/src/app/degree-planner/store/effects/course.effects.ts
@@ -131,8 +131,11 @@ export class CourseEffects {
       const { subjectCode, termCode, courseId, newIndex } = action.payload;
       let addCourse$: Observable<CourseBase>;
 
-      // When adding to an active term use cart endpoint
-      if (activeTerms.find(term => term.equals(termCode))) {
+      const isActiveTerm = activeTerms.filter(term => term.equals(termCode));
+      const isPrimaryPlan = visibleDegreePlan
+        ? visibleDegreePlan.primary
+        : false;
+      if (isActiveTerm && isPrimaryPlan) {
         addCourse$ = this.api.addCourseToCart(subjectCode, courseId, termCode);
       } else {
         addCourse$ = this.api.addCourse(
-- 
GitLab