Bug 479503 - Importing multidisc releases from MusicBrainz mixes tracks as a single release
Summary: Importing multidisc releases from MusicBrainz mixes tracks as a single release
Status: RESOLVED FIXED
Alias: None
Product: tellico
Classification: Applications
Component: general (show other bugs)
Version: 3.5.3
Platform: Arch Linux Linux
: NOR minor
Target Milestone: ---
Assignee: Robby Stephenson
URL:
Keywords:
: 479504 (view as bug list)
Depends on:
Blocks:
 
Reported: 2024-01-07 15:19 UTC by balseiro
Modified: 2025-02-08 21:39 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 4.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description balseiro 2024-01-07 15:19:24 UTC
SUMMARY

Importing a release with several discs shows a single tracklist instead of the track list per each disc.

STEPS TO REPRODUCE
1. In example, import this release https://musicbrainz.org/release/7a50a9e5-ca9c-41a8-9714-16313a1c27d8


OBSERVED RESULT

Single tracklist presented

EXPECTED RESULT

Track lists per disc
Comment 1 Tobias Fella 2024-01-07 17:15:31 UTC
*** Bug 479504 has been marked as a duplicate of this bug. ***
Comment 2 Robby Stephenson 2024-01-13 16:32:35 UTC
Git commit 4434a01752832cc1f7f6f56af47bd3b254fe3aa9 by Robby Stephenson.
Committed on 13/01/2024 at 17:32.
Pushed by rstephenson into branch 'master'.

Update Discogs, MusicBrainze, and iTunes to separate multi-disc tracks

Create new fields for each disc, similar to what the audio file
importer does. Add unit tests. One drawback is that the field will be
visible for all albums in the collection.

MusicBrainz doesn't have a consistent way of representing the disc and
track numbers, so a regular expression is used to try to parse them.
FIXED-IN: 4.0

M  +4    -0    ChangeLog
M  +37   -3    src/fetch/discogsfetcher.cpp
M  +48   -18   src/fetch/itunesfetcher.cpp
M  +2    -1    src/fetch/itunesfetcher.h
M  +1    -2    src/fetch/musicbrainzfetcher.cpp
M  +40   -0    src/tests/discogsfetchertest.cpp
M  +1    -0    src/tests/discogsfetchertest.h
M  +27   -2    src/tests/itunesfetchertest.cpp
M  +1    -0    src/tests/itunesfetchertest.h
M  +30   -0    src/tests/musicbrainzfetchertest.cpp
M  +1    -0    src/tests/musicbrainzfetchertest.h
M  +73   -24   xslt/musicbrainz2tellico.xsl

https://invent.kde.org/office/tellico/-/commit/4434a01752832cc1f7f6f56af47bd3b254fe3aa9
Comment 3 Robby Stephenson 2025-02-08 21:39:14 UTC
Git commit 17045967d67d32050d26295a3f966572738680db by Robby Stephenson.
Committed on 08/02/2025 at 21:35.
Pushed by rstephenson into branch '4.1'.

Add hidden option to collapse multi-disc tracks into single field

This provides the behavior from before Tellico 4.0 and instead of
creating multiple fields for each disc, it collapses all tracks
together.

To enable, open the config file, find the Data Source group for the
Discogs, Musicbrainz or itunes source, and add
Split Tracks By Disc=false
Related: bug 499401
FIXED-IN: 4.1.1

M  +4    -0    ChangeLog
M  +16   -2    src/fetch/discogsfetcher.cpp
M  +1    -0    src/fetch/discogsfetcher.h
M  +15   -1    src/fetch/itunesfetcher.cpp
M  +1    -0    src/fetch/itunesfetcher.h
M  +24   -3    src/fetch/musicbrainzfetcher.cpp
M  +1    -0    src/fetch/musicbrainzfetcher.h
M  +37   -1    src/tests/discogsfetchertest.cpp
M  +1    -0    src/tests/discogsfetchertest.h
M  +24   -0    src/tests/itunesfetchertest.cpp
M  +1    -0    src/tests/itunesfetchertest.h
M  +28   -0    src/tests/musicbrainzfetchertest.cpp
M  +1    -0    src/tests/musicbrainzfetchertest.h

https://invent.kde.org/office/tellico/-/commit/17045967d67d32050d26295a3f966572738680db