-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Saving images from url with special characters #1921 #1932
Conversation
end | ||
|
||
it "returns a file name" do | ||
assert_equal "paperclip-%C3%B6%C3%A4%C3%BC%E5%AD%97%C2%B4%C2%BD%E2%99%A5%C3%98%C2%B2%C3%88.png", @subject.original_filename |
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.
Line is too long. [129/80]
One quick comment, and if you can handle the Hound complaints, I'll totally pull this in. Thanks! |
url = "https://github.com/thoughtbot/paperclip-öäü字´½♥زÈ.png" | ||
subject = Paperclip.io_adapters.for(url) | ||
filename = 'paperclip-%C3%B6%C3%A4%C3%BC%E5%AD%97%C2%B4%C2%BD%E2%99%A5'\ | ||
'%C3%98%C2%B2%C3%88.png' |
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.
Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
@jyurek I don't get this errors on my local machine. |
Why is this not merged? Having the same problems with UTF-8 characters in the URI (e.g. |
Thanks for your work! @Piioo, can you please squash your commits together and rebase on top of latest master? If CI is green, we'll merge. Thanks! ❤️ |
I've got a passing CI build, will rebase myself: https://travis-ci.org/thoughtbot/paperclip/builds/128930480. Thank you! |
#1921 Now it is possible to save images from url with special characters like http://www.foo/öäü.jpg