correctly detects signatures from mobile devices using "Sent from my

...."

Author: Drew Batshaw

lib/email_reply_parser.rb

@@ -107,7 +107,6 @@ def read(text)
 
   private
     EMPTY = "".freeze
-
     ### Line-by-Line Parsing
 
     # Scans the given line of text and figures out which fragment it belongs
@@ -127,7 +126,7 @@ def scan_line(line)
       # Mark the current Fragment as a signature if the current line is empty
       # and the Fragment starts with a common signature indicator.
       if @fragment && line == EMPTY
-        if @fragment.lines.last =~ /[\-\_]\s*$/
+        if @fragment.lines.last =~ /([\-\_]\s*$)|(^\w+ #{"Sent from my".reverse}$)/
           @fragment.signature = true
           finish_fragment
         end

test/email_reply_parser_test.rb

@@ -87,6 +87,17 @@ def test_parse_out_just_top_for_outlook_reply
     assert_equal "Outlook with a reply", EmailReplyParser.parse_reply(body)
   end
 
+  def test_parse_out_sent_from_iPhone
+    body = IO.read EMAIL_FIXTURE_PATH.join("email_iPhone.txt").to_s
+    assert_equal "Here is another email", EmailReplyParser.parse_reply(body)
+  end
+
+  def test_parse_out_sent_from_BlackBerry
+    body = IO.read EMAIL_FIXTURE_PATH.join("email_BlackBerry.txt").to_s
+    assert_equal "Here is another email", EmailReplyParser.parse_reply(body)
+  end
+
+
   def test_parse_reply
     body = IO.read EMAIL_FIXTURE_PATH.join("email_1_2.txt").to_s
     assert_equal EmailReplyParser.read(body).visible_text, EmailReplyParser.parse_reply(body)

test/emails/email_BlackBerry.txt

@@ -0,0 +1,3 @@
+Here is another email
+
+Sent from my BlackBerry

test/emails/email_iPhone.txt

@@ -0,0 +1,3 @@
+Here is another email
+
+Sent from my iPhone