From 6d5920fa2d23cbccf2cf5ff0852ac3b9536161d0 Mon Sep 17 00:00:00 2001 From: epriestley Date: Thu, 28 Jan 2021 16:30:53 -0800 Subject: [PATCH] Improve help for "bin/policy unlock" to make it explicit that you can name objects with a PHID Summary: See . Test Plan: Ran `bin/policy help unlock`, saw explicit "you can use a PHID" documentation. Differential Revision: https://secure.phabricator.com/D21530 --- .../PhabricatorPolicyManagementUnlockWorkflow.php | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/src/applications/policy/management/PhabricatorPolicyManagementUnlockWorkflow.php b/src/applications/policy/management/PhabricatorPolicyManagementUnlockWorkflow.php index 64a32b7186..eff06f9d5f 100644 --- a/src/applications/policy/management/PhabricatorPolicyManagementUnlockWorkflow.php +++ b/src/applications/policy/management/PhabricatorPolicyManagementUnlockWorkflow.php @@ -8,9 +8,16 @@ final class PhabricatorPolicyManagementUnlockWorkflow ->setName('unlock') ->setSynopsis( pht( - 'Unlock an object which has policies that prevent it from being '. - 'viewed or edited.')) - ->setExamples('**unlock** --view __user__ __object__') + 'Unlock one or more objects by changing their view policies, edit '. + 'policies, or owners.')) + ->setHelp( + pht( + 'Identify each __object__ by passing an object name '. + '(like "T123") or a PHID (like "PHID-ABCD-1234...").'. + "\n\n". + 'Not every type of object has an editable view policy, edit '. + 'policy, or owner, so not all modes will work with all objects. ')) + ->setExamples('**unlock** --view __user__ __object__ ...') ->setArguments( array( array(