beetbox / beets

music library manager and MusicBrainz tagger
http://beets.io/
MIT License
12.92k stars 1.82k forks source link

fix(import): don't throw away album flexible fields #5388

Closed ThinkChaos closed 2 months ago

ThinkChaos commented 3 months ago

Description

I believe this fixes #4785, but I might be misunderstanding the description and it might already be fixed.

Tagging @JOJ0 since they created both the bug linked above, and 5bf4e3d92f5b4010a19cd3e8222fbf28b7b844e0 which mentions it.

To Do

ThinkChaos commented 2 months ago

Thanks for the review!

JOJ0 commented 2 months ago

@ThinkChaos thank you very much for this PR! I love the refactoring you did to the *_id fetching code! So much cleaner and shorter now! Also many thanks to @snejus for taking over with a review, much appreciated! ❤️