Timestamps are being escaped when used with Paperclip 4.2.1

[maxim]

The part right here:
https://github.com/jrgifford/delayed_paperclip/blob/master/lib/delayed_paperclip/url_generator.rb#L14-L18

Was fixed in Paperclip here: thoughtbot/paperclip@ecfb5e2

But it gets monkeypatched by delayed_paperclip.

[ScotterC]

Thanks @maxim! Any chance you could PR that over?

[maxim]

@ScotterC Perhaps we should get rid of alias_method_chain, instead use inheritance to write our own UrlGenerator based on paperclip's. Maybe I'll try that as a less coupled way of integrating with paperclip, let's see how that'd look.

[maxim]

@ScotterC oh just noticed it's a module, nevermind, will look more into it

[maxim]

I added an unfinished pull request, explanation there #127

[ScotterC]

👍 Thanks @maxim

[consti]

Thanks for looking into that @maxim - I'm running into the same issue and hope there'll be a working fix, soonish. Let me know if you need any help!

[ScotterC]

We'll let you know @consti! @maxim and I spent half of this past Sunday untangling the UrlGenerator. Hoping to have a revamped, cleaner piece of code soon

[roymckenzie]

@maxim and @ScotterC, is this open issue related to why I am seeing:

imagename.jpg%3F1421493843 instead of imagename.jpg?1421493843 and my images are broken?

If it is not related I can open up a new issue.

Thank you for actively maintaining this gem! I'm looking forward to using it!

[consti]

imagename.jpg%3F1421493843 instead of imagename.jpg?1421493843 and my
images are broken?

Exactly, that's the issue! :)

[maxim]

Until we push the fix it's best to lock into paperclip no greater than 4.2.0.

[roymckenzie]

Thanks @maxim!