Page MenuHomePhabricator

Handle case when Differential Revision diff is an empty commit
Needs ReviewPublic

Authored by artms on Nov 6 2020, 7:00 PM.
Tags
None
Referenced Files
F12838986: D21491.id.diff
Thu, Mar 28, 6:56 PM
Unknown Object (File)
Wed, Mar 27, 2:03 AM
Unknown Object (File)
Sat, Mar 23, 3:46 AM
Unknown Object (File)
Feb 4 2024, 12:31 AM
Unknown Object (File)
Jan 16 2024, 9:11 AM
Unknown Object (File)
Jan 10 2024, 1:25 AM
Unknown Object (File)
Jan 5 2024, 1:16 PM
Unknown Object (File)
Dec 25 2023, 11:39 AM
Subscribers

Details

Reviewers
None
Group Reviewers
Blessed Reviewers
Summary

Diff with empty changeset causes Differential Revision rendering failure (see: https://secure.phabricator.com/D21490).
Such situation might happen if such empty commit was commited to repository and Phabricator picks it up to close corresponding Differential Revsion (this happened in our instance).
It is possible to also reproduce it with heavily modified arcanist where "emptiness" checks are on purpose removed (D21490 case).

Test Plan

Pre patch generate Diff with empty changeset D21490. Try opening - fails. Apply patch - try opening - works, change as expected is displayed as empty

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Skipped
Build Status
Buildable 24965
Build 34447: Run Core Tests
Build 34446: arc lint + arc unit