HomePhorge

Fix reticle for diff-of-diffs

Description

Fix reticle for diff-of-diffs

Summary:
The older logic was incorrect:

  • It chose change.left for data.on_right being true.
  • 'O' and 'N' mean 'old' and 'new', not 'left' and 'right'. In diff-of-diffs, both sides are 'N'.

So, select the changeset ID correctly (pick the right side one for on_right), and select the new file prefix correctly (N for new, O for old).

Test Plan: Waved my mouse over some inline comments in a diff-of-diffs, got reasonable-looking reticles.

Reviewers: vrana, btrahan

Reviewed By: vrana

CC: aran

Maniphest Tasks: T1076

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

Details

Provenance
epriestleyAuthored on Apr 7 2012, 10:39 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP7507981f2264: Don't fatal with inaccessible database
Branches
Unknown
Tags
Unknown

Event Timeline