diff options
| author | Guo Xiang Tan <tgx_world@hotmail.com> | 2014-11-22 21:42:19 +0800 |
|---|---|---|
| committer | Guo Xiang Tan <tgx_world@hotmail.com> | 2014-11-23 08:23:26 +0800 |
| commit | 46041c520809714b3937e7c79c2f220018a4a111 (patch) | |
| tree | 9a8d7b424765ac519f6b75e5f0fb2fe4c860ff10 /actionpack/lib/action_dispatch | |
| parent | 555f95bd0621b8d2e666f6989927eee7d4b7f42d (diff) | |
| download | rails-46041c520809714b3937e7c79c2f220018a4a111.tar.gz rails-46041c520809714b3937e7c79c2f220018a4a111.tar.bz2 rails-46041c520809714b3937e7c79c2f220018a4a111.zip | |
Anchor should not be appended when set to nil/false.
Fixes https://github.com/rails/rails/issues/17714.
Diffstat (limited to 'actionpack/lib/action_dispatch')
| -rw-r--r-- | actionpack/lib/action_dispatch/http/url.rb | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/actionpack/lib/action_dispatch/http/url.rb b/actionpack/lib/action_dispatch/http/url.rb index 6b8dcaf497..22c0de2ac2 100644 --- a/actionpack/lib/action_dispatch/http/url.rb +++ b/actionpack/lib/action_dispatch/http/url.rb @@ -68,7 +68,9 @@ module ActionDispatch end def add_anchor(path, anchor) - path << "##{Journey::Router::Utils.escape_fragment(anchor.to_param.to_s)}" + if anchor + path << "##{Journey::Router::Utils.escape_fragment(anchor.to_param)}" + end end def extract_domain_from(host, tld_length) |