Add a quoted body stripping pattern for "> On ... at ... wrote:"
ClosedPublic
Actions

Authored by epriestley on Oct 30 2013, 1:22 AM.

Tags

None

Referenced Files

	F15467896: D7440.id16805.diff
	Thu, Apr 3, 6:18 PM

	F15467893: D7440.id16762.diff
	Thu, Apr 3, 6:17 PM

	F15467892: D7440.id.diff
	Thu, Apr 3, 6:17 PM

	F15467891: D7440.diff
	Thu, Apr 3, 6:17 PM

	F15467199: D7440.diff
	Thu, Apr 3, 1:42 PM

	F15461239: D7440.id.diff
	Tue, Apr 1, 6:31 AM

	F15459225: D7440.diff
	Mon, Mar 31, 11:09 AM

	F15447420: D7440.diff
	Thu, Mar 27, 11:47 PM

View All Files

Subscribers

aran

scottmac

Details

Reviewers

btrahan

Commits

Restricted Diffusion Commit
rP05884e504240: Add a quoted body stripping pattern for "> On ... at ... wrote:"

Summary

See @scottmac's reply in T3982. It looks like his email client uses the standard quote string, but includes it in the quoted block.

Test Plan

Added a failing unit test, made it pass.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

My client is British in case it makes it worse :)

hehehe

btrahan accepted this revision.Oct 30 2013, 7:29 PM

Closed by commit rP05884e504240 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

metamta/

parser/

PhabricatorMetaMTAEmailBodyParser.php

2 lines

__tests__/

PhabricatorMetaMTAEmailBodyParserTestCase.php

6 lines

Diff 16805

View Options

src/applications/metamta/parser/PhabricatorMetaMTAEmailBodyParser.php

View Options

src/applications/metamta/parser/tests/PhabricatorMetaMTAEmailBodyParserTestCase.php

Add a quoted body stripping pattern for "> On ... at ... wrote:"ClosedPublicActions