From 475afe4a5b51941203a61d9d32590d978cc8a513 Mon Sep 17 00:00:00 2001 From: epriestley Date: Wed, 25 Sep 2013 13:39:47 -0700 Subject: [PATCH] Show full names for attached diffs and tasks in email in Maniphest Summary: Ref T2217. Use `getLinkName()` instead of `getName()` so that we get, e.g., alincoln attached a revision: D123 Chop some logs ...instead of: alincoln attached a revision: D123 Test Plan: Attached stuff, looked at the email, saw full object name. Reviewers: btrahan, asherkin Reviewed By: btrahan CC: aran Maniphest Tasks: T2217 Differential Revision: https://secure.phabricator.com/D7127 --- .../transactions/storage/PhabricatorApplicationTransaction.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/applications/transactions/storage/PhabricatorApplicationTransaction.php b/src/applications/transactions/storage/PhabricatorApplicationTransaction.php index 4442cd038f..b1c96f1dc7 100644 --- a/src/applications/transactions/storage/PhabricatorApplicationTransaction.php +++ b/src/applications/transactions/storage/PhabricatorApplicationTransaction.php @@ -190,7 +190,7 @@ abstract class PhabricatorApplicationTransaction if ($this->renderingTarget == self::TARGET_HTML) { return $this->getHandle($phid)->renderLink(); } else { - return hsprintf('%s', $this->getHandle($phid)->getName()); + return hsprintf('%s', $this->getHandle($phid)->getLinkName()); } }