Minor, fail gracefully if there are data integrity problems until I can fix oauth transactions.
This commit is contained in:
@@ -75,7 +75,12 @@ final class PhabricatorPeopleQuery extends PhabricatorOffsetPagedQuery {
|
||||
mpull($users, 'getPHID'));
|
||||
$emails = mpull($emails, null, 'getUserPHID');
|
||||
foreach ($users as $user) {
|
||||
$user->attachPrimaryEmail($emails[$user->getPHID()]);
|
||||
// Fail gracefully if we have data integrity problems.
|
||||
if (empty($emails[$user->getPHID()])) {
|
||||
$user->attachPrimaryEmail(new PhabricatorUserEmail());
|
||||
} else {
|
||||
$user->attachPrimaryEmail($emails[$user->getPHID()]);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user