Page MenuHomePhabricator

Fix a HunkQuery issue where no hunks load at all
ClosedPublic

Authored by epriestley on Apr 22 2014, 10:29 PM.
Tags
None
Referenced Files
F15515096: D8842.id20979.diff
Fri, Apr 18, 7:39 AM
F15489831: D8842.diff
Fri, Apr 11, 12:35 PM
F15445632: D8842.id.diff
Mar 27 2025, 3:07 PM
F15443673: D8842.id20979.diff
Mar 27 2025, 5:36 AM
F15440845: D8842.diff
Mar 26 2025, 2:57 PM
F15424401: D8842.id20993.diff
Mar 22 2025, 10:03 PM
F15391431: D8842.id20993.diff
Mar 15 2025, 9:50 AM
F15391171: D8842.id20993.diff
Mar 15 2025, 8:32 AM
Subscribers

Details

Summary

If you create a diff with no hunks (e.g., it adds a single empty file), we never attachHunks() so we throw on getHunks().

Instead, make sure changesets get hunks attached if they expect it.

Test Plan

Created a new diff with a single empty file in it.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Fix a HunkQuery issue where no hunks load at all.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.

@zeeg, this should fix the issue you ran into yesterday.

btrahan edited edge metadata.
This revision is now accepted and ready to land.Apr 23 2014, 7:00 PM
epriestley updated this revision to Diff 20993.

Closed by commit rP5f033d580c58 (authored by @epriestley).