Updating results can overwrite existing results if desired

Author: dakrauth

src/picker/migrations/0017_alter_team_options_remove_pickerfavorite_user_and_more.py

@@ -0,0 +1,31 @@
+# Generated by Django 5.2.4 on 2025-11-26 01:25
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("picker", "0016_picker"),
+    ]
+
+    operations = [
+        migrations.AlterModelOptions(
+            name="team",
+            options={"base_manager_name": "objects", "ordering": ("name",)},
+        ),
+        migrations.RemoveField(
+            model_name="pickerfavorite",
+            name="user",
+        ),
+        migrations.RemoveField(
+            model_name="pickermembership",
+            name="user",
+        ),
+        migrations.RemoveField(
+            model_name="pickset",
+            name="user",
+        ),
+        migrations.DeleteModel(
+            name="Preference",
+        ),
+    ]

src/picker/models/picks.py

@@ -518,7 +518,7 @@ def _update_points(self, result):
                 self.points = score
                 self.save()
 
-    def update_results(self, results):
+    def update_results(self, results, overwrite=False):
         """
         results schema: {'sequence': 1, 'season': 2018, 'games': [{
             "home": "HOME",
@@ -544,12 +544,30 @@ def update_results(self, results):
 
         count = 0
         all_games = list(self.games.select_related("home", "away"))
-        incomplete_games = [
-            g for g in all_games if g.home.abbr in completed and g.status == g.Status.UNPLAYED
-        ]
-        for game in incomplete_games:
+
+        def get_score(value):
+            if isinstance(value, int):
+                return value
+
+            if isinstance(value, str) and value.isdigit():
+                return int(value)
+
+            return 0
+
+        if overwrite:
+            games_to_update = all_games
+        else:
+            games_to_update = [
+                    g
+                    for g in all_games
+                    if g.home and (g.home.abbr in completed) and g.status == g.Status.UNPLAYED
+                ]
+
+        for game in games_to_update:
             result = completed.get(game.home.abbr, None)
             if result:
+                game.home_score = get_score(result.get("home_score"))
+                game.away_score = get_score(result.get("away_score"))
                 winner = result["winner"]
                 game.winner = (
                     game.home
@@ -573,7 +591,9 @@ def winners(self, group):
                     yield r.picker
 
     def pickset_query(self, group):
-        return self.picksets.filter(picker__picker_memberships__group=group).select_related("picker")
+        return self.picksets.filter(picker__picker_memberships__group=group).select_related(
+            "picker"
+        )
 
     def update_pick_status(self):
         for group in PickerGrouping.objects.active():

src/picker/models/sports.py

@@ -534,7 +534,11 @@ class Meta:
         ordering = ("start_time", "away")
 
     def __str__(self):
-        return "{} @ {} {}".format(self.away.abbr, self.home.abbr, self.gameset)
+        return "{} @ {} {}".format(
+            self.away.abbr if self.away else "TBD",
+            self.home.abbr if self.home else "TBD",
+            self.gameset,
+        )
 
     def to_dict(self):
         return {

src/picker/templates/picker/_base/results/season.html

@@ -16,7 +16,7 @@ <h1>{{ season }} Results</h1>
             </td>
             <td>{% for w in winners %}
                 <div style="display: inline-block; margin-left: 8px;">
-                    <strong class="pickername">{{ w.picker }}</strong>
+                    <strong class="pickername">{{ w }}</strong>
                 </div>{% if not forloop.last %}, {% endif %}
                 {% endfor %}
             </td>