Page MenuHomePhabricator

Fix a remarkup rule for `[x](y)` links when "y" is the empty string
ClosedPublic

Authored by epriestley on Feb 11 2019, 9:26 PM.

Details

Summary

See https://discourse.phabricator-community.org/t/uninitialized-string-offset-error-in-phutilremarkupdocumentlinkrule/2379. This test is incorrect (should be against $uri, not $uri[0]) and should happen before we access $uri[0]).

Test Plan
  • Entered [x](), no more error in the logs.
  • Entered [x](http://example.com), got a link.

Diff Detail

Repository
rPHU libphutil
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Feb 11 2019, 9:26 PM
epriestley requested review of this revision.Feb 11 2019, 9:27 PM
amckinley accepted this revision.Feb 11 2019, 11:08 PM
This revision is now accepted and ready to land.Feb 11 2019, 11:08 PM
This revision was automatically updated to reflect the committed changes.