From d255348762f1de897ed1a9f8f5f28f3cbead67e5 Mon Sep 17 00:00:00 2001 From: David Baker Date: Thu, 2 Nov 2017 13:26:39 +0000 Subject: [PATCH] Remove leave check 1. It's wrong because it doesn't consider the ban state 2. This should be caught by whether they have permission to send the event in the first place rather than worrying about whether it has notification permission or not --- src/models/room-state.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/models/room-state.js b/src/models/room-state.js index 3a995dad4..bed6d9417 100644 --- a/src/models/room-state.js +++ b/src/models/room-state.js @@ -403,7 +403,7 @@ RoomState.prototype._maySendEventOfType = function(eventType, userId, state) { */ RoomState.prototype.mayTriggerNotifOfType = function(notifLevelKey, userId) { const member = this.getMember(userId); - if (!member || member.membership == 'leave') { + if (!member) { return false; }