BC-5119-Prevent-creating-new-neXboards (Server)

apps/server/src/modules/learnroom/controller/rooms.controller.ts

@@ -2,7 +2,7 @@
 import { ApiTags } from '@nestjs/swagger';
 import { RequestTimeout } from '@shared/common';
 import { ICurrentUser, Authenticate, CurrentUser } from '@modules/authentication';
-import { CopyApiResponse, CopyMapper } from '@modules/copy-helper';
+import { CopyApiResponse, CopyElementType, CopyMapper } from '@modules/copy-helper';
 import { serverConfig } from '@modules/server/server.config';
 import { RoomBoardResponseMapper } from '../mapper/room-board-response.mapper';
 import { CourseCopyUC } from '../uc/course-copy.uc';

apps/server/src/modules/learnroom/service/board-copy.service.ts

@@ -51,6 +51,7 @@ export class BoardCopyService {
 		const references: BoardElement[] = this.extractReferences(elements);
 
 		let boardCopy: Board = new Board({ references, course: destinationCourse });
+
 		let status: CopyStatus = {
 			title: 'board',
 			type: CopyElementType.BOARD,
@@ -84,7 +85,6 @@ export class BoardCopyService {
 			if (element.target === undefined) {
 				return Promise.reject(new Error('Broken boardelement - not pointing to any target entity'));
 			}
-
 			if (element.boardElementType === BoardElementType.Task && isTask(element.target)) {
 				return this.copyTask(element.target, user, destinationCourse).then((status) => [pos, status]);
 			}

apps/server/src/modules/learnroom/service/course-copy.service.ts

@@ -43,16 +43,25 @@
 		const existingNames = existingCourses.map((course: Course) => course.name);
 		const copyName = this.copyHelperService.deriveCopyName(newName || originalCourse.name, existingNames);
 
-		// copy course and board
+			// copy course and board
 		const courseCopy = await this.copyCourseEntity({ user, originalCourse, copyName });
 		const boardStatus = await this.boardCopyService.copyBoard({ originalBoard, destinationCourse: courseCopy, user });
+		const filteredBoardStatus = this.filterOutNeXboardFromCopyStatus(boardStatus);
 		const finishedCourseCopy = await this.finishCourseCopying(courseCopy);
-
 		const courseStatus = this.deriveCourseStatus(originalCourse, finishedCourseCopy, boardStatus);
 
 		return courseStatus;
 	}
 
+	private filterOutNeXboardFromCopyStatus(boardStatus: CopyStatus): CopyStatus {
+		if (boardStatus.elements) {
+			boardStatus.elements = boardStatus.elements.filter(elementStatus =>
+				elementStatus.type !== CopyElementType.LESSON_CONTENT_NEXBOARD
+			);
+		}
+		return boardStatus;
+	}
+
 	private async copyCourseEntity(params: CourseCopyParams): Promise<Course> {
 		const { originalCourse, user, copyName } = params;
 		const courseCopy = new Course({
@@ -100,7 +109,6 @@
 		if (courseGroupsExist) {
 			elements.push({ type: CopyElementType.COURSEGROUP_GROUP, status: CopyStatusEnum.NOT_IMPLEMENTED });
 		}
-
 		const status = {
 			title: courseCopy.name,
 			type: CopyElementType.COURSE,
@@ -110,5 +118,7 @@
 			elements,
 		};
 		return status;
 	}
+
+
 }