Replacing an each with reduce

The functionality has not changed, but the code is more elegant by
using `reduce` instead of `each`.

This way no accumulator needs to be declared, no explicit return is
needed.

1 files changed, 2 insertions, 4 deletions

diff --git a/activemodel/lib/active_model/errors.rb b/activemodel/lib/active_model/errors.rb
index fc8034f9c7..f7126c1eb8 100644
--- a/activemodel/lib/active_model/errors.rb
+++ b/activemodel/lib/active_model/errors.rb
@@ -251,11 +251,9 @@ module ActiveModel
     #   person.errors.to_hash(true) # => {:name=>["name cannot be nil"]}
     def to_hash(full_messages = false)
       if full_messages
-        messages = {}
-        self.messages.each do |attribute, array|
-          messages[attribute] = array.map { |message| full_message(attribute, message) }
+        self.messages.reduce({}) do |messages, (attribute, array)|
+          messages.merge!(attribute => array.map { |message| full_message(attribute, message) })
         end
-        messages
       else
         self.messages.dup
       end