HomePhorge

Fix an issue with symbol lookup identifying path names in Diffusion

Description

Fix an issue with symbol lookup identifying path names in Diffusion

Summary:
Depends on D18939. Ref T13047. Symbol lookup can be activated from a diff (in Differential or Diffusion) or from the static view of a file at a particular commit.

In the latter case, we need to figure out the path a little differently. The character and line number approaches still work as written.

Test Plan:

  • Command-clicked symbols in the Diffusion browse view with blame on and off; saw path, line and char populate properly.
  • Command-clicked symbols in Differential diff view to check I didn't break anything.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T13047

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

Details

Provenance
epriestleyAuthored on Jan 26 2018, 6:42 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPfdc36677ba4f: Provide character position information to symbol queries
Branches
Unknown
Tags
Unknown

Event Timeline