log non existing tasks

Author: Ken Hasenbank

api/src/models/Course.ts

@@ -137,7 +137,6 @@ courseSchema.statics.importJSON = async function (course: ICourse, admin: IUser)
       // 1. Duplicate -> 'name (copy)', 2. Duplicate -> 'name (copy 2)', 3. Duplicate -> 'name (copy 3)', ...
       course.name = origName + ((i > 0) ? ' (copy' + ((i > 1) ? ' ' + i : '') + ')' : '');
       isCourseDuplicated = (await Course.findOne({name: course.name})) !== null;
-      console.log(course.name + ' -> ' + ((isCourseDuplicated) ? 'Duplicate' : 'OK'));
       i++;
     } while (isCourseDuplicated);
     const savedCourse = await new Course(course).save();

api/src/models/units/TaskUnit.ts

@@ -5,6 +5,7 @@ import {ITaskModel, Task} from '../Task';
 import {ITask} from '../../../../shared/models/task/ITask';
 import {InternalServerError} from 'routing-controllers';
 import {ILectureModel, Lecture} from '../Lecture';
+import * as winston from 'winston';
 
 interface ITaskUnitModel extends ITaskUnit, mongoose.Document {
   exportJSON: () => Promise<ITaskUnit>;
@@ -49,8 +50,13 @@ taskUnitSchema.methods.exportJSON = async function() {
 
   for (const taskId of tasks) {
     const task: ITaskModel = await Task.findById(taskId);
-    const taskExport = await task.exportJSON();
-    obj.tasks.push(taskExport);
+
+    if (task) {
+      const taskExport = await task.exportJSON();
+      obj.tasks.push(taskExport);
+    } else {
+      winston.log('warn', 'task(' + taskId + ') was referenced by unit(' + this._id + ') but does not exist anymore');
+    }
   }
 
   return obj;