This project is archived and is in readonly mode.

Changeset [befa77fc18ba54c1be89553466312039c1073f02] by Piotr Sarnacki

September 3rd, 2010 @ 09:59 PM

Fix generating urls with mounted helpers in view context

There were actually 2 problems with this one:
script_name was added to options as a string and then it was used in RouteSet#url_for with usage of <<, which was changing the original script_name the second issue was with with_routes method. It was called in RoutesProxy to modify routes in view_context, but url_helpers in views is just delegating it to controller, so another _with_routes call is needed there http://github.com/rails/rails/commit/befa77fc18ba54c1be895534663120...

Committed by Piotr Sarnacki

  • M actionpack/lib/action_controller/metal/url_for.rb
  • M actionpack/lib/action_dispatch/routing/route_set.rb
  • M actionpack/lib/action_view/helpers/url_helper.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>