Merge pull request #28 from team-plain/empty-line

Support no empty line before quote header

Author: turt2live

lib/Parser.js

@@ -128,7 +128,7 @@ class Parser {
                     this._addFragment(fragment, fragments);
 
                     fragment = null;
-                } else if (line.length === 0 && this._isQuoteHeader(lastLine)) {
+                } else if (this._isQuoteHeader(lastLine)) {
                     fragment.isQuoted = true;
                     this._addFragment(fragment, fragments);
 

test/ParserTest.js

@@ -449,6 +449,17 @@ describe('the Parser', function () {
         assert.equal(/Was this/.test(fragments[1].getContent()), true, "Second fragment has wrong content");
     });
 
+    it('should parse an email where the quote header does not have a preceeding empty line', function () {
+        var parser = new Parser();
+        var fixture = util.getFixture("email_quote_header_without_new_line.txt");
+        var email = parser.parse(fixture);
+        var fragments = email.getFragments();
+
+        assert.equal(/^ Gran Via, 1, 28004, Madrid$/.test(email.getVisibleText()), true)
+        assert.equal(/^ Gran Via, 1, 28004, Madrid/.test(fragments[0].getContent()), true, "First fragment has wrong content");
+        assert.equal(/^    On Wednesday, 23 February 2022/.test(fragments[1].getContent()), true, "Second fragment has wrong content");
+    });
+
     it('should parse visible text that looks like a quote header (second)', function () {
         var parser = new Parser();
         var fixture = util.getFixture("email_20.txt");

test/fixtures/email_quote_header_without_new_line.txt

@@ -0,0 +1,8 @@
+
+ Gran Via, 1, 28004, Madrid
+    On Wednesday, 23 February 2022, 14:46:52 CET, Alicia
+ <[email protected]> wrote:
+
+ Hello,
+May I get your postal address?
+Thanks