BC-7743 mongoose upgrade

package.json

@@ -215,10 +215,10 @@
 		"moment": "^2.19.2",
 		"mongodb-memory-server-global": "^9.1.8",
 		"mongodb-uri": "^0.9.7",
-		"mongoose": "^6.12.3",
-		"mongoose-delete": "^0.5.4",
+		"mongoose": "^8.5.1",
+		"mongoose-delete": "^1.0.2",
 		"mongoose-id-validator": "^0.6.0",
-		"mongoose-lean-virtuals": "^0.8.1",
+		"mongoose-lean-virtuals": "^0.9.1",
 		"mongoose-shortid-nodeps": "git://github.com/leeroybrun/mongoose-shortid-nodeps.git",
 		"moodle-client": "^0.5.2",
 		"nanoid": "^3.3.4",

src/services/consent/services/consent.deprecated.js

@@ -31,7 +31,7 @@ class ConsentService {
 			// check for _basonType is need for internal request
 			// and because of id could also include an $in or simular
 			// eslint-disable-next-line no-underscore-dangle
-			if (typeof userId === 'string' || userId._bsontype === 'ObjectID') {
+			if (typeof userId === 'string' || userId._bsontype === 'ObjectId') {
 				const user = await this.modelService.get(userId);
 				return {
 					total: 1,

src/services/fileStorage/proxy-service.js

@@ -322,7 +322,7 @@ const fileStorageService = {
 		}
 
 		return permissionPromise()
-			.then(() => FileModel.update({ _id }, update).exec())
+			.then(() => FileModel.updateOne({ _id }, update).exec())
 			.catch((err) => new Forbidden(err));
 	},
 };
@@ -690,7 +690,7 @@ const renameService = {
 				if (!obj) {
 					return new NotFound('The given directory/file was not found!');
 				}
-				return FileModel.update({ _id }, { name: newName }).exec();
+				return FileModel.updateOne({ _id }, { name: newName }).exec();
 			})
 			.catch((err) => new Forbidden(err));
 	},
@@ -872,7 +872,7 @@ const filePermissionService = {
 						}),
 				];
 
-				return FileModel.update(
+				return FileModel.updateOne(
 					{ _id },
 					{
 						$set: { permissions },

src/services/lesson/hooks/index.js

@@ -128,7 +128,7 @@ const setPosition = async (context) => {
 	const { courseId, courseGroupId } = context.data;
 	if (courseId || courseGroupId) {
 		const query = courseId ? { courseId } : { courseGroupId };
-		context.data.position = await LessonModel.count(query).exec(); // next free position
+		context.data.position = await LessonModel.countDocuments(query).exec(); // next free position
 	}
 
 	return context;

src/services/news/hooks/news.hooks.js

@@ -2,7 +2,7 @@ const { newsHistoryModel } = require('../model');
 
 const deleteNewsHistory = async (context) => {
 	if (context.id) {
-		await newsHistoryModel.remove({ parentId: context.id });
+		await newsHistoryModel.deleteOne({ parentId: context.id });
 	}
 	return context;
 };

src/services/role/model.js

@@ -33,7 +33,7 @@ const roleSchema = new Schema(
 // https://mongoosejs.com/docs/middleware.html
 const mongooseOperationsForClearCache = [
 	'findOneAndDelete',
-	'findOneAndRemove',
+	'findOneAndDelete',
 	'findOneAndUpdate',
 	'deleteMany',
 	'deleteOne',

src/services/role/utils/permissions.js

@@ -22,7 +22,7 @@ const ROLES = {
 const definePermissions = (env, role, ...permissions) => {
 	if (Configuration.get(env)) {
 		// set defaul permission
-		Role.update(
+		Role.updateOne(
 			{
 				name: role,
 			},
@@ -37,7 +37,7 @@ const definePermissions = (env, role, ...permissions) => {
 		).exec();
 	} else {
 		// remove defaul permission
-		Role.update(
+		Role.updateOne(
 			{
 				name: role,
 			},

src/services/sync/repo/user.repo.js

@@ -95,7 +95,7 @@ const updateUser = async (userId, changedUser) => {
 };
 
 const deleteUser = async (userId) => {
-	await userModel.remove({ _id: userId }).lean().exec();
+	await userModel.deleteOne({ _id: userId }).lean().exec();
 };
 
 const findImportUsersBySchoolAndName = async (schoolId, firstName, lastName) => {

src/services/user-group/hooks/courses.js

@@ -60,7 +60,7 @@ const addWholeClassToCourse = async (hook) => {
 			const userIds = _.flattenDeep(groupUsers).map((groupUser) => groupUser.userId);
 			const uniqueUserIds = _.uniqWith(userIds, (a, b) => a === b);
 
-			await CourseModel.update({ _id: course._id }, { $addToSet: { userIds: { $each: uniqueUserIds } } }).exec();
+			await CourseModel.updateOne({ _id: course._id }, { $addToSet: { userIds: { $each: uniqueUserIds } } }).exec();
 
 			return undefined;
 		});
@@ -78,7 +78,7 @@ const addWholeClassToCourse = async (hook) => {
 			// flatten deep arrays and remove duplicates
 			studentIds = _.uniqWith(_.flattenDeep(studentIds), (e1, e2) => JSON.stringify(e1) === JSON.stringify(e2));
 
-			await CourseModel.update({ _id: course._id }, { $addToSet: { userIds: { $each: studentIds } } }).exec();
+			await CourseModel.updateOne({ _id: course._id }, { $addToSet: { userIds: { $each: studentIds } } }).exec();
 
 			return hook;
 		});
@@ -118,7 +118,7 @@ const deleteWholeClassFromCourse = (hook) => {
 					const userIds = _.flattenDeep(groupUsers).map((groupUser) => groupUser.userId);
 					const uniqueUserIds = _.uniqWith(userIds, (a, b) => a === b);
 
-					await CourseModel.update(
+					await CourseModel.updateOne(
 						{ _id: course._id },
 						{ $pull: { userIds: { $in: uniqueUserIds } } },
 						{ multi: true }
@@ -142,7 +142,7 @@ const deleteWholeClassFromCourse = (hook) => {
 				studentIds = _.uniqWith(_.flattenDeep(studentIds), (e1, e2) => JSON.stringify(e1) === JSON.stringify(e2));
 
 				// remove class students from course DB and from hook body to not patch them back
-				await CourseModel.update(
+				await CourseModel.updateOne(
 					{ _id: course._id },
 					{ $pull: { userIds: { $in: studentIds } } },
 					{ multi: true }