From cffbd30d7e3ef451039082944e0f8f88f36d5150 Mon Sep 17 00:00:00 2001 From: Xeniac Date: Thu, 23 Nov 2017 22:01:38 +0100 Subject: [PATCH] Fixed: enumerate the Seasonrankings starting with 1 Fixed: Logging error when a value changed from/to None --- src/mahjong_ranking/managers.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mahjong_ranking/managers.py b/src/mahjong_ranking/managers.py index 08560b4..674f734 100644 --- a/src/mahjong_ranking/managers.py +++ b/src/mahjong_ranking/managers.py @@ -184,7 +184,7 @@ class SeasonRankingManager(models.Manager): if until or force_recalc: for ranking in self.filter(season=season): ranking.recalculate(until=until) - for placement, ranking in enumerate(self.season_rankings(season)): + for placement, ranking in enumerate(self.season_rankings(season), start=1): ranking.placement = placement ranking.save(force_update=True, update_fields=['placement']) @@ -249,5 +249,5 @@ class KyuDanRankingManager(models.Manager): LOGGER.warning( "%(user)s recalc shows differences in %(attr)s! old: %(old)d, new: %(new)d", {'user': ranking.user, 'attr': attr, - 'old': old_value, 'new': getattr(ranking, attr)} + 'old': old_value or 0, 'new': getattr(ranking, attr) or 0} )