summaryrefslogtreecommitdiffstats
path: root/models/boards.js
diff options
context:
space:
mode:
authorBenjamin Tissoires <benjamin.tissoires@redhat.com>2019-01-25 14:10:10 +0100
committerBenjamin Tissoires <benjamin.tissoires@redhat.com>2019-01-25 14:15:08 +0100
commit8d81aca4398c29eaa5236b92053c33a957b1bcf4 (patch)
treecb95da3e5fc1df4de1c11c0a4d7f199919c0d64f /models/boards.js
parent542cc75dc4a4bf392cac72345ab013cf59c67ad3 (diff)
downloadwekan-8d81aca4398c29eaa5236b92053c33a957b1bcf4.tar.gz
wekan-8d81aca4398c29eaa5236b92053c33a957b1bcf4.tar.bz2
wekan-8d81aca4398c29eaa5236b92053c33a957b1bcf4.zip
api: fix set_board_member_permission
If the data is passed as a boolean, through json, data.toLowerCase() raises an error. Also define query which we are returning in case of success.
Diffstat (limited to 'models/boards.js')
-rw-r--r--models/boards.js9
1 files changed, 7 insertions, 2 deletions
diff --git a/models/boards.js b/models/boards.js
index 99480ca7..1d6472cc 100644
--- a/models/boards.js
+++ b/models/boards.js
@@ -1113,9 +1113,14 @@ if (Meteor.isServer) {
Authentication.checkBoardAccess(req.userId, boardId);
const board = Boards.findOne({ _id: boardId });
function isTrue(data){
- return data.toLowerCase() === 'true';
+ try {
+ return data.toLowerCase() === 'true';
+ }
+ catch (error) {
+ return data;
+ }
}
- board.setMemberPermission(memberId, isTrue(isAdmin), isTrue(isNoComments), isTrue(isCommentOnly), req.userId);
+ const query = board.setMemberPermission(memberId, isTrue(isAdmin), isTrue(isNoComments), isTrue(isCommentOnly), req.userId);
JsonRoutes.sendResult(res, {
code: 200,