Page MenuHomePhabricator

Trim and URI encode symbol names before building URIs from them
ClosedPublic

Authored by epriestley on Oct 29 2019, 4:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jun 18 2025, 2:14 PM
Unknown Object (File)
Jun 18 2025, 7:15 AM
Unknown Object (File)
May 13 2025, 6:56 PM
Unknown Object (File)
May 6 2025, 9:35 AM
Unknown Object (File)
May 5 2025, 4:26 PM
Unknown Object (File)
Apr 12 2025, 7:55 PM
Unknown Object (File)
Apr 10 2025, 8:14 PM
Unknown Object (File)
Apr 5 2025, 12:43 PM
Subscribers
None

Details

Summary

Fixes T13437. This URI construction was just missing URI encoding.

Also, trim the symbol because my test case ended up catching "#define\n" as symbol text.

Test Plan
  • Configured a repository to have PHP symbols.
  • Touched a ".php" file with "#define" in it.
  • Diffed the change.
  • Command-clicked "#define" in the UI, in Safari/MacOS, to jump to the definition.
    • Before: taken to a nonsense page where "#define" became an anchor.
    • After: taken to symbol search for "#define".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Oct 29 2019, 4:48 PM
This revision was automatically updated to reflect the committed changes.