HomePhabricator

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

Description

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.

Reviewers: btrahan, zeeg

Reviewed By: zeeg

CC: aran

Differential Revision: https://secure.phabricator.com/D7732

Details

Provenance
epriestleyAuthored on Dec 6 2013, 11:47 PM
epriestleyPushed on Dec 6 2013, 11:47 PM
Reviewer
zeeg
Differential Revision
D7732: Fix over-matching of quoted text for message bodies beginning with "On..."
Parents
rP3ad4be4d9377: Multicolor headers
Branches
Unknown
Tags
Unknown

Event Timeline