From 993b73596adfc8e5a69b32a7cdabc2c913676aef Mon Sep 17 00:00:00 2001 From: epriestley Date: Fri, 13 Jun 2014 07:06:53 -0700 Subject: [PATCH] Fix wording of 'bin/remove' prompt for repositories Summary: This UI recommends `bin/remove destroy X`, but should recommend `bin/remove destroy rX` (with `r`), because the remove script now takes any object monogram. The older script was repository-specific, so it only took the callsign. Test Plan: {F166042} Reviewers: putnam, chad Reviewed By: chad Subscribers: epriestley Differential Revision: https://secure.phabricator.com/D9512 --- .../controller/DiffusionRepositoryEditDeleteController.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/applications/diffusion/controller/DiffusionRepositoryEditDeleteController.php b/src/applications/diffusion/controller/DiffusionRepositoryEditDeleteController.php index c754c4fa90..c7a40fa6ea 100644 --- a/src/applications/diffusion/controller/DiffusionRepositoryEditDeleteController.php +++ b/src/applications/diffusion/controller/DiffusionRepositoryEditDeleteController.php @@ -29,8 +29,8 @@ final class DiffusionRepositoryEditDeleteController 'If you really want to delete the repository, run this command from '. 'the command line:'); $command = csprintf( - 'phabricator/ $ ./bin/remove destroy %s', - $repository->getCallsign()); + 'phabricator/ $ ./bin/remove destroy %R', + $repository->getMonogram()); $text_2 = pht('Repositories touch many objects and as such deletes are '. 'prohibitively expensive to run from the web UI.'); $body = phutil_tag(