HomePhabricator

Fix ambiguous URI parsing in Youtube Remarkup rule

Description

Fix ambiguous URI parsing in Youtube Remarkup rule

Summary:
Fixes T12867. Also:

  • Simplify the code a little.
  • Stop mutating this on text/mobile -- there's no inherent value in the "youtu.be" link so I think this just changes the text the user wrote unnecessarily.

Test Plan:

Screen Shot 2017-06-23 at 7.32.50 AM.png (552×1 px, 395 KB)

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12867

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

Details

Provenance
epriestleyAuthored on Jun 23 2017, 2:34 PM
epriestleyPushed on Jun 23 2017, 3:43 PM
Reviewer
chad
Differential Revision
D18149: Fix ambiguous URI parsing in Youtube Remarkup rule
Parents
rP219ae8b6c950: Remove old "Landing Strategy" code
Branches
Unknown
Tags
Unknown
Tasks
T12867: Youtube remarkup rule fails to parse "ambiguous URI"
Build Status
Buildable 17559
Build 23562: Run Core Tests