dedex icon indicating copy to clipboard operation
dedex copied to clipboard

Release Album Assignment Change

Open onelazydev opened this issue 2 years ago • 0 comments

Previously, only an "Album" or "ClassicalAlbum" could be assigned to ddexReleaseAlbum within the SimpleAlbum class.

As per the Release Type List (https://kb.ddex.net/display/dsrf30avs/5.5+ReleaseType), that does not cover all instances of music.

At a minimum you would need Album, ClassicalAlbum and Single, however you also have to handle userDefined values too.

Given that DDEX state (https://kb.ddex.net/display/HBK/Release+Types+for+TrackReleases) that track releases must use TrackRelease on tracks within a release, it makes slightly more sense to flip the logic and apply the release status to the Release that is not TrackRelease.

Added a small test for UserDefined EP and all tests pass.

onelazydev avatar Nov 04 '22 11:11 onelazydev