Page MenuHomePhabricator

D7903.diff
No OneTemporary

D7903.diff

Index: src/applications/herald/adapter/HeraldCommitAdapter.php
===================================================================
--- src/applications/herald/adapter/HeraldCommitAdapter.php
+++ src/applications/herald/adapter/HeraldCommitAdapter.php
@@ -404,8 +404,11 @@
if (!$revision) {
return null;
}
- $status_accepted = ArcanistDifferentialRevisionStatus::ACCEPTED;
- if ($revision->getStatus() != $status_accepted) {
+ // after a revision is accepted, it can be closed (say via arc land)
+ // so use this function to figure out if it was accepted at one point
+ // *and* not later rejected... what a function!
+ $reviewed_by = $revision->loadReviewedBy();
+ if (!$reviewed_by) {
return null;
}
return $revision->getPHID();

File Metadata

Mime Type
text/plain
Expires
Sat, Mar 15, 4:05 AM (2 w, 4 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7682692
Default Alt Text
D7903.diff (838 B)

Event Timeline