Merge pull request #54 from matrix-org/kegan/hide-fs-users

Do not show conf user join/parts
matrix-org · Dec 16, 2015 · 186b727 · 186b727
2 parents 524563d + 09e0d10
commit 186b727
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/components/structures/RoomView.js b/src/components/structures/RoomView.js
@@ -600,6 +600,12 @@ module.exports = React.createClass({
             if (!EventTile.haveTileForEvent(mxEv)) {
                 continue;
             }
+            if (this.props.ConferenceHandler && mxEv.getType() === "m.room.member") {
+                if (this.props.ConferenceHandler.isConferenceUser(mxEv.getSender()) ||
+                        this.props.ConferenceHandler.isConferenceUser(mxEv.getStateKey())) {
+                    continue; // suppress conf user join/parts
+                }
+            }
 
             var continuation = false;
             var last = false;

diff --git a/src/components/views/rooms/RoomList.js b/src/components/views/rooms/RoomList.js
@@ -182,7 +182,7 @@ module.exports = React.createClass({
                             return m.userId !== me.userId
                         })[0];
                         var ConfHandler = self.props.ConferenceHandler;
-                        if (ConfHandler && ConfHandler.isConferenceUser(otherMember)) {
+                        if (ConfHandler && ConfHandler.isConferenceUser(otherMember.userId)) {
                             // console.log("Hiding conference 1:1 room %s", room.roomId);
                             shouldShowRoom = false;
                         }