paperclipftp icon indicating copy to clipboard operation
paperclipftp copied to clipboard

`@[]' is not allowed as an instance variable name

Open Kelend opened this issue 13 years ago • 6 comments

I'm getting this error:

`@[]' is not allowed as an instance variable name

when using paperclipftp with Rails 3.1

Kelend avatar Oct 13 '11 20:10 Kelend

As far as I know, paperclipftp is tested to work only with Rails up to 3.0. You are pretty much on your own with 3.1 (yet). But if you could figure out a problem and provide a patch for it, it would be just awesome :)

olek avatar Oct 13 '11 23:10 olek

Looks like the issue isn't with Rails 3.1 but a real recent update to paperclip. I'll do a pull request with what we did to fix it.

Kelend avatar Oct 14 '11 14:10 Kelend

Hey Kelend, do you know if this prob is resolved yet? I too get the same error. Many thanks!!

docgecko avatar Oct 24 '11 13:10 docgecko

I have an open pull request that fixes the issue, in the mean time you could use my fork; git://github.com/twotoasters/paperclipftp.git

Kelend avatar Oct 24 '11 14:10 Kelend

I had a look at that pull request and while I agree with the fix, it will break paperclipftp for people that use older version of paperclip. And since paperclip known to introduce non-backward compatible changes on periodical basis, I would imagine that we are not the only company reluctant to do frequent upgrades for paperclip gem itself. It would be nice if paperclipftp applied that new 'options' code conditionally.

olek avatar Oct 24 '11 14:10 olek

Hi Kelend, thank you, your fork is very useful. It solved the problem for me.

@olek it would be useful if the readme page at least pointed to Kelend's solution for those people using Rails 3.1. Also, another thing that could save me time is - when used on heroku, :ftp_passive_mode should be set to true (:ftp_passive_mode => true) Olek, I'd do the pull request for this, but I'm newb and I don't know the process :/

bruno- avatar Dec 15 '11 06:12 bruno-