Getting this error on calling destroy on Tire Persistence model: NameError: undefined local variable or method `image' for about carrierwave-tire HOT 5 CLOSED

Hi princejoseph, the bug should be fixed now. Could you please update the gem and try again?

from carrierwave-tire.

@dpisarewski yes the bug is fixed. I had actually overrided the method and wrote almost same code in the fix in an initializer to get past the error at that time.

Thanks

from carrierwave-tire.

@dpisarewski sorry, I hadnt tested the fix correctly (I had updated the gem, but I hadnt removed the initializer that I had written for the fix, so it was overriding the gem code). I tested it today again, and found the code is giving another error. I have found the issue, and has found the solution.

In lib/carrierwave/tire.rb:46

46 +            FileUtils.rm_rf File.dirname(send(column)).file.path) if send(column).file.present?

it should be

FileUtils.rm_rf File.dirname(send('#{column}')).file.path) if send('#{column}').file.present?

This is what I had written in my fix in the initializer. The send parameter should send the column name as string.

This might also work (looks more consistent with the preceding code):

FileUtils.rm_rf File.dirname(send(:#{column})).file.path) if send(:#{column}).file.present?

from carrierwave-tire.

Would you like to make a pull request for the fix?

from carrierwave-tire.

New to github and first open source contribution :)

from carrierwave-tire.