diff options
author | Erik Johnston <erik@matrix.org> | 2016-06-03 10:58:24 +0100 |
---|---|---|
committer | Erik Johnston <erik@matrix.org> | 2016-06-03 10:58:24 +0100 |
commit | 696d7c59377a5ae9f2613f7cb44c304e56c68b9b (patch) | |
tree | 537ee15a637018443cef4b2d13fdb043ad2c4182 /synapse/push | |
parent | Merge pull request #821 from matrix-org/dbkr/email_unsubscribe (diff) | |
parent | Add get_users_with_read_receipts_in_room cache (diff) | |
download | synapse-696d7c59377a5ae9f2613f7cb44c304e56c68b9b.tar.xz |
Merge pull request #809 from matrix-org/erikj/cache_receipts_in_room
Add get_users_with_read_receipts_in_room cache
Diffstat (limited to 'synapse/push')
-rw-r--r-- | synapse/push/bulk_push_rule_evaluator.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/synapse/push/bulk_push_rule_evaluator.py b/synapse/push/bulk_push_rule_evaluator.py index 25f2fb9da4..1e5c4b073c 100644 --- a/synapse/push/bulk_push_rule_evaluator.py +++ b/synapse/push/bulk_push_rule_evaluator.py @@ -87,13 +87,13 @@ def evaluator_for_event(event, hs, store): all_in_room = yield store.get_users_in_room(room_id) all_in_room = set(all_in_room) - receipts = yield store.get_receipts_for_room(room_id, "m.read") + users_with_receipts = yield store.get_users_with_read_receipts_in_room(room_id) # any users with pushers must be ours: they have pushers user_ids = set(users_with_pushers) - for r in receipts: - if hs.is_mine_id(r['user_id']) and r['user_id'] in all_in_room: - user_ids.add(r['user_id']) + for uid in users_with_receipts: + if hs.is_mine_id(uid) and uid in all_in_room: + user_ids.add(uid) # if this event is an invite event, we may need to run rules for the user # who's been invited, otherwise they won't get told they've been invited |