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)
Wed, May 7, 5:50 AM
Unknown Object (File)
Apr 10 2025, 12:01 AM
Unknown Object (File)
Apr 8 2025, 3:31 PM
Unknown Object (File)
Apr 8 2025, 10:35 AM
Unknown Object (File)
Apr 7 2025, 7:47 AM
Unknown Object (File)
Apr 6 2025, 3:38 PM
Unknown Object (File)
Apr 1 2025, 8:54 PM
Unknown Object (File)
Mar 29 2025, 9:20 AM
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