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

	F18470770: D7440.diff
	Tue, Sep 2, 6:26 PM

	F18470762: D7440.diff
	Tue, Sep 2, 6:26 PM

	F18106802: D7440.diff
	Aug 10 2025, 11:23 PM

	F18086907: D7440.diff
	Aug 6 2025, 1:02 AM

	F18009906: D7440.id16805.diff
	Aug 2 2025, 3:39 PM

	F17958749: D7440.id.diff
	Aug 1 2025, 7:19 AM

	F17945008: D7440.diff
	Jul 31 2025, 11:08 AM

	F17645828: D7440.diff
	Jul 12 2025, 12:04 AM

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