html_to_plain_text 1.0.1 → 1.0.2

data/VERSION

@@ -1 +1 @@
-1.0.1
+1.0.2

data/lib/html_to_plain_text.rb

@@ -16,9 +16,10 @@ module HtmlToPlainText
   OL = "ol".freeze
   UL = "ul".freeze
   LI = "li".freeze
-  NUMBERS = ["1", "a"]
-  ABSOLUTE_URL_PATTERN = /^[a-z]+:\/\/[a-z0-9]/i
-  HTML_PATTERN = /[<&]/
+  NUMBERS = ["1", "a"].freeze
+  ABSOLUTE_URL_PATTERN = /^[a-z]+:\/\/[a-z0-9]/i.freeze
+  HTML_PATTERN = /[<&]/.freeze
+  TRAILING_WHITESPACE = /[ \t]+$/.freeze
   
   # Helper instance method for converting HTML into plain text. This method simply calls HtmlToPlainText.plain_text.
   def plain_text(html)
@@ -63,8 +64,10 @@ module HtmlToPlainText
           convert_node_to_plain_text(node, out, child_options(node, options))
           
           if node.name == BR
+            out.sub!(TRAILING_WHITESPACE, '')
             out << "\n"
           elsif node.name == HR
+            out.sub!(TRAILING_WHITESPACE, '')
             out << "\n" unless out.end_with?("\n")
             out << "-------------------------------\n"
           elsif node.name == TD || node.name == TH
@@ -104,7 +107,7 @@ module HtmlToPlainText
     # Add double line breaks between paragraph elements. If line breaks already exist,
     # new ones will only be added to get to two.
     def append_paragraph_breaks(out)
-      out.chomp!(" ")
+      out.sub!(TRAILING_WHITESPACE, '')
       if out.end_with?("\n")
         out << "\n" unless out.end_with?("\n\n")
       else
@@ -115,7 +118,7 @@ module HtmlToPlainText
     # Add a single line break between block elements. If a line break already exists,
     # none will be added.
     def append_block_breaks(out)
-      out.chomp!(" ")
+      out.sub!(TRAILING_WHITESPACE, '')
       out << "\n" unless out.end_with?("\n")
     end
     

data/spec/html_to_plain_text_spec.rb

@@ -2,22 +2,22 @@ require 'spec_helper'
 
 describe HtmlToPlainText do
   it "should format paragraph tags" do
-    html = "<h1>Test</h1><h2>More Test</h2><p>This is a test</p>"
+    html = "<h1>Test</h1><h2>More Test</h2>\t \t<p>\n\tThis is a test\n</p>"
     HtmlToPlainText.plain_text(html).should == "Test\n\nMore Test\n\nThis is a test"
   end
   
   it "should format block tags" do
-    html = "<div>Test</div><div>More Test<div>This is a test</div></div>"
+    html = "<div>Test</div><div>More Test<div>\t This is a test\t </div></div>"
     HtmlToPlainText.plain_text(html).should == "Test\nMore Test\nThis is a test"
   end
   
   it "should format <br> tags" do
-    html = "<div>Test</div><br><div>More Test<br />This is a test"
+    html = "<div>Test</div><br><div>More Test \t <br />This is a test"
     HtmlToPlainText.plain_text(html).should == "Test\n\nMore Test\nThis is a test"
   end
   
   it "should format <hr> tags" do
-    html = "<div>Test</div><hr><div>More Test<hr />This is a test"
+    html = "<div>Test</div><hr><div>More Test \t <hr />This is a test"
     HtmlToPlainText.plain_text(html).should == "Test\n-------------------------------\nMore Test\n-------------------------------\nThis is a test"
   end
   

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: html_to_plain_text
 version: !ruby/object:Gem::Version 
-  hash: 21
+  hash: 19
   prerelease: 
   segments: 
   - 1
   - 0
-  - 1
-  version: 1.0.1
+  - 2
+  version: 1.0.2
 platform: ruby
 authors: 
 - Brian Durand
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-06-21 00:00:00 -05:00
+date: 2011-08-05 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency