summary refs log tree commit diff
diff options
context:
space:
mode:
authorMark Haines <mark.haines@matrix.org>2015-12-02 10:49:35 +0000
committerMark Haines <mark.haines@matrix.org>2015-12-02 10:49:35 +0000
commitc30cdb0d6881a3f463574043637072b71d1a3ab2 (patch)
treed1c280444a732d24143617695f01efe2ba5e973c
parentMerge branch 'develop' into markjh/edu_frequency (diff)
downloadsynapse-c30cdb0d6881a3f463574043637072b71d1a3ab2.tar.xz
Add comments
-rw-r--r--synapse/handlers/federation.py3
-rw-r--r--synapse/handlers/room.py3
2 files changed, 6 insertions, 0 deletions
diff --git a/synapse/handlers/federation.py b/synapse/handlers/federation.py
index a03a5f494c..5f3562b5b5 100644
--- a/synapse/handlers/federation.py
+++ b/synapse/handlers/federation.py
@@ -239,6 +239,9 @@ class FederationHandler(BaseHandler):
                 )
                 prev_state = context.current_state.get((event.type, event.state_key))
                 if not prev_state or prev_state.membership != Membership.JOIN:
+                    # Only fire user_joined_room if the user has acutally
+                    # joined the room. Don't bother if the user is just
+                    # changing their profile info.
                     user = UserID.from_string(event.state_key)
                     yield user_joined_room(self.distributor, user, event.room_id)
 
diff --git a/synapse/handlers/room.py b/synapse/handlers/room.py
index 1e18038e15..116a998c42 100644
--- a/synapse/handlers/room.py
+++ b/synapse/handlers/room.py
@@ -527,6 +527,9 @@ class RoomMemberHandler(BaseHandler):
 
         prev_state = context.current_state.get((event.type, event.state_key))
         if not prev_state or prev_state.membership != Membership.JOIN:
+            # Only fire user_joined_room if the user has acutally joined the
+            # room. Don't bother if the user is just changing their profile
+            # info.
             user = UserID.from_string(event.user_id)
             yield user_joined_room(self.distributor, user, room_id)