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
Unknown Object (File)
Wed, Nov 20, 9:14 PM
Unknown Object (File)
Sun, Nov 17, 6:58 AM
Unknown Object (File)
Wed, Nov 13, 2:48 PM
Unknown Object (File)
Tue, Nov 12, 5:54 AM
Unknown Object (File)
Sat, Nov 9, 6:50 AM
Unknown Object (File)
Sat, Nov 9, 6:39 AM
Unknown Object (File)
Sat, Nov 9, 6:26 AM
Unknown Object (File)
Sat, Nov 9, 6:05 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