From ad22508e4fea5a5c90d7972ab7c0ec2388eef324 Mon Sep 17 00:00:00 2001 From: Joshua Spence Date: Thu, 26 Jun 2014 04:52:34 +1000 Subject: [PATCH] Always show repository in Differential emails. Summary: Ref T5137. A slight modification to D9609, such that the repository is always included in Differential emails. Otherwise "Accepted", "Closed" and "Requested Changes To" emails don't include the repository. Test Plan: Not tested. Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: epriestley, Korvin Maniphest Tasks: T5137 Differential Revision: https://secure.phabricator.com/D9728 --- .../differential/customfield/DifferentialRepositoryField.php | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/applications/differential/customfield/DifferentialRepositoryField.php b/src/applications/differential/customfield/DifferentialRepositoryField.php index 80d00a4eb8..0bd430ec7b 100644 --- a/src/applications/differential/customfield/DifferentialRepositoryField.php +++ b/src/applications/differential/customfield/DifferentialRepositoryField.php @@ -152,10 +152,6 @@ final class DifferentialRepositoryField PhabricatorApplicationTransactionEditor $editor, array $xactions) { - if (!$editor->getDiffUpdateTransaction($xactions)) { - return; - } - $repository = $this->getObject()->getRepository(); if ($repository === null) { return;