Use Redcarpet::Render::XHTML instead of our custom renderer
authorTom Hughes <tom@compton.nu>
Thu, 29 Nov 2012 18:59:43 +0000 (18:59 +0000)
committerTom Hughes <tom@compton.nu>
Thu, 29 Nov 2012 18:59:43 +0000 (18:59 +0000)
Now that my :link_attributes patch has been merged we don't need
the custom renderer to add the rel=nofollow attributes.

Gemfile
Gemfile.lock
lib/rich_text.rb

diff --git a/Gemfile b/Gemfile
index 7656092f1407d4dff7c98aec9ee2a3d58a1a0f55..143029b1391bc0ec7b195daaee7579a0f92894b4 100644 (file)
--- a/Gemfile
+++ b/Gemfile
@@ -34,7 +34,7 @@ gem 'rack-cors'
 gem 'ruby-openid', '>= 2.2.0'
 
 # Markdown formatting support
-gem 'redcarpet'
+gem 'redcarpet', '>= 2.2.0'
 
 # Character conversion support for ruby 1.8
 gem 'iconv', :platforms => :ruby_18
index 5d4372f2a1670950a8ca3f1ec867d854a8767a53..ea355c4b39cc56d51eb2db0d6008af035ce74df4 100644 (file)
@@ -189,7 +189,7 @@ DEPENDENCIES
   rack-cors
   rails (= 3.2.9)
   rails-i18n (>= 0.6.3)
-  redcarpet
+  redcarpet (>= 2.2.0)
   rinku (>= 1.2.2)
   ruby-openid (>= 2.2.0)
   sanitize
index 87ea58933579e93c043d1ed28adf5f9ac7a0f094..7d8010af1785ea6c9b275baf3fb572f997192ad2 100644 (file)
@@ -82,27 +82,13 @@ module RichText
   private
 
     def html_parser
-      @@html_renderer ||= Renderer.new({
-        :filter_html => true, :safe_links_only => true
+      @@html_renderer ||= Redcarpet::Render::XHTML.new({
+        :filter_html => true, :safe_links_only => true, :link_attributes => { :rel => "nofollow" }
       })
       @@html_parser ||= Redcarpet::Markdown.new(@@html_renderer, {
         :no_intra_emphasis => true, :autolink => true, :space_after_headers => true
       })
     end
-
-    class Renderer < Redcarpet::Render::XHTML
-      def link(link, title, alt_text)
-        "<a rel=\"nofollow\" href=\"#{link}\">#{alt_text}</a>"
-      end
-
-      def autolink(link, link_type)
-        if link_type == :email
-          "<a rel=\"nofollow\" href=\"mailto:#{link}\">#{link}</a>"
-        else
-          "<a rel=\"nofollow\" href=\"#{link}\">#{link}</a>"
-        end
-      end 
-    end
   end
 
   class Text < Base