Merge pull request #65 from pengwynn/autolink-skip-tags

Allow passing skip_tags in autolink filter context

Author: jch

lib/html/pipeline/autolink_filter.rb

@@ -5,18 +5,22 @@ class Pipeline
     # HTML Filter for auto_linking urls in HTML.
     #
     # Context options:
-    #   :autolink - boolean whether to autolink urls
-    #   :flags    - additional Rinku flags. See https://github.com/vmg/rinku
+    #   :autolink  - boolean whether to autolink urls
+    #   :skip_tags - HTML tags inside which autolinking will be skipped.
+    #                See Rinku.skip_tags
+    #   :flags     - additional Rinku flags. See https://github.com/vmg/rinku
     #
     # This filter does not write additional information to the context.
     class AutolinkFilter < Filter
       def call
         return html if context[:autolink] == false
+
+        skip_tags = context[:skip_tags]
         flags = 0
         flags |= context[:flags] if context[:flags]
 
-        Rinku.auto_link(html, :urls, nil, %w[a script kbd pre code], flags)
+        Rinku.auto_link(html, :urls, nil, skip_tags, flags)
       end
     end
   end
-end
+end

test/html/pipeline/autolink_filter_test.rb

@@ -19,4 +19,12 @@ def test_autolink_flags
     assert_equal '<p>"<a href="http://github">http://github</a>"</p>',
       AutolinkFilter.to_html('<p>"http://github"</p>', :flags => Rinku::AUTOLINK_SHORT_DOMAINS)
   end
+
+  def test_autolink_skip_tags
+    assert_equal '<code>"http://github.com"</code>',
+      AutolinkFilter.to_html('<code>"http://github.com"</code>')
+
+    assert_equal '<code>"<a href="http://github.com">http://github.com</a>"</code>',
+      AutolinkFilter.to_html('<code>"http://github.com"</code>', :skip_tags => %w(kbd script))
+  end
 end