fixed runtime error when iterating over cache and deleting elements

SpeedProg · Nov 24, 2020 · 44a1449 · 44a1449
1 parent b76d899
commit 44a1449
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/waitlist/utility/timetracking/timer.py b/waitlist/utility/timetracking/timer.py
@@ -84,7 +84,7 @@ def check_fleets(self):
                         self.register_fleet_time(fleet_id, self.cache[fleet_id])
                         del self.cache[fleet_id]
 
-                for fleet_id in self.cache.keys():
+                for fleet_id in list(self.cache.keys()):
                     logger.debug('Checking members in fleet with id=%s', fleet_id)
                     if fleet_id in fleet_ids:
                         # these ones we need to check for missing members, because they left the fleet
@@ -96,7 +96,7 @@ def check_fleets(self):
                         # if we get stale data, because e.g. we have no valid api key
                         # just skip this fleet
                         if fleet_new_data is None:
-                            logger.info('Fleet with id=%s is not in cache anymore also its key still exists in database, removing', fleet_id)
+                            logger.info('Fleet with id=%s is not in global fleet cache anymore although its key still exists in our cache, removing', fleet_id)
                             # the fleet disappeared register remaining mebers time
                             self.register_fleet_time(fleet_id, self.cache[fleet_id])
                             del self.cache[fleet_id]