No need to iterate over the categories if the user isn't tracking them

Author: alexiri

outlook/indico_outlook/plugin.py

@@ -376,16 +376,16 @@ def _record_change(self, event, user, action):
                 return
             if user in event.favorite_of and self._user_tracks_favorite_events(user):
                 return
-            for category in reversed(event.category.chain_query.all()):
-                if (
-                    user in category.favorite_of
-                    and self._user_tracks_favorite_categories(user)
-                    and event.can_access(user)
-                ):
-                    return
-                # Stop once we reach the visibility horizon of the event
-                if category is event.category.real_visibility_horizon:
-                    break
+            if self._user_tracks_favorite_categories(user):
+                for category in reversed(event.category.chain_query.all()):
+                    if (
+                        user in category.favorite_of
+                        and event.can_access(user)
+                    ):
+                        return
+                    # Stop once we reach the visibility horizon of the event
+                    if category is event.category.real_visibility_horizon:
+                        break
 
         g.outlook_changes.append((event, user, action))