Added polymorphic association support #11

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants
Contributor

pfenig commented Apr 4, 2012

Thought you may be interested in this. It certainly hasn't been flushed out. Definitely a cleaner solution out there but this got me what I needed. I used Kernel.const_get instead of the fully flushed out ActiveSupport constantize as you use elsewhere. This works only for the standard polymorphic association, no special options. Any feedback appreciated.

Owner

pfenig commented on 3154e1a Apr 4, 2012

That should have said "without ./" in the message.

Contributor

ggrocco commented Jun 13, 2012

Has any plan for this issue become part of the the new version? I'm passing for the same problem...

Owner

rtomayko commented Jun 14, 2012

@ggrocco Sorry, I just haven't had a second to wrap my head around this.

Owner

rtomayko commented Sep 6, 2012

Merged in #16. Thanks!

rtomayko closed this Sep 6, 2012

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