Turn url_params keys into strings

gcapizzi · Aug 10, 2013 · 2b9bfc6 · 2b9bfc6
1 parent 4dc4f2e
commit 2b9bfc6
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 5 deletions.
diff --git a/app/actions/accounts_actions.rb b/app/actions/accounts_actions.rb
@@ -20,7 +20,7 @@ def call(env)
 
     class ShowAccount < AccountsAction
       def call(env)
-        id = req(env).url_params[:account_id].to_i
+        id = req(env).url_params['account_id'].to_i
         account = @repository.get(id) or return not_found
         body = @renderer.render(account)
         ok(body)
@@ -31,7 +31,7 @@ class UpdateAccount < AccountsAction
       def call(env)
         req = req(env)
 
-        id = req.url_params[:account_id].to_i
+        id = req.url_params['account_id'].to_i
         account = @repository.get(id) or return not_found
 
         account.set(req.params)
@@ -59,7 +59,7 @@ def call(env)
 
     class DeleteAccount < AccountsAction
       def call(env)
-        id = req(env).url_params[:account_id].to_i
+        id = req(env).url_params['account_id'].to_i
         account = @repository.get(id) or return not_found
 
         if @repository.destroy(account)

diff --git a/app/actions/transactions_actions.rb b/app/actions/transactions_actions.rb
@@ -12,7 +12,7 @@ def initialize(accounts_repository, renderer)
       end
 
       def call(env)
-        account_id = req(env).url_params[:account_id].to_i
+        account_id = req(env).url_params['account_id'].to_i
         account = @accounts_repository.get(account_id)
         body = @renderer.render(account.transactions)
         ok(body)

diff --git a/app/utils.rb b/app/utils.rb
@@ -4,7 +4,13 @@ module Scrooge
 
   class Request < Rack::Request
     def url_params
-      env['rack.routing_args']
+      stringify_keys(env['rack.routing_args'])
+    end
+
+    private
+
+    def stringify_keys(hash)
+      Hash[hash.to_a.map { |key, val| [key.to_s, val] }]
     end
   end