From 2dea11047aa8180e932fca1864a97cdb7840e8b6 Mon Sep 17 00:00:00 2001 From: Joshua Spence Date: Sun, 4 Jan 2015 10:07:55 +1100 Subject: [PATCH] Don't write inverse edges for `DifferentialRevisionHasReviewerEdgeType` 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 --- .../edge/DifferentialRevisionHasReviewerEdgeType.php | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/applications/differential/edge/DifferentialRevisionHasReviewerEdgeType.php b/src/applications/differential/edge/DifferentialRevisionHasReviewerEdgeType.php index 3a1c42816e..e7ed1217be 100644 --- a/src/applications/differential/edge/DifferentialRevisionHasReviewerEdgeType.php +++ b/src/applications/differential/edge/DifferentialRevisionHasReviewerEdgeType.php @@ -9,10 +9,6 @@ final class DifferentialRevisionHasReviewerEdgeType return DifferentialReviewerForRevisionEdgeType::EDGECONST; } - public function shouldWriteInverseTransactions() { - return true; - } - public function getTransactionAddString( $actor, $add_count,