Changeset View
Changeset View
Standalone View
Standalone View
src/applications/differential/view/DifferentialRevisionListView.php
Show First 20 Lines • Show All 54 Lines • ▼ Show 20 Lines | public function render() { | ||||
$reviewer_limit = 7; | $reviewer_limit = 7; | ||||
$reviewer_phids = array(); | $reviewer_phids = array(); | ||||
$reviewer_more = array(); | $reviewer_more = array(); | ||||
$handle_phids = array(); | $handle_phids = array(); | ||||
foreach ($this->revisions as $key => $revision) { | foreach ($this->revisions as $key => $revision) { | ||||
$reviewers = $revision->getReviewers(); | $reviewers = $revision->getReviewers(); | ||||
// Don't show reviewers who have resigned. The "Reviewers" constraint | |||||
// does not respect these reviewers and they largely don't count as | |||||
// reviewers. | |||||
foreach ($reviewers as $reviewer_key => $reviewer) { | |||||
if ($reviewer->isResigned()) { | |||||
unset($reviewers[$reviewer_key]); | |||||
} | |||||
} | |||||
if (count($reviewers) > $reviewer_limit) { | if (count($reviewers) > $reviewer_limit) { | ||||
$reviewers = array_slice($reviewers, 0, $reviewer_limit); | $reviewers = array_slice($reviewers, 0, $reviewer_limit); | ||||
$reviewer_more[$key] = true; | $reviewer_more[$key] = true; | ||||
} else { | } else { | ||||
$reviewer_more[$key] = false; | $reviewer_more[$key] = false; | ||||
} | } | ||||
$phids = mpull($reviewers, 'getReviewerPHID'); | $phids = mpull($reviewers, 'getReviewerPHID'); | ||||
▲ Show 20 Lines • Show All 191 Lines • Show Last 20 Lines |