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
sanitize playlist names #2258
sanitize playlist names #2258
Conversation
Make e. g. - name: $albumartist/$year-$album.m3u' work when album data includes non fs-friendly characters
This is a good idea; thanks! Can you please look into the test failures? Perhaps some tests need to be updated to stay in sync. Then, would you mind adding a quick changelog entry? |
@@ -186,7 +187,7 @@ def update_playlists(self, lib): | |||
# As we allow tags in the m3u names, we'll need to iterate through | |||
# the items and generate the correct m3u file names. | |||
for item in items: | |||
m3u_name = item.evaluate_template(name, True) | |||
m3u_name = item.evaluate_template(sanitized_name, True) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm; are you sure you want to be sanitizing the template instead of the result of filling in the template?
Thank you! ✨ I've merged this. |
Make e. g.
work when album data includes non fs-friendly characters