cloud/music_library
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix top artists and top album counts

Browse Source 
Before, each track count would be multiplied by the number of collected
records. Not a problem when there's just one, but if I have more than
one then it would get weird quickly.
This commit is contained in:
Claudio Ortolina
2025-10-07 15:56:17 +03:00
parent 7412c89011
commit 61aeef4de1
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/music_library/scrobble_activity.ex
+4 -4
View File
@@ -231,7 +231,7 @@ defmodule MusicLibrary.ScrobbleActivity do
album_title: fragment("json_extract(album, '$.title')"), album_title: fragment("json_extract(album, '$.title')"),
artist_name: fragment("json_extract(artist, '$.name')"), artist_name: fragment("json_extract(artist, '$.name')"),
artist_musicbrainz_id: fragment("json_extract(artist, '$.musicbrainz_id')"), artist_musicbrainz_id: fragment("json_extract(artist, '$.musicbrainz_id')"),
play_count: count(t.scrobbled_at_uts), play_count: count(t.scrobbled_at_uts, :distinct),
cover_url: fragment("max(?)", t.cover_url), cover_url: fragment("max(?)", t.cover_url),
album_musicbrainz_id: fragment("json_extract(album, '$.musicbrainz_id')"), album_musicbrainz_id: fragment("json_extract(album, '$.musicbrainz_id')"),
collected_record_id: cr.record_id, collected_record_id: cr.record_id,
@@ -266,7 +266,7 @@ defmodule MusicLibrary.ScrobbleActivity do
album_title: fragment("json_extract(album, '$.title')"), album_title: fragment("json_extract(album, '$.title')"),
artist_name: fragment("json_extract(artist, '$.name')"), artist_name: fragment("json_extract(artist, '$.name')"),
artist_musicbrainz_id: fragment("json_extract(artist, '$.musicbrainz_id')"), artist_musicbrainz_id: fragment("json_extract(artist, '$.musicbrainz_id')"),
play_count: count(t.scrobbled_at_uts), play_count: count(t.scrobbled_at_uts, :distinct),
cover_url: fragment("max(?)", t.cover_url), cover_url: fragment("max(?)", t.cover_url),
album_musicbrainz_id: fragment("json_extract(album, '$.musicbrainz_id')"), album_musicbrainz_id: fragment("json_extract(album, '$.musicbrainz_id')"),
collected_record_id: cr.record_id, collected_record_id: cr.record_id,
@@ -298,7 +298,7 @@ defmodule MusicLibrary.ScrobbleActivity do
name: fragment("json_extract(artist, '$.name')"), name: fragment("json_extract(artist, '$.name')"),
musicbrainz_id: fragment("json_extract(artist, '$.musicbrainz_id')"), musicbrainz_id: fragment("json_extract(artist, '$.musicbrainz_id')"),
image_hash: ai.image_data_hash, image_hash: ai.image_data_hash,
play_count: count(t.scrobbled_at_uts) play_count: count(t.scrobbled_at_uts, :distinct)
}, },
order_by: [desc: count(t.scrobbled_at_uts)], order_by: [desc: count(t.scrobbled_at_uts)],
limit: ^limit limit: ^limit
@@ -335,7 +335,7 @@ defmodule MusicLibrary.ScrobbleActivity do
name: fragment("json_extract(artist, '$.name')"), name: fragment("json_extract(artist, '$.name')"),
musicbrainz_id: fragment("json_extract(artist, '$.musicbrainz_id')"), musicbrainz_id: fragment("json_extract(artist, '$.musicbrainz_id')"),
image_hash: ai.image_data_hash, image_hash: ai.image_data_hash,
play_count: count(t.scrobbled_at_uts) play_count: count(t.scrobbled_at_uts, :distinct)
}, },
order_by: [desc: count(t.scrobbled_at_uts)], order_by: [desc: count(t.scrobbled_at_uts)],
limit: ^limit limit: ^limit