Avoid dragging ActionView::Helpers::TextHelper into RichText objects
authorTom Hughes <tom@compton.nu>
Mon, 19 Mar 2012 11:59:54 +0000 (11:59 +0000)
committerTom Hughes <tom@compton.nu>
Mon, 19 Mar 2012 12:01:02 +0000 (12:01 +0000)
Dragging in TextHelper gets us a truncate method which hides the one
we inherited from String and which we want.

lib/rich_text.rb

index 92be890..2b7e310 100644 (file)
@@ -8,7 +8,17 @@ module RichText
     end
   end
 
+  class SimpleFormat
+    include ActionView::Helpers::TextHelper
+
+    def sanitize(text)
+      Sanitize.clean(text, Sanitize::Config::OSM).html_safe
+    end
+  end
+
   class Base < String
+    include ActionView::Helpers::TagHelper
+
     def spam_score
       link_count = 0
       link_size = 0
@@ -31,6 +41,10 @@ module RichText
 
   protected
 
+    def simple_format(text)
+      SimpleFormat.new.simple_format(text)
+    end
+
     def linkify(text)
       if text.html_safe?
         Rinku.auto_link(text, :urls, tag_options(:rel => "nofollow")).html_safe
@@ -41,15 +55,12 @@ module RichText
   end
 
   class HTML < Base
-    include ActionView::Helpers::TextHelper
-    include ActionView::Helpers::TagHelper
-
     def to_html
       linkify(sanitize(simple_format(self)))
     end
 
     def to_text
-      self
+      self.to_s
     end
 
   private
@@ -65,7 +76,7 @@ module RichText
     end
 
     def to_text
-      self
+      self.to_s
     end
 
   private
@@ -95,14 +106,12 @@ module RichText
   end
 
   class Text < Base
-    include ActionView::Helpers::TextHelper
-
     def to_html
       linkify(simple_format(ERB::Util.html_escape(self)))
     end
 
     def to_text
-      self
+      self.to_s
     end
   end
 end