Page MenuHomeDevCentral

Fix null exception when Phabricator repository is private
ClosedPublic

Authored by dereckson on Jan 28 2017, 00:01.
Tags
None
Referenced Files
F7828542: D872.diff
Sat, May 3, 15:31
Unknown Object (File)
Thu, May 1, 08:20
Unknown Object (File)
Sat, Apr 26, 22:37
Unknown Object (File)
Sat, Apr 26, 02:47
Unknown Object (File)
Fri, Apr 25, 12:05
Unknown Object (File)
Mon, Apr 21, 05:16
Unknown Object (File)
Sun, Apr 20, 22:48
Unknown Object (File)
Sat, Apr 19, 15:44
Subscribers
None

Details

Summary

When a repository is private and can't be reached by the bot account
used to fetch Phabricator information, a null exception occurs in
PhabricatorStory::getRepositoryPHID method.

This change handles the case an API query doesn't contain the
required information.

Fixes T1136.

Test Plan

Fire again the notification.

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Passed
Unit
No Test Coverage
Branch
T1136 (branched from master)
Build Status
Buildable 1346
Build 1594: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Jan 28 2017, 00:02
This revision was automatically updated to reflect the committed changes.