Fix link filter for urls like:

http://www.google.com/#q=nikola+tesla&ct=tesla09&oi=ddle&fp=Xmf0jJ9P_V0
fabrikagency · Jul 10, 2009 · 39ae892 · 39ae892
1 parent fd3cac2
commit 39ae892
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/lib/auto_html/filters/link.rb b/lib/auto_html/filters/link.rb
@@ -28,7 +28,7 @@ def auto_link_re
         (?:\.[-\w]+)*            # remaining subdomains or domain
         (?::\d+)?                # port
         (?:/(?:[~\w\+@%=\(\)-]|(?:[,.;:'][^\s$]))*)* # path
-        (?:\?[\w\+@%&=.;:-]+)?     # query string
+        (?:\?|\#[\w\+@%&=.;:-]+)?     # query string
         (?:\#[\w\-]*)?           # trailing anchor
       )
       ([[:punct:]]|<|$|)       # trailing text

diff --git a/test/unit/filters/link_test.rb b/test/unit/filters/link_test.rb
@@ -6,7 +6,12 @@ def test_transform
     result = auto_html("http://vukajlija.com") { link }
     assert_equal '<a href="http://vukajlija.com">http://vukajlija.com</a>', result
   end
-
+
+  def test_transform_complex_url
+    result = auto_html("http://www.google.com/#q=nikola+tesla&ct=tesla09&oi=ddle&fp=Xmf0jJ9P_V0") { link }
+    assert_equal '<a href="http://www.google.com/#q=nikola+tesla&ct=tesla09&oi=ddle&fp=Xmf0jJ9P_V0">http://www.google.com/#q=nikola+tesla&ct=tesla09&oi=ddle&fp=Xmf0jJ9P_V0</a>', result
+  end
+
   def test_transform_with_options
     result = auto_html("http://rors.org") { link :rel => "nofollow", :target => "_blank" }
     assert_equal '<a href="http://rors.org" rel="nofollow" target="_blank">http://rors.org</a>', result