github-mirrors/wekan
1
0
Fork 0
mirror of https://github.com/wekan/wekan.git synced 2025-04-22 21:17:18 -04:00
Code Issues Projects Releases Packages Wiki Activity

Prevent data loss.

Browse source 
Thanks to xet7 !
This commit is contained in:
Lauri Ojansivu 2019-05-09 16:17:53 +03:00
parent c850669777
commit e1b016cf3d
1 changed files with 16 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

26
models/users.js
View file

@ -690,16 +690,22 @@ if (Meteor.isServer) {
}, {unique: true});
});
Users.before.remove((userId, doc) => {
Boards
.find({members: {$elemMatch: {userId: doc._id, isAdmin: true}}})
.forEach((board) => {
// If only one admin for the board
if (board.members.filter((e) => e.isAdmin).length === 1) {
Boards.remove(board._id);
}
});
});
// OLD WAY THIS CODE DID WORK: When user is last admin of board,
// if admin is removed, board is removed.
// NOW THIS IS COMMENTED OUT, because other board users still need to be able
// to use that board, and not have board deleted.
// Someone can be later changed to be admin of board, by making change to database.
// TODO: Add UI for changing someone as board admin.
//Users.before.remove((userId, doc) => {
// Boards
// .find({members: {$elemMatch: {userId: doc._id, isAdmin: true}}})
// .forEach((board) => {
// // If only one admin for the board
// if (board.members.filter((e) => e.isAdmin).length === 1) {
// Boards.remove(board._id);
// }
// });
//});
// Each board document contains the de-normalized number of users that have
// starred it. If the user star or unstar a board, we need to update this