[release] Bump to version 59.5

- Fix bug where all variants for board games are in form

PROJECT.org

@@ -88,7 +88,7 @@ fetching and simple saving.
 *** Metadata sources
 **** Scraper
 
-* Backlog [0/26] :vrobbler:project:personal:
+* Backlog [0/24] :vrobbler:project:personal:
 ** TODO [#C] After transition to linux add curl_cffi as webpage scrapper again :webpages:metadata:
 ** TODO [#C] Create small utility to clean up tracks scrobbled with wonky playback times :bug:music:scrobbles:
 :PROPERTIES:
@@ -619,7 +619,14 @@ The Edit log form should have from top to bottom:
   - Expansion ids (which should a multi-select widget of expansions for this game)
   - Location (which should be a drop down of BoardGameLocations for this user)
 
-** TODO Fix bug in fetching expansions for board games :boardgames:
+* Version 59.5 [1/1]
+** DONE [#A] Fix bug where all variants for board games are in form :boardgames:
+:PROPERTIES:
+:ID:       9c4cd193-580a-5b33-8832-1feffea7bd53
+:END:
+
+* Version 59.4 [2/2]
+** DONE Fix bug in fetching expansions for board games :boardgames:
 :PROPERTIES:
 :ID:       17995312-e76e-4a50-b591-0eab78cb59ab
 :END:
@@ -642,11 +649,12 @@ The Edit log form should have from top to bottom:
 boardgamegeek.exceptions.BGGApiError: invalid data for game id: 242117
 #+end_src
 
-** TODO Board games should have genres extracted from family data :boardgames:metadata:
+** DONE Board games should have genres extracted from family data :boardgames:metadata:
 :PROPERTIES:
 :ID:       7214b270-dccc-4b98-ac58-ff4f76c8cda9
 :END:
 
+
 * Version 59.3 [2/2]
 ** DONE Exclude some board games from auto-expansion imports :boardgames:
 :PROPERTIES:

pyproject.toml

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

vrobbler/apps/boardgames/models.py

@@ -442,8 +442,8 @@ class BoardGame(ScrobblableMixin):
             game.publisher = publisher
 
         skip_expansions = (
-            game.bggeek_id
-            and game.bggeek_id.isdigit()
+            game.bggeek_id is not None
+            and str(game.bggeek_id).isdigit()
             and int(game.bggeek_id) in settings.SKIP_AUTO_EXPANSION_DOWNLOAD
         ) or any(
             c == "Collectible Card Games" for c in categories

vrobbler/apps/scrobbles/views.py

@@ -51,6 +51,7 @@ from music.aggregators import (
     scrobble_counts,
     week_of_scrobbles,
 )
+from boardgames.models import BoardGame, BoardGameVariant
 from pendulum.parsing.exceptions import ParserError
 from profiles.models import UserProfile
 from profiles.utils import now_user_timezone
@@ -1236,8 +1237,6 @@ class ScrobbleDetailView(DetailView):
         return self.object.media_obj.logdata_cls().form()
 
     def _update_board_game_widgets(self, form):
-        from boardgames.models import BoardGame
-
         if not isinstance(self.object.media_obj, BoardGame):
             return
 
@@ -1248,6 +1247,9 @@ class ScrobbleDetailView(DetailView):
             form.fields["expansion_ids"].queryset = expansions
 
         if "variant_ids" in form.fields:
+            form.fields["variant_ids"].queryset = BoardGameVariant.objects.filter(
+                board_game=self.object.media_obj
+            )
             form.fields["variant_ids"].widget.attrs["data-board-game-id"] = (
                 self.object.media_obj.id
             )
@@ -1957,8 +1959,6 @@ class EmbeddableTopBoardGamesWidget(BaseEmbeddableWidget):
     scrobble_filter = {"scrobble__played_to_completion": True}
 
     def get_items(self, user, start_date, end_date):
-        from boardgames.models import BoardGame
-
         return super().get_items(user, start_date, end_date, BoardGame)