don't strip valueless query strings
parent
dc32d286bd
commit
9b7e4018b0
|
@ -3,6 +3,7 @@ module UrlHelper
|
||||||
return if url.blank?
|
return if url.blank?
|
||||||
url = Addressable::URI.parse(url)
|
url = Addressable::URI.parse(url)
|
||||||
return url.to_s if url.query.blank?
|
return url.to_s if url.query.blank?
|
||||||
|
return unless '='.in?(url.query)
|
||||||
params = CGI.parse(url.query)
|
params = CGI.parse(url.query)
|
||||||
params.delete_if do |key|
|
params.delete_if do |key|
|
||||||
k = key.downcase
|
k = key.downcase
|
||||||
|
@ -29,7 +30,7 @@ module UrlHelper
|
||||||
].include?(k)
|
].include?(k)
|
||||||
false
|
false
|
||||||
end
|
end
|
||||||
url.query = URI.encode_www_form(params)
|
url.query = params.to_query
|
||||||
return url.to_s
|
return url.to_s
|
||||||
rescue Addressable::URI::InvalidURIError, IDN::Idna::IdnaError
|
rescue Addressable::URI::InvalidURIError, IDN::Idna::IdnaError
|
||||||
return '#'
|
return '#'
|
||||||
|
|
Loading…
Reference in New Issue