Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Missing artist image when artist name has illegal character #913

Closed
samkottler opened this issue Nov 5, 2021 · 2 comments
Closed

Missing artist image when artist name has illegal character #913

samkottler opened this issue Nov 5, 2021 · 2 comments
Labels
Bug

Comments

@samkottler
Copy link

In particular, AC/DC has the slash in it which means I cannot save the artist image as AC/DC.* Is there a way to either manually select the image to use or an alternative character that the scanner will recognize?
@paulijar
Copy link
Collaborator

paulijar commented Nov 5, 2021

Good point. There's currently no way around this, but I'll try to figure out something for the next version.

@paulijar paulijar added the Bug label Nov 5, 2021
paulijar added a commit that referenced this issue Nov 21, 2021
@paulijar
Allow replacing forbidden characters on artist image name with '_'
6f51963 
It was previously impossible to set artist image for an artist having
slash '/' in its name. Also characters <>:"\|?* could be problematic
because these are forbidden in file names on Windows. Any of these
characters may now be replaced with an underscore '_' in the image file
name which is matched to the artist.

refs #913
@paulijar
Copy link
Collaborator

In Music v1.5.0, you can now supply image called "AC_DC.*" to overcome the problem. Also all the characters <>:"\|?* may now be replaced with underscores _ in the image filename (but you don't have to, as the slash / is the only actually forbidden character on Linux file names).

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@samkottler @paulijar