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]: ND_SCANNER_GENRESEPARATORS not working? #2981

Open
3 tasks done
SkipperLappen opened this issue Apr 28, 2024 · 4 comments
Open
3 tasks done

[Bug]: ND_SCANNER_GENRESEPARATORS not working? #2981

SkipperLappen opened this issue Apr 28, 2024 · 4 comments
Labels
bug triage New bug reports that need to be evaluated

Comments

@SkipperLappen
Copy link

I confirm that:

  • I have searched the existing open AND closed issues to see if an issue already exists for the bug I've encountered
  • I'm using the latest version (your issue may have been fixed already)

Version

0.52.0 (92a98cd)

Current Behavior

I have ND_SCANNER_GENRESEPARATORS set as ";." but my genres keep separating by "/"....i.e. "Techno (Peak / Time)" is split in two.

Expected Behavior

Not split by "/" when not specified in ND_SCANNER_GENRESEPARATORS

Steps To Reproduce

No response

Environment

- OS: Linux 6.1.79-Unraid x86_64
- Browser: Safari 17.4.1 (19618.1.15.11.14)
- Client: MacOS 14.4.1 (23E224)

How Navidrome is installed?

Binary (from downloads page)

Configuration

No response

Relevant log output

No response

Anything else?

No response

Code of Conduct

  • I agree to follow Navidrome's Code of Conduct
@SkipperLappen SkipperLappen added bug triage New bug reports that need to be evaluated labels Apr 28, 2024
@deluan
Copy link
Member

deluan commented Apr 28, 2024

  • Was this working for you in previous versions?
  • This config option is only applied at import/scan time. If you change its value, you will need to do a Full Scan for it to take effect. Have you done that?

@SkipperLappen
Copy link
Author

  • Was this working for you in previous versions?

No, never worked for me.

  • This config option is only applied at import/scan time. If you change its value, you will need to do a Full Scan for it to take effect. Have you done that?

No, but will try that and get back to you

@metalheim
Copy link
Contributor

Can you try adding a navidrome.toml configuration file and setting Scanner.GenreSeparators from there?

https://www.navidrome.org/docs/usage/configuration-options/#advanced-configuration

Im quite sure that this config option works for me as I added a bunch of Extra Genre seperators

@certuna
Copy link
Contributor

certuna commented May 5, 2024

I think this may be an upstream bug in TagLib - if I'm not mistaken, it always parses / as a genre separator in id3v2.3 and id3v2.4 even though it's against the specs.

Navidrome gets the genres as an array from TagLib, so if it splits the value upstream, there's nothing we can do, really.

I'll have to do some testing to verify this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug triage New bug reports that need to be evaluated
Projects
None yet
Development

No branches or pull requests

4 participants