diff options
| author | Phil Darnowsky <pdarnows@yahoo.com> | 2009-10-07 14:49:38 -0400 |
|---|---|---|
| committer | Michael Koziarski <michael@koziarski.com> | 2009-10-15 09:49:07 +1300 |
| commit | 1b3195b63ca44f0a70b61b75fcf4991cb2fbb944 (patch) | |
| tree | 1afde88f4b690d4d1a58a27068c67cb45f35536d /actionpack/test/template/test_case_test.rb | |
| parent | a41c6c35cadf75bfd4bf0a17113ae37d628896e8 (diff) | |
| download | rails-1b3195b63ca44f0a70b61b75fcf4991cb2fbb944.tar.gz rails-1b3195b63ca44f0a70b61b75fcf4991cb2fbb944.tar.bz2 rails-1b3195b63ca44f0a70b61b75fcf4991cb2fbb944.zip | |
ActionView.url_for doesn't escape by default
ActionView::Helpers::UrlHelper#url_for used to escape the URLs it generated by
default. This was most commonly seen when generating a path with multiple
query parameters, e.g.
url_for(:controller => :foo, :action => :bar, :this => 123, :that => 456)
would return
http://example.com/foo/bar?that=456&this=123
escaping an ampersand that shouldn't be escaped. This is both wrong and
inconsistent with the behavior of ActionController#url_for, and is changed.
Signed-off-by: Michael Koziarski <michael@koziarski.com>
Diffstat (limited to 'actionpack/test/template/test_case_test.rb')
0 files changed, 0 insertions, 0 deletions