Page MenuHomePhabricator
Differential D7974

Fix excessively aggressive capturing of hyperlinks in Remarkup
ClosedPublic
Actions

Authored by epriestley on Jan 15 2014, 7:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 19, 12:07 AM
Unknown Object (File)
Mon, Dec 9, 1:26 PM
Unknown Object (File)
Sun, Dec 8, 6:30 AM
Unknown Object (File)
Nov 27 2024, 11:16 AM
Unknown Object (File)
Nov 19 2024, 2:06 PM
Unknown Object (File)
Oct 24 2024, 5:03 PM
Unknown Object (File)
Oct 24 2024, 2:28 PM
Unknown Object (File)
Oct 23 2024, 7:35 PM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Firehed
Maniphest Tasks
T4319: Remarkup text rendering bug
Commits
rPHU22285e941272: Fix excessively aggressive capturing of hyperlinks in Remarkup
Summary

Fixes T4319. See that task for examples. The "ungreedy" hyperlink rule is still a bit too greedy.

Test Plan

Added failing tests, made them pass.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Firehed accepted this revision.Jan 15 2014, 7:30 PM

Confirmed to fix T4319 on my install.

epriestley closed this revision.Jan 15 2014, 7:31 PM

Closed by commit rPHU22285e941272 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
markup/
engine/
__tests__/
remarkup/
6 lines
18 lines
remarkup/
4 lines
markuprule/
7 lines

Diff 18044

View Options

src/markup/engine/__tests__/remarkup/link-brackets.txt

Loading...
View Options

src/markup/engine/__tests__/remarkup/link-mixed.txt

Loading...
View Options

src/markup/engine/remarkup/PhutilRemarkupBlockStorage.php

Loading...
View Options

src/markup/engine/remarkup/markuprule/PhutilRemarkupRuleHyperlink.php

Loading...
Phabricator · Built by Phacility