diff --git a/website/server/libs/user/index.js b/website/server/libs/user/index.js index b36c09bb9d..118e0586d9 100644 --- a/website/server/libs/user/index.js +++ b/website/server/libs/user/index.js @@ -237,7 +237,7 @@ export async function update (req, res, { isV3 = false }) { user.flags.lastNewStuffRead = lastNewsPost._id; } } else if (key === 'preferences.tasks.mirrorGroupTasks') { - user.preferences.mirrorGroupTasks = _.intersection(groupsToMirror, matchingGroupsArray); + user.preferences.tasks.mirrorGroupTasks = _.intersection(groupsToMirror, matchingGroupsArray); } else if (acceptablePUTPaths[key]) { let adjustedVal = val; if (key === 'stats.lvl' && val > common.constants.MAX_LEVEL_HARD_CAP) { diff --git a/website/server/models/group.js b/website/server/models/group.js index 143394eb89..4324a3545a 100644 --- a/website/server/models/group.js +++ b/website/server/models/group.js @@ -1475,8 +1475,7 @@ schema.methods.unlinkTask = async function groupUnlinkTask ( }; delete unlinkingTask.group.assignedUsersDetail[user._id]; - const assignedUserIndex = unlinkingTask.group.assignedUsers.indexOf(user._id); - unlinkingTask.group.assignedUsers.splice(assignedUserIndex, 1); + unlinkingTask.group.assignedUsers = _.keys(unlinkingTask.group.assignedUsersDetail); unlinkingTask.markModified('group'); const promises = [unlinkingTask.save()];