HomePhorge

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, 3:47 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP3ad4be4d9377: Multicolor headers
Branches
Unknown
Tags
Unknown

Event Timeline