Make Paperclip more metaclass friendly #979

Closed
wants to merge 4 commits into
from

Projects

None yet

5 participants

@alanho

I used paperclip in a metaclass, but the self.class does not work on metaclasses. The instance method of the same name (which behave exactly the same) works.

@travisbot

This pull request passes (merged 91aecd8 into 70f0f1f).

@sikachu
thoughtbot, inc. member

Would you mind adding test for this? Say, let's create a meta class then add an attachment for it. I want to make sure that I won't introduce a regression in the future.

@jyurek
thoughtbot, inc. member

Yes, I'd like to see a test case for this, if for nothing else than to see an example, as I'm not sure when this would come in handy.

@alanho

sorry it took me a while to learn how to write a test case (shame on me). i've added in a test case for meta class

@travisbot

This pull request passes (merged 10ef3cd into 4bea897).

@mike-burns
thoughtbot, inc. member

Merged, thanks!

@mike-burns mike-burns closed this Sep 7, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment