diff options
author | eileencodes <eileencodes@gmail.com> | 2015-09-02 06:35:44 -0400 |
---|---|---|
committer | eileencodes <eileencodes@gmail.com> | 2015-09-02 09:18:46 -0400 |
commit | ec14aad419381b502f510a9b3360f0e211e41066 (patch) | |
tree | 2e2a88802d6890185a872ff7fa35ec82778b4006 /activesupport/lib/active_support/notifications.rb | |
parent | 1829943943f9f52e02ca15430490a53d1a1ec0b5 (diff) | |
download | rails-ec14aad419381b502f510a9b3360f0e211e41066.tar.gz rails-ec14aad419381b502f510a9b3360f0e211e41066.tar.bz2 rails-ec14aad419381b502f510a9b3360f0e211e41066.zip |
Fix route creation when format is a blank string
Commit bff61ba, while reducing allocations, caused a regression when an empty
format is passed to a route.
This can happen in cases where you're using an anchor tag, for example:
`https://example.com/parent/575256966.#child_1032289285`.
Because of this change `format` was getting sent in
`parameterized_parts` when previously it was not included. This resulted
in blank `format`'s being returned as `.` when if there was an extension
included it would be `.extension`. Since there was no extension this
caused incorrect URL's.
The test shows this would result in `/posts/show/1.` instead of
`/posts/show/1` which causes bad urls since the format is not present.
Diffstat (limited to 'activesupport/lib/active_support/notifications.rb')
0 files changed, 0 insertions, 0 deletions