archive/phabricator
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Don't write inverse edges for DifferentialRevisionHasReviewerEdgeType

Browse Source 
Summary: The default behavior was inadvertedly changed in D11074. This restores the original behavior.

Test Plan: Added a project reviewer to a diff, saw no inverse transaction recorded.

Reviewers: Krenair, epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D11181
This commit is contained in:
Joshua Spence
2015-01-04 10:07:55 +11:00
parent 21315febc2
commit 2dea11047a
1 changed files with 0 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/applications/differential/edge/DifferentialRevisionHasReviewerEdgeType.php
View File

@@ -9,10 +9,6 @@ final class DifferentialRevisionHasReviewerEdgeType
return DifferentialReviewerForRevisionEdgeType::EDGECONST; return DifferentialReviewerForRevisionEdgeType::EDGECONST;
} }
public function shouldWriteInverseTransactions() {
return true;
}
public function getTransactionAddString( public function getTransactionAddString(
$actor, $actor,
$add_count, $add_count,