Merge pull request #606 from terceiro/review-score

Exclude NULLs from talk review score average (this is more sensible and matches the SQL average displayed in the admin interface).
CTPUG · Jul 6, 2021 · 6b0cafe · 6b0cafe
2 parents 63f40f3 + 19cb402
commit 6b0cafe
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/wafer/talks/models.py b/wafer/talks/models.py
@@ -309,7 +309,7 @@ def is_late_submission(self):
     @property
     def review_score(self):
         # Overridden in admin, to allow sorting
-        reviews = [review.avg_score for review in self.reviews.all() if review.avg_score]
+        reviews = [review.avg_score for review in self.reviews.all() if review.avg_score is not None]
         if not reviews:
             return None
         return sum(reviews) / len(reviews)