Page MenuHomePhabricator

Fix a Diffusion issue where commits that do not show changesets would incorrectly try to render changesets
ClosedPublic

Authored by epriestley on Apr 27 2020, 5:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jan 28, 4:21 AM
Unknown Object (File)
Tue, Jan 21, 3:45 AM
Unknown Object (File)
Dec 16 2024, 5:48 AM
Unknown Object (File)
Dec 16 2024, 5:48 AM
Unknown Object (File)
Dec 16 2024, 5:48 AM
Unknown Object (File)
Dec 15 2024, 2:33 AM
Unknown Object (File)
Dec 14 2024, 3:59 PM
Unknown Object (File)
Nov 27 2024, 11:09 PM
Subscribers
None

Details

Summary

See https://discourse.phabricator-community.org/t/loading-certain-svn-commits-cause-unhandled-exception/3795/.

Commits with no changesets (for example, deleted commits) don't generate a "$changesets".

Test Plan

Viewed a commit with no changesets. Before change: exception. After change: saw unusual commit state.

Diff Detail

Repository
rP Phabricator
Branch
changeset1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24242
Build 33390: Run Core Tests
Build 33389: arc lint + arc unit