Page MenuHomePhabricator

When building a synthetic diff in Differential, adjust diff properties correctly
ClosedPublic

Authored by epriestley on Apr 25 2019, 10:09 PM.
Tags
None
Referenced Files
F12835792: D20478.diff
Thu, Mar 28, 3:24 PM
Unknown Object (File)
Tue, Mar 19, 3:28 PM
Unknown Object (File)
Tue, Mar 19, 3:28 PM
Unknown Object (File)
Tue, Mar 19, 3:28 PM
Unknown Object (File)
Tue, Mar 19, 3:27 PM
Unknown Object (File)
Sun, Mar 10, 4:03 AM
Unknown Object (File)
Jan 31 2024, 5:32 PM
Unknown Object (File)
Jan 23 2024, 9:32 AM
Subscribers
None

Details

Reviewers
amckinley
Summary

See PHI1218. When rendering "A vs B", we currently show the properties of diff A without modification.

Instead, take properties from the same place we're taking change details.

See T12664 for a followup.

Test Plan
  • In diff A, removed "+x" from a file.
  • In diff B, changed the file but did not remove "+x" from it.
  • Diffed B vs A.
    • Before change: UI incorrectly shows "+x" removed (both sides incorrect, just showing the change from diff A).
    • After change: UI shows 100644 -> null, which is half right.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable