Merge pull request sparklemotion#314 from pacop/master

Fixed Issue 313
quangphamfork · May 17, 2013 · bc2203c · bc2203c
2 parents 14d251b + de72198
commit bc2203c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/mechanize/page/image.rb b/lib/mechanize/page/image.rb
@@ -162,10 +162,10 @@ def url
       if page.bases[0] then
         page.bases[0].href + src
       else
-        page.uri + src
+        page.uri + Mechanize::Util.uri_escape(src)
       end
     else
-      URI src
+      URI Mechanize::Util.uri_escape(src)
     end
   end
 

diff --git a/test/test_mechanize_page_image.rb b/test/test_mechanize_page_image.rb
@@ -54,6 +54,7 @@ def test_caption
   def test_url
     assert_equal ".jpg", img('src' => @src).extname
     assert_equal "http://example/a.jpg", img('src' => @src).url.to_s
+    assert_equal "http://example/a%20.jpg", img('src' => 'http://example/a .jpg' ).url.to_s
   end
 
   def test_url_base