Move to after we know not None

This commit is contained in:
Eric Eastwood 2022-12-23 14:20:06 -06:00
parent 0f2dda20b2
commit ea0489ba13

View file

@ -139,14 +139,6 @@ class ApplicationServiceScheduler:
to refresh the device lists of, or those that the application service need no to refresh the device lists of, or those that the application service need no
longer track the device lists of. longer track the device lists of.
""" """
# XXX: Special patch just for Gitter which we should remove after the import,
# https://github.com/matrix-org/synapse/pull/14729
#
# Ignore events that come from our own users. We probably already know about
# them and sent them ourself.
events = [event for event in events if not self._hs.is_mine_id(event.sender)]
# We purposefully allow this method to run with empty events/ephemeral # We purposefully allow this method to run with empty events/ephemeral
# collections, so that callers do not need to check iterable size themselves. # collections, so that callers do not need to check iterable size themselves.
if ( if (
@ -157,6 +149,13 @@ class ApplicationServiceScheduler:
): ):
return return
# XXX: Special patch just for Gitter which we should remove after the import,
# https://github.com/matrix-org/synapse/pull/14729
#
# Ignore events that come from our own users. We probably already know about
# them and sent them ourself.
events = [event for event in events if not self._hs.is_mine_id(event.sender)]
if events: if events:
self.queuer.queued_events.setdefault(appservice.id, []).extend(events) self.queuer.queued_events.setdefault(appservice.id, []).extend(events)
if ephemeral: if ephemeral: