beet
beet copied to clipboard
Overlays output wrong resource names
When loading a pack with overlays in 1.21, the overlays will output older resource names (functions instead of function).
Notes
Likely a consequence of this code snippet: https://github.com/mcbeet/beet/blob/ba4a36283f28b6477f4321023d571e5f9345c20b/beet/library/base.py#L823-L826
When 1.21+ support was added, we needed a way to know how to output the correct resource naming since importing would support both the old and the new. A source map was introduced to figure out the correct naming based on the pack format. Since overlay packs don't keep track of a pack format, it defaults to the wrong one.
Recommended Changes
- Overlay packs should likely match the pack format that's listed in the
overlayssection of thepack.mcmetafile. This might be a range, so perhaps defaulting to the newest number in said range would be the best.- Inheriting the
pack_formatfrom the parent pack would likely end up with incorrect behavior as often you are writing overlays for older pack formats.
- Inheriting the