Revert "Use Redcarpet::Render::XHTML instead of our custom renderer"
authorTom Hughes <tom@compton.nu>
Thu, 29 Nov 2012 22:22:22 +0000 (22:22 +0000)
committerTom Hughes <tom@compton.nu>
Thu, 29 Nov 2012 22:22:22 +0000 (22:22 +0000)
This reverts commit 4bb95b37735fac25467af21418cc0a973497219c.

Gemfile
Gemfile.lock
lib/rich_text.rb

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