Page MenuHomePhabricator

D17511.diff
No OneTemporary

D17511.diff

diff --git a/src/applications/differential/herald/DifferentialReviewersHeraldAction.php b/src/applications/differential/herald/DifferentialReviewersHeraldAction.php
--- a/src/applications/differential/herald/DifferentialReviewersHeraldAction.php
+++ b/src/applications/differential/herald/DifferentialReviewersHeraldAction.php
@@ -81,18 +81,17 @@
$value = array();
foreach ($phids as $phid) {
- $value[$phid] = array(
- 'data' => array(
- 'status' => $new_status,
- ),
- );
+ if ($is_blocking) {
+ $value[] = 'blocking('.$phid.')';
+ } else {
+ $value[] = $phid;
+ }
}
- $edgetype_reviewer = DifferentialRevisionHasReviewerEdgeType::EDGECONST;
+ $reviewers_type = DifferentialRevisionReviewersTransaction::TRANSACTIONTYPE;
$xaction = $adapter->newTransaction()
- ->setTransactionType(PhabricatorTransactions::TYPE_EDGE)
- ->setMetadataValue('edge:type', $edgetype_reviewer)
+ ->setTransactionType($reviewers_type)
->setNewValue(
array(
'+' => $value,

File Metadata

Mime Type
text/plain
Expires
Wed, Mar 5, 9:35 AM (16 h, 38 m)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7224721
Default Alt Text
D17511.diff (1 KB)

Event Timeline