Request not being optional on routes

[fedesoria]

The following will raise ArgumentError (wrong number of arguments (1 for 2))

match '/redirect/:id', :to => redirect {|params| "/hello/#{User.find(params[:id]).name}"}

this was fixed when introduced |params, request|

Found out that that 6c57177 this commit fixes the problem but hasn't been pushed to a release.

Gracias.

[rafaelfranca]

The redirect blocks with one argument is deprecated. You should use with two arguments like this:

match '/redirect/:id', :to => redirect {|params, request| "/hello/#{User.find(params[:id]).name}"}

[drogus]

The commit that you linked to actually just removed deprecation. request as optional parameter was deprecated and it's removed in master. You can still pass only one parameter on 3.2.x, but you will get deprecation warning: https://github.com/rails/rails/blob/3-2-stable/actionpack/lib/action_dispatch/routing/redirection.rb#L102

Does it clear things out or is your problem different?

[rafaelfranca]

Hey guys, I confirmed that when you pass only one argument to the redirect block one exception is raised. I'm working on it to fix and reopening this issue.