-
Notifications
You must be signed in to change notification settings - Fork 117
Added --minimal-slugify argument that only removes "\ | / : ? * " < >" #277
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
base: master
Are you sure you want to change the base?
Added --minimal-slugify argument that only removes "\ | / : ? * " < >" #277
Conversation
3b2f617 to
acba3b8
Compare
|
What is the use-case? An example would be helpful. |
|
Sorry, I had problems with git and forgot to add a description. This is useful for people not using any id in template and trying to download albums with characters that would be removed by slugify, such as https://deathbedtapes.bandcamp.com/album/-, https://sonnovdrone.bandcamp.com/album/--162 and https://woodstruth.bandcamp.com/album/-, since slugify removes the characters it'd fail to create a folder and just download the tracks to base-dir. This would only remove illegal characters for filenames and folders (at least for windows, I'm not sure about other OS), so the paths would be as close as possible to the original titles and avoid folders failing to be created. |
|
Would not |
|
Doing Doing |
|
Ah I see, I assumed the album name was the problem here, in that case: My issue is that slugification does a bit more than filtering chars out and this is essentially doing |
|
I thought of this mainly for batch downloading, where it's not reasonable to whitelist everything, and using
But doing But if it's going to be changed soon then maybe it's not needed. |
|
Depends on how the new implementation turns out really, for example remember #267 which lead to #269 ? python-slugify has max length and word boundary support built in so that may get refactored but I'm not sure yet how simply putting together presets with it will be, I may still need to leave the option open to go scorched earth, skip slugification, and use this method. |
No description provided.