Page MenuHomePhabricator

Fix ambiguous URI parsing in Youtube Remarkup rule
ClosedPublic

Authored by epriestley on Jun 23 2017, 2:37 PM.

Details

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)

Diff Detail

Repository
rP Phabricator
Branch
youtube1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17556
Build 23558: Run Core Tests
Build 23557: arc lint + arc unit