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

[Bug] Slash character ("/") in artist tag makes it appear as 2 separate artists. #723

Closed
1 task done
Zany-Jokengineer opened this issue Dec 23, 2024 · 4 comments
Closed
1 task done
Labels
state: done Has been fixed or implemented type: bug Something isn't working
Milestone

Comments

@Zany-Jokengineer
Copy link

Zany-Jokengineer commented Dec 23, 2024

Description

As you can see, the name of the band AC/DC appears as two separate artists - AC and DC; changing the tag to "AC/DC" in quotation marks doesn't help - it is interpreted as artists "AC and DC".

image

Steps to Reproduce

Add an .opus song with AC/DC in the artist tag to one of the configured folders, rescan, and confirm AC and DC "artists" added.

Expected Behavior

The slash character should not cause the artist name to be divided at all, or there should be another way to have it correctly interpreted (quotation marks, backslash escaping, an app setting, etc.).

Screenshots

No response

Additional Context

No response

Device

No response

OS

Android 14

Version

2024.12.115

Contribution Guidelines

@Zany-Jokengineer Zany-Jokengineer added the type: bug Something isn't working label Dec 23, 2024
@zyrouge
Copy link
Owner

zyrouge commented Dec 30, 2024

Could you check if / is added as an artist tag separator in settings?

@zyrouge zyrouge added the reason: needs more info Requires more information label Dec 30, 2024
@dietmarczekay
Copy link

I have the same issue, AC/DC is listed two times. as AC and DC. The "/" is listed in artist tag beside ";" and "," and "+"

@zyrouge
Copy link
Owner

zyrouge commented Dec 30, 2024

There is no way to escape them, currently. Maybe, something like \ prefix should escape them. Meanwhile, you could just remove / from tag separators.

@Zany-Jokengineer
Copy link
Author

I removed / from separators in the settings and it solves the problem.
Feel free to close the issue.

@zyrouge zyrouge removed the reason: needs more info Requires more information label Jan 15, 2025
@zyrouge zyrouge added this to the v2025.01.x milestone Jan 15, 2025
@zyrouge zyrouge added the state: done Has been fixed or implemented label Jan 15, 2025
@zyrouge zyrouge closed this as completed Jan 15, 2025
lazykern added a commit to lazykern/Symphony that referenced this issue Jan 19, 2025
refactor: escapeable tag separators (zyrouge#723)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state: done Has been fixed or implemented type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants