ROENROLL-1554 preserve course order inside saved-for-later

src/app/degree-planner/saved-for-later-container/saved-for-later-container.component.ts

@@ -11,6 +11,7 @@ import { DegreePlannerState } from '@app/degree-planner/store/state';
 import {
   AddSaveForLater,
   RemoveCourse,
+  MoveCourseInsideSFL,
 } from '@app/degree-planner/store/actions/course.actions';
 import * as selectors from '@app/degree-planner/store/selectors';
 import { distinctUntilChanged } from 'rxjs/operators';
@@ -69,6 +70,14 @@ export class SavedForLaterContainerComponent implements OnInit {
           recordId: course.id as number,
         }),
       );
+    } else if (newContainerId === previousContainerId) {
+      const course = event.item.data as SavedForLaterCourse;
+      this.store.dispatch(
+        new MoveCourseInsideSFL({
+          courseId: course.courseId,
+          newIndex: event.currentIndex,
+        }),
+      );
     }
   }
 }

src/app/degree-planner/store/actions/course.actions.ts

@@ -13,6 +13,8 @@ export enum CourseActionTypes {
   MoveCourseBetweenTerms = '[Course] Move Between Terms',
   MoveCourseBetweenTermsSuccess = '[Course] Move Between Terms (Success)',
 
+  MoveCourseInsideSFL = '[Course] Move Inside SFL',
+
   AddSaveForLater = '[Course] Add Save for Later',
   AddSaveForLaterSuccess = '[Course] Add Save for Later (Success)',
 
@@ -29,6 +31,11 @@ export class MoveCourseInsideTerm implements Action {
   ) {}
 }
 
+export class MoveCourseInsideSFL implements Action {
+  public readonly type = CourseActionTypes.MoveCourseInsideSFL;
+  constructor(public payload: { courseId: string; newIndex: number }) {}
+}
+
 export class MoveCourseBetweenTerms implements Action {
   public readonly type = CourseActionTypes.MoveCourseBetweenTerms;
   constructor(

src/app/degree-planner/store/reducer.ts

@@ -28,6 +28,7 @@ import {
   AddCourseSuccess,
   RemoveSaveForLater,
   AddSaveForLater,
+  MoveCourseInsideSFL,
 } from '@app/degree-planner/store/actions/course.actions';
 import {
   NoteActionTypes,
@@ -68,6 +69,7 @@ type SupportedActions =
   | WriteNoteSuccess
   | DeleteNote
   | MoveCourseInsideTerm
+  | MoveCourseInsideSFL
   | MoveCourseBetweenTerms
   | RemoveCourse
   | AddCourse
@@ -334,6 +336,21 @@ export function degreePlannerReducer(
       return state;
     }
 
+    case CourseActionTypes.MoveCourseInsideSFL: {
+      const { courseId, newIndex } = action.payload;
+      const courses = state.savedForLaterCourses.slice();
+      const course = courses.find(c => c.courseId === courseId);
+
+      if (course) {
+        const oldIndex = courses.findIndex(c => c.courseId === courseId);
+        courses.splice(oldIndex, 1);
+        courses.splice(newIndex, 0, course);
+        return { ...state, savedForLaterCourses: courses };
+      }
+
+      return state;
+    }
+
     case CourseActionTypes.MoveCourseBetweenTerms: {
       const {
         to: toTermCode,