[scrobbles] Fix LookupError in scrobble start and long-plays media lookups (868e2a91)
This commit is contained in:
17
PROJECT.org
17
PROJECT.org
@ -746,6 +746,23 @@ Import gym sessions and routines from a Flexify backup.
|
||||
- `WorkoutImport` model (`BaseFileImportMixin`, new `file_hash` field for
|
||||
change detection), admin, DRF `workout-imports` viewset, planned-exercises
|
||||
block on the routine detail template, and MCP routine dict update.
|
||||
** TODO [#B] Fix LookupError on scrobble start/long-plays views :bug:scrobbles:videos:
|
||||
:PROPERTIES:
|
||||
:ID: 868e2a91-c974-4656-a8ae-b90a1f94ae74
|
||||
:END:
|
||||
|
||||
- `PLAY_AGAIN_MEDIA["channels"]` and `LONG_PLAY_MEDIA["papers"]` keyed models
|
||||
by app labels that don't exist (`Channel` lives in `videos`, `Paper` in
|
||||
`books`), so `scrobble_start`, `scrobble_longplay_finish`, and the long-plays
|
||||
views raised `LookupError: No installed app with label 'channels'` for every
|
||||
request.
|
||||
- `scrobble_start` now special-cases `channels` to the `videos` app and skips
|
||||
unknown app labels; a shared `get_long_play_media_model()` helper in
|
||||
`scrobbles/utils.py` maps the `papers` label to `books` for the long-play
|
||||
lookups.
|
||||
- Also fixed `Channel.scrobble_for_user` which still used the pre-refactor
|
||||
`Scrobble.create_or_update(media_object=...)` signature (would raise
|
||||
`TypeError` once the lookup reached it).
|
||||
* Version 64.5 [1/1]
|
||||
** DONE [#B] Clean up issues with org-mode notes :orgmode:notes:scrobbles:
|
||||
:PROPERTIES:
|
||||
|
||||
Reference in New Issue
Block a user