Permalink
Browse files

Using URI.parser.unescape to avoid warnings

  • Loading branch information...
1 parent 2428580 commit 64ce0918a6391db785fd4d95f121312bd0eb77df @arunagw arunagw committed Aug 2, 2013
Showing with 2 additions and 2 deletions.
  1. +2 −2 actionpack/lib/action_view/helpers/url_helper.rb
@@ -528,13 +528,13 @@ def current_page?(options)
return false unless request.get? || request.head?
- url_string = URI.unescape(url_for(options)).force_encoding(Encoding::BINARY)
+ url_string = URI.parser.unescape(url_for(options)).force_encoding(Encoding::BINARY)
# We ignore any extra parameters in the request_uri if the
# submitted url doesn't have any either. This lets the function
# work with things like ?order=asc
request_uri = url_string.index("?") ? request.fullpath : request.path
- request_uri = URI.unescape(request_uri).force_encoding(Encoding::BINARY)
+ request_uri = URI.parser.unescape(request_uri).force_encoding(Encoding::BINARY)
if url_string =~ /^\w+:\/\//
url_string == "#{request.protocol}#{request.host_with_port}#{request_uri}"

0 comments on commit 64ce091

Please sign in to comment.