From 589de27bcd3e11b093bc9ce8fe9ee3c8f149a766 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jos=C3=A9=20Valim?= Date: Wed, 22 Sep 2010 23:28:52 +0200 Subject: [PATCH] Stop using inject({}) to build hashes. It is harder to read and slower. --- lib/devise/strategies/authenticatable.rb | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/lib/devise/strategies/authenticatable.rb b/lib/devise/strategies/authenticatable.rb index 23c70664..700442d6 100644 --- a/lib/devise/strategies/authenticatable.rb +++ b/lib/devise/strategies/authenticatable.rb @@ -122,10 +122,8 @@ module Devise # Returns values from the request object. def request_values keys = request_keys.respond_to?(:keys) ? request_keys.keys : request_keys - keys.inject({}) do |hash, key| - hash[key] = self.request.send(key) - hash - end + values = keys.map { |k| self.request.send(k) } + Hash[keys.zip(values)] end # Parse authentication keys considering if they should be enforced or not.