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

Allow setting of escape url as a default option #1061

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@garethrees
Contributor

garethrees commented Oct 24, 2012

Allow developers to escape URLs by default.

Example

Paperclip::Attachment.default_options.update({
  :escape_url => false
})

Why?

See this issue on Stack Overflow.

TL;DR: Need to call .url(:style, :escape => false) for use with Rackspace CloudFiles. Don't want to have to do this for every image call.

@jyurek

This comment has been minimized.

Show comment
Hide comment
@jyurek

jyurek Nov 2, 2012

Contributor

Pulled this in! Thanks!

Contributor

jyurek commented Nov 2, 2012

Pulled this in! Thanks!

@jyurek jyurek closed this Nov 2, 2012

@garethrees

This comment has been minimized.

Show comment
Hide comment
@garethrees

garethrees Nov 2, 2012

Contributor

@jyurek many thanks! Would it be possible to get this in the 2.7 branch? Stuck on Rails 2.3. Happy to test and make a PR.

Contributor

garethrees commented Nov 2, 2012

@jyurek many thanks! Would it be possible to get this in the 2.7 branch? Stuck on Rails 2.3. Happy to test and make a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment