[release] Bump to version 56.2

- Fix bug in creating people when importing course plays

PROJECT.org

@@ -604,6 +604,12 @@ independent of the email flow it was originally creatdd for
 
 ** TODO [#B] Is there way to create unique slugs for media instances :media_types:
 
+* Version 56.2 [1/1]
+** DONE [#A] Fix bug in creating people when importing course plays :discgolf:bug:
+:PROPERTIES:
+:ID:       255e9886-098b-39ae-1077-25e43223660e
+:END:
+
 * Version 56.1 [1/1]
 ** DONE [#A] Add tests to discgolf app :discgolf:tests:
 :PROPERTIES:

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "vrobbler"
-version = "56.1"
+version = "56.2"
 description = ""
 authors = ["Colin Powell <colin@unbl.ink>"]
 

vrobbler/apps/discgolf/utils.py

@@ -15,9 +15,12 @@ def _parse_udisc_datetime(raw: str) -> datetime:
     return parse_datetime(raw)
 
 
-def _resolve_player(name: str) -> Person:
-    person, _ = Person.objects.get_or_create(name=name.strip())
-    return person
+def _resolve_player(name: str, user_id: int) -> Person:
+    name = name.strip()
+    existing = Person.objects.filter(name=name, created_by_id=user_id).first()
+    if existing:
+        return existing
+    return Person.objects.create(name=name, created_by_id=user_id)
 
 
 def import_udisc_csv(
@@ -95,10 +98,10 @@ def import_udisc_csv(
 
         if is_teams:
             people = player_name.split("+")
-            person_ids = [_resolve_player(p.strip()).id for p in people]
+            person_ids = [_resolve_player(p.strip(), user_id).id for p in people]
             hole_scores["person_ids"] = person_ids
         else:
-            person = _resolve_player(player_name)
+            person = _resolve_player(player_name, user_id)
             hole_scores["person_id"] = person.id
 
         scores[player_name] = hole_scores