Merge pull request #1187 from thedarkone/polymorphic-class-fix

Handle polymorphic_type NOT NULL columns
This commit is contained in:
Santiago Pastorino
2011-05-21 09:43:42 -07:00
2 changed files with 12 additions and 1 deletions

View File

@@ -19,7 +19,7 @@ module ActiveRecord
def klass
type = owner[reflection.foreign_type]
type && type.constantize
type.presence && type.constantize
end
def raise_on_type_mismatch(record)

View File

@@ -158,6 +158,17 @@ class BelongsToAssociationsTest < ActiveRecord::TestCase
assert_not_nil Company.find(3).firm_with_condition, "Microsoft should have a firm"
end
def test_polymorphic_association_class
sponsor = Sponsor.new
assert_nil sponsor.association(:sponsorable).send(:klass)
sponsor.sponsorable_type = '' # the column doesn't have to be declared NOT NULL
assert_nil sponsor.association(:sponsorable).send(:klass)
sponsor.sponsorable = Member.new :name => "Bert"
assert_equal Member, sponsor.association(:sponsorable).send(:klass)
end
def test_with_polymorphic_and_condition
sponsor = Sponsor.create
member = Member.create :name => "Bert"