Merge pull request #26381 from kamipo/extract_duplicated_create_for_association

Extract duplicated `create` and `create!` definition for association

activerecord/lib/active_record/associations/association.rb

@@ -173,6 +173,14 @@ def initialize_attributes(record, except_from_scope_attributes = nil) #:nodoc:
         set_inverse_instance(record)
       end
 
+      def create(attributes = {}, &block)
+        _create_record(attributes, &block)
+      end
+
+      def create!(attributes = {}, &block)
+        _create_record(attributes, true, &block)
+      end
+
       private
 
         def find_target?

activerecord/lib/active_record/associations/collection_association.rb

@@ -113,14 +113,6 @@ def build(attributes = {}, &block)
         end
       end
 
-      def create(attributes = {}, &block)
-        _create_record(attributes, &block)
-      end
-
-      def create!(attributes = {}, &block)
-        _create_record(attributes, true, &block)
-      end
-
       # Add +records+ to this association. Returns +self+ so method calls may
       # be chained. Since << flattens its argument list and inserts each record,
       # +push+ and +concat+ behave identically.

activerecord/lib/active_record/associations/singular_association.rb

@@ -23,14 +23,6 @@ def writer(record)
         replace(record)
       end
 
-      def create(attributes = {}, &block)
-        _create_record(attributes, &block)
-      end
-
-      def create!(attributes = {}, &block)
-        _create_record(attributes, true, &block)
-      end
-
       def build(attributes = {})
         record = build_record(attributes)
         yield(record) if block_given?