crumble icon indicating copy to clipboard operation
crumble copied to clipboard

Rails 2.3.14 and ruby 1.9.2 will produce invalid URL

Open next2you opened this issue 13 years ago • 1 comments

...:action => something}) would result in an invalid URL

This is a quick fix that "works for me". The problem was that:

{ :action => "" } will result in appending {} to the URL, e.g

/home/%7B%7D with ruby 1.9.2

in 1.8.7 it is correctly

/home/

The fix will ignore empty values for any keys passed in through the params hash.

next2you avatar Feb 21 '12 11:02 next2you

Thanks for the PR, happy to merge it in, but would you mind adding a test for this if possible at all?

roidrage avatar Mar 12 '12 13:03 roidrage