diff options
| author | Patrick Cloke <clokep@users.noreply.github.com> | 2021-01-05 10:53:15 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-01-05 10:53:15 -0500 |
| commit | 37eaf9c27271197320d4bedcceaf58d746935e53 (patch) | |
| tree | 4758ca9622e660351b56f7c6e47bacbd39550faf /synapse/push/emailpusher.py | |
| parent | Add type hints to the receipts and user directory handlers. (#8976) (diff) | |
| download | synapse-37eaf9c27271197320d4bedcceaf58d746935e53.tar.xz | |
Fix-up assertions about last stream token in push (#9020)
The last stream token is always known and we do not need to handle none.
Diffstat (limited to 'synapse/push/emailpusher.py')
| -rw-r--r-- | synapse/push/emailpusher.py | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/synapse/push/emailpusher.py b/synapse/push/emailpusher.py index d2eff75a58..4ac1b31748 100644 --- a/synapse/push/emailpusher.py +++ b/synapse/push/emailpusher.py @@ -157,7 +157,6 @@ class EmailPusher(Pusher): being run. """ start = 0 if INCLUDE_ALL_UNREAD_NOTIFS else self.last_stream_ordering - assert start is not None unprocessed = await self.store.get_unread_push_actions_for_user_in_range_for_email( self.user_id, start, self.max_stream_ordering ) @@ -220,12 +219,8 @@ class EmailPusher(Pusher): ) async def save_last_stream_ordering_and_success( - self, last_stream_ordering: Optional[int] + self, last_stream_ordering: int ) -> None: - if last_stream_ordering is None: - # This happens if we haven't yet processed anything - return - self.last_stream_ordering = last_stream_ordering pusher_still_exists = await self.store.update_pusher_last_stream_ordering_and_success( self.app_id, |