Fix over-matching of quoted text for message bodies beginning with "On..."
ClosedPublic
Actions

Authored by epriestley on Dec 6 2013, 11:41 PM.

Tags

None

Referenced Files

	F18749332: D7732.id.diff
	Sat, Oct 4, 3:23 AM

	F18745778: D7732.diff
	Fri, Oct 3, 9:21 AM

	F18590710: D7732.id.diff
	Sep 12 2025, 6:55 AM

	F18453345: D7732.id.diff
	Sep 1 2025, 3:32 AM

	F18434677: D7732.diff
	Aug 31 2025, 9:32 AM

	F18015843: D7732.id17465.diff
	Aug 2 2025, 6:01 PM

	F17953706: D7732.id.diff
	Aug 1 2025, 3:22 AM

	F17944392: D7732.diff
	Jul 31 2025, 9:59 AM

View All Files

Subscribers

aran

Details

Reviewers

zeeg
btrahan

Commits

Restricted Diffusion Commit
rPf69793184e0b: Fix over-matching of quoted text for message bodies beginning with "On..."

Summary

A user sent a message to Phabricator which looked like:

On blah blah blah ?

On <date>, <user> wrote:
> blah blah blah

The current algorithm is too aggressive and thinks lines 1-3 are all the "On ... wrote:" string. Instead, patch only the most recent "On".

Test Plan

Added a failing test and made it pass.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

literally only read the test code

Closed by commit rPf69793184e0b (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

metamta/

parser/

PhabricatorMetaMTAEmailBodyParser.php

27 lines

__tests__/

PhabricatorMetaMTAEmailBodyParserTestCase.php

14 lines

Diff 17465

View Options

src/applications/metamta/parser/PhabricatorMetaMTAEmailBodyParser.php

View Options

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

Fix over-matching of quoted text for message bodies beginning with "On..."ClosedPublicActions