marekrei / smuploader

Upload / download library for SmugMug.com

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Titlecasing sometimes not helpful.

operations-smco opened this issue · comments

I found

return "-".join([re.sub(r'[\W_]+', '', x) for x in name.strip().split()]).title()

was a bit problematic in that I didn't want titlecasing, so dropped .title() from the code.

Would be good to have a user driven option to turn this on or off.

True. However, because the "nice name" has many restrictions, the original string needs to be modified anyway, and it's subjective what are the exact best modifications.

I'll leave this comment open as a suggestion for future work.