Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Metadata Extraction from FFMPEG with Leading Zeros in Track Numbers and Mixed-Case Tags and Add Support for Discnumber #511

Merged
merged 2 commits into from
Jun 22, 2024

Conversation

yuygfgg
Copy link

@yuygfgg yuygfgg commented Jun 19, 2024

For Opus files with track numbers starting with 0 and mixed-case tag names, Airsonic fails to extract metadata. This occurs because airsonic strictly adheres to lowercase tags and java interprets numbers starting with 0 as octal numbers, causing track numbers 08 and 09 to be ignored.
截屏2024-06-19 21 33 57

IMAGE 2024-06-19 17:23:40

This patch addresses the issue by removing all leading zeros and making the tag comparison case-insensitive.
截屏2024-06-19 17 30 09

@yuygfgg yuygfgg changed the title Fix Metadata Extraction for Opus Files with Leading Zeros in Track Numbers and Mixed-Case Tags Fix Metadata Extraction from FFMPEG with Leading Zeros in Track Numbers and Mixed-Case Tags and Add Support for Discnumber Jun 20, 2024
@yuygfgg
Copy link
Author

yuygfgg commented Jun 20, 2024

also add support for muiti-disc albums

original:
IMAGE 2024-06-20 10:30:13

patched:
IMAGE 2024-06-20 10:30:29

Copy link
Owner

@kagemomiji kagemomiji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kagemomiji kagemomiji merged commit 8e6bcbf into kagemomiji:main Jun 22, 2024
13 checks passed
@yuygfgg yuygfgg deleted the patch-1 branch June 22, 2024 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants