Merge pull request #1429 from brigade/association-inheritance

bf4 · bf4 · commit 1d8226098f61 · 2016-01-12T17:25:41.000-06:00
Remove unnecessary `dup` in `ActiveModel::Serializer::Associations#associate`
diff --git a/lib/active_model/serializer/associations.rb b/lib/active_model/serializer/associations.rb
@@ -76,8 +76,6 @@ def has_one(name, options = {}, &block)
         # @api private
         #
         def associate(reflection)
-          self._reflections = _reflections.dup
-
           self._reflections << reflection
         end
       end

Original file line number	Diff line number	Diff line change
`@@ -76,8 +76,6 @@ def has_one(name, options = {}, &block)`
`76`	`76`	`# @api private`
`77`	`77`	`#`
`78`	`78`	`def associate(reflection)`
`79`		`- self._reflections = _reflections.dup`
`80`		`-`
`81`	`79`	`self._reflections << reflection`
`82`	`80`	`end`
`83`	`81`	`end`