This project is archived and is in readonly mode.

Changeset [df0a7bfb8fa3f0255fe4ab082863f27c43c278d9] by Andrew White

August 22nd, 2010 @ 08:18 PM

Cache the symbolized path parameters using a instance variable in the request object rather than the environment hash. This it to prevent stale parameters in later routing constraints/redirects as only the normal path parameters are set by Rack::Mount.

Also if a constraint proc arity is more than one, pass the symbolized path parameters
as the first argument to match redirect proc args and provide easier access.

[#5157 state:resolved]

Signed-off-by: José Valim jose.valim@gmail.com
http://github.com/rails/rails/commit/df0a7bfb8fa3f0255fe4ab082863f2...

Committed by Andrew White

  • M actionpack/lib/action_dispatch/http/parameters.rb
  • M actionpack/lib/action_dispatch/routing/mapper.rb
  • M actionpack/test/dispatch/routing_test.rb

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>