Merge pull request #5977 from oscardelben/refactor_serializable_hash

Minor refactor of serializable hash
rails · Apr 25, 2012 · 217e9c0 · 217e9c0
2 parents 2aabf8c + a631dd9
commit 217e9c0
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/activemodel/lib/active_model/serialization.rb b/activemodel/lib/active_model/serialization.rb
@@ -78,8 +78,7 @@ def serializable_hash(options = nil)
       hash = {}
       attribute_names.each { |n| hash[n] = read_attribute_for_serialization(n) }
 
-      method_names = Array(options[:methods]).select { |n| respond_to?(n) }
+      Array(options[:methods]).each { |m| hash[m.to_s] = send(m) if respond_to?(m) }
-      method_names.each { |n| hash[n.to_s] = send(n) }
 
       serializable_add_includes(options) do |association, records, opts|
         hash[association.to_s] = if records.is_a?(Enumerable)