Page MenuHomePhabricator

D8494.id.diff
No OneTemporary

D8494.id.diff

diff --git a/src/applications/differential/customfield/DifferentialReviewersField.php b/src/applications/differential/customfield/DifferentialReviewersField.php
--- a/src/applications/differential/customfield/DifferentialReviewersField.php
+++ b/src/applications/differential/customfield/DifferentialReviewersField.php
@@ -152,10 +152,22 @@
}
public function readValueFromCommitMessage($value) {
+ $current_reviewers = $this->getObject()->getReviewerStatus();
+ $current_reviewers = mpull($current_reviewers, null, 'getReviewerPHID');
+
$reviewers = array();
foreach ($value as $phid) {
- $reviewers[] = new DifferentialReviewer($phid, array());
+ $reviewer = idx($current_reviewers, $phid);
+ if ($reviewer) {
+ $reviewers[] = $reviewer;
+ } else {
+ $data = array(
+ 'status' => DifferentialReviewerStatus::STATUS_ADDED,
+ );
+ $reviewers[] = new DifferentialReviewer($phid, $data);
+ }
}
+
$this->setValue($reviewers);
return $this;

File Metadata

Mime Type
text/plain
Expires
Mar 21 2025, 11:47 AM (5 w, 3 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7615599
Default Alt Text
D8494.id.diff (1 KB)

Event Timeline