diff options
author | Benjamin Tissoires <benjamin.tissoires@redhat.com> | 2019-01-25 14:10:10 +0100 |
---|---|---|
committer | Benjamin Tissoires <benjamin.tissoires@redhat.com> | 2019-01-25 14:15:08 +0100 |
commit | 8d81aca4398c29eaa5236b92053c33a957b1bcf4 (patch) | |
tree | cb95da3e5fc1df4de1c11c0a4d7f199919c0d64f /models | |
parent | 542cc75dc4a4bf392cac72345ab013cf59c67ad3 (diff) | |
download | wekan-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')
-rw-r--r-- | models/boards.js | 9 |
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, |