Merge pull request #26865 from...

Merge pull request #26865 from sergey-alekseev/uniqueness-validation-scope-with-polymorphic-association

fix the uniqueness validation scope with a polymorphic association

activerecord/CHANGELOG.md

+*   Fix the uniqueness validation scope with a polymorphic association.
+
+    *Sergey Alekseev*
+
 *   Raise ActiveRecord::RecordNotFound from collection `*_ids` setters
     for unknown IDs with a better error message.
 

activerecord/lib/active_record/validations/uniqueness.rb

@@ -85,11 +85,10 @@ def build_relation(klass, attribute, value) # :nodoc:
 
       def scope_relation(record, relation)
         Array(options[:scope]).each do |scope_item|
-          if reflection = record.class._reflect_on_association(scope_item)
-            scope_value = record.send(reflection.foreign_key)
-            scope_item  = reflection.foreign_key
+          scope_value = if record.class._reflect_on_association(scope_item)
+            record.association(scope_item).reader
           else
-            scope_value = record._read_attribute(scope_item)
+            record._read_attribute(scope_item)
           end
           relation = relation.where(scope_item => scope_value)
         end

activerecord/test/cases/validations/uniqueness_validation_test.rb

@@ -6,6 +6,9 @@
 require "models/event"
 require "models/dashboard"
 require "models/uuid_item"
+require "models/author"
+require "models/person"
+require "models/essay"
 
 class Wizard < ActiveRecord::Base
   self.abstract_class = true
@@ -163,6 +166,19 @@ def test_validate_uniqueness_with_object_scope
     assert !r2.valid?, "Saving r2 first time"
   end
 
+  def test_validate_uniqueness_with_polymorphic_object_scope
+    Essay.validates_uniqueness_of(:name, scope: :writer)
+
+    a = Author.create(name: "Sergey")
+    p = Person.create(first_name: "Sergey")
+
+    e1 = a.essays.create(name: "Essay")
+    assert e1.valid?, "Saving e1"
+
+    e2 = p.essays.create(name: "Essay")
+    assert e2.valid?, "Saving e2"
+  end
+
   def test_validate_uniqueness_with_composed_attribute_scope
     r1 = ReplyWithTitleObject.create "title" => "r1", "content" => "hello world"
     assert r1.valid?, "Saving r1"