diff options
author | Maxime Quandalle <maxime@quandalle.com> | 2015-09-05 01:31:25 +0200 |
---|---|---|
committer | Maxime Quandalle <maxime@quandalle.com> | 2015-09-05 01:37:55 +0200 |
commit | 82b322777852b0f7b3f93295a0dbcc0c238bd20d (patch) | |
tree | cb75983a021909ac96f99bdb1448cd2ebee72b48 /client/components/sidebar/sidebar.js | |
parent | 6a92963e2a79e2819b7492a0389c1c48a9467ba3 (diff) | |
download | wekan-82b322777852b0f7b3f93295a0dbcc0c238bd20d.tar.gz wekan-82b322777852b0f7b3f93295a0dbcc0c238bd20d.tar.bz2 wekan-82b322777852b0f7b3f93295a0dbcc0c238bd20d.zip |
Fix member permission modification
Fixes #280
Diffstat (limited to 'client/components/sidebar/sidebar.js')
-rw-r--r-- | client/components/sidebar/sidebar.js | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/client/components/sidebar/sidebar.js b/client/components/sidebar/sidebar.js index 51f3e0e4..ccc7e5aa 100644 --- a/client/components/sidebar/sidebar.js +++ b/client/components/sidebar/sidebar.js @@ -206,7 +206,7 @@ Template.addMemberPopup.helpers({ }); Template.addMemberPopup.events({ - 'click .pop-over-member-list li:not(.disabled)'() { + 'click .js-select-member'() { const userId = this._id; const currentBoard = Boards.findOne(Session.get('currentBoard')); const currentMembersIds = _.pluck(currentBoard.members, 'userId'); @@ -233,14 +233,14 @@ Template.addMemberPopup.events({ }, }); -Template.addMemberPopup.onRendered(() => { +Template.addMemberPopup.onRendered(function() { this.find('.js-search-member input').focus(); }); Template.changePermissionsPopup.events({ 'click .js-set-admin, click .js-set-normal'(event) { const currentBoard = Boards.findOne(Session.get('currentBoard')); - const memberIndex = getMemberIndex(currentBoard, this.user._id); + const memberIndex = getMemberIndex(currentBoard, this.userId); const isAdmin = $(event.currentTarget).hasClass('js-set-admin'); Boards.update(currentBoard._id, { @@ -254,10 +254,13 @@ Template.changePermissionsPopup.events({ Template.changePermissionsPopup.helpers({ isAdmin() { - return this.user.isBoardAdmin(); + const user = Users.findOne(this.userId); + return user.isBoardAdmin(); }, + isLastAdmin() { - if (!this.user.isBoardAdmin()) + const user = Users.findOne(this.userId); + if (!user.isBoardAdmin()) return false; const currentBoard = Boards.findOne(Session.get('currentBoard')); const nbAdmins = _.where(currentBoard.members, { isAdmin: true }).length; |