I have a callback url string params[:callback]
and I need to append a query string \"&result=true\"
and redirect the user. The better way I found o
I wan't to bring update to this topic, because any of the solutions didn't work me.
The reason being, that it seems that callback.query_values
returns Nil
if the actual url doesn't have existing query values.
Therefore if you have urls such as: http://www.foo.com
and http://www.foo.com?bar=1
you should use the following code:
url = "http://www.foo.com" # or params[:callback] obviously. :)
callback = Addressable::URI.parse(url)
callback.query_values = (callback.query_values || {}).merge({
result: true
})
redirect_to callback.to_s
Cheers.
You can try with merge
request.parameters.merge({:result => true})
this will add your parameter to the ones already defined.
years later, I find a better solution of this problem.
Get the value from the super
first, then do any tweaks we need using Addressable
def url_for(options={})
val = super
if params[:locale].present?
parsed = Addressable::URI.parse(val)
query_array = parsed.query_values(Array) || []
query_array << ['locale', params[:locale]]
parsed.query_values = query_array
val = parsed.to_s
end
val
end
I think you're pretty close to optimal. you could crush out a line or two, but it doesn't really gain you anything.
callback = Addressable::URI.parse(params[:callback])
callback.query_values = callback.query_values.merge {:results => 'true' }
redirect_to callback.to_s
If the callback is always inside your application, you might have some other options with varying degrees of coolness, but you didn't specify if that was the case or not.
url = params[:callback]
redirect_to url + (url.include?('?') ? '&' : '?') + 'result=true'
callback.query_values = callback.query_values.merge({"result" => "true"})