From 39dfcf4c899a735c6e4adea97d44ff28414f9599 Mon Sep 17 00:00:00 2001 From: epriestley Date: Wed, 6 Apr 2016 16:05:38 -0700 Subject: [PATCH] Provide nicer string for trying to move a task to its current columns Summary: Ref T6027. We got a not-very-user-friendly default string before. Test Plan: Selected "Move", didn't change the dropdown, hit submit. Now, got a nice human-readable description of the issue. Reviewers: chad Reviewed By: chad Maniphest Tasks: T6027 Differential Revision: https://secure.phabricator.com/D15649 --- .../storage/PhabricatorApplicationTransaction.php | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/applications/transactions/storage/PhabricatorApplicationTransaction.php b/src/applications/transactions/storage/PhabricatorApplicationTransaction.php index 571a913f74..e3fe7707f6 100644 --- a/src/applications/transactions/storage/PhabricatorApplicationTransaction.php +++ b/src/applications/transactions/storage/PhabricatorApplicationTransaction.php @@ -727,6 +727,10 @@ abstract class PhabricatorApplicationTransaction return pht('This object is already in that space.'); case PhabricatorTransactions::TYPE_EDGE: return pht('Edges already exist; transaction has no effect.'); + case PhabricatorTransactions::TYPE_COLUMNS: + return pht( + 'You have not moved this object to any columns it is not '. + 'already in.'); } return pht(