Page MenuHomePhabricator

Don't fatal when viewing tags pointing at commits we haven't imported/parsed yet
ClosedPublic

Authored by epriestley on Jul 28 2017, 5:41 PM.
Tags
None
Referenced Files
F18782895: D18290.id43983.diff
Mon, Oct 13, 2:54 AM
F18780492: D18290.id.diff
Sun, Oct 12, 4:36 PM
F18733696: D18290.id.diff
Tue, Sep 30, 10:40 PM
F18705524: D18290.id.diff
Sun, Sep 28, 11:43 AM
F18700558: D18290.diff
Sat, Sep 27, 3:23 PM
F18630940: D18290.id43983.diff
Sep 16 2025, 1:41 PM
F18630937: D18290.id43983.diff
Sep 16 2025, 1:41 PM
F18593207: D18290.diff
Sep 12 2025, 2:10 PM
Subscribers
None
Tokens
"Like" token, awarded by jlap.

Details

Summary

In Diffusion, the "Tags" view may read commits which haven't imported or parsed yet, and thus don't have loadable objects.

Most of this logic tests for if ($commit), but the author part did not. Instead, don't render author information if $commit is not present.

Test Plan
  • Loaded tags view with commits present.
  • Faked $commit = null;, loaded tag view, got this instead of a fatal:

Screen Shot 2017-07-28 at 10.39.21 AM.png (903×1 px, 113 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable