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

Remove sanitize_title #164

Closed
phihag opened this issue Sep 15, 2011 · 0 comments
Closed

Remove sanitize_title #164

phihag opened this issue Sep 15, 2011 · 0 comments

Comments

@phihag
Copy link
Contributor

@phihag phihag commented Sep 15, 2011

Currently, each extractor calls sanitize_title in it's own and even makes up its own stitle. We should do both steps in process_info for all extractors.

title should be the original title (and we can assert isinstance(title, unicode)).
sanitized_title should be a title one can use in file names (i.e. without slash, and on Windows, without multiple RTL/LTR marks)
We can then make up a simplified title ascii_title for maximum compatibility.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.