diff options
| author | Luke Barnard <lukeb@openmarket.com> | 2017-04-12 14:36:20 +0100 |
|---|---|---|
| committer | Luke Barnard <lukeb@openmarket.com> | 2017-04-12 14:36:20 +0100 |
| commit | b9557064bf6003a666b8fb6813dd3618fe9e48b4 (patch) | |
| tree | ce2cee50fe81f42be61fcc6591f7b337c13580bb /synapse/storage | |
| parent | More null-guard changes (diff) | |
| download | synapse-b9557064bf6003a666b8fb6813dd3618fe9e48b4.tar.xz | |
Simplify is_event_after logic
Diffstat (limited to 'synapse/storage')
| -rw-r--r-- | synapse/storage/events.py | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/synapse/storage/events.py b/synapse/storage/events.py index 702bd64b2e..221cb563d8 100644 --- a/synapse/storage/events.py +++ b/synapse/storage/events.py @@ -2161,18 +2161,11 @@ class EventsStore(SQLBaseStore): @defer.inlineCallbacks def is_event_after(self, event_id1, event_id2): - is_after = True - + """Returns True if event_id1 is after event_id2 in the stream + """ to_1, so_1 = yield self._get_event_ordering(event_id1) to_2, so_2 = yield self._get_event_ordering(event_id2) - - # Prevent updating if the existing marker is ahead in the stream - if to_1 > to_2: - is_after = False - elif to_1 == to_2 and so_1 >= so_2: - is_after = False - - defer.returnValue(is_after) + defer.returnValue(to_1 > to_2 and so_1 > so_2) @defer.inlineCallbacks def _get_event_ordering(self, event_id): |