From 3d44208e4f6559fb79e5dbd9fe7e8804b4d65865 Mon Sep 17 00:00:00 2001 From: epriestley Date: Mon, 6 Feb 2017 05:01:03 -0800 Subject: [PATCH] Clarify that "account.editable" no longer extends to profile pictures Summary: Fixes T12216. I'd like to remove this option eventually, but just narrow its scope in the config description for now. Test Plan: Read config description. Reviewers: chad Reviewed By: chad Maniphest Tasks: T12216 Differential Revision: https://secure.phabricator.com/D17317 --- .../PhabricatorAuthenticationConfigOptions.php | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/src/applications/config/option/PhabricatorAuthenticationConfigOptions.php b/src/applications/config/option/PhabricatorAuthenticationConfigOptions.php index bd1c8965eb..2239d98858 100644 --- a/src/applications/config/option/PhabricatorAuthenticationConfigOptions.php +++ b/src/applications/config/option/PhabricatorAuthenticationConfigOptions.php @@ -84,11 +84,13 @@ final class PhabricatorAuthenticationConfigOptions 'Determines whether or not basic account information is editable.')) ->setDescription( pht( - 'Is basic account information (email, real name, profile '. - 'picture) editable? If you set up Phabricator to automatically '. - 'synchronize account information from some other authoritative '. - 'system, you can disable this to ensure information remains '. - 'consistent across both systems.')), + 'This option controls whether users can edit account email '. + 'addresses and profile real names.'. + "\n\n". + 'If you set up Phabricator to automatically synchronize account '. + 'information from some other authoritative system, you can '. + 'prevent users from making these edits to ensure information '. + 'remains consistent across both systems.')), $this->newOption('account.minimum-password-length', 'int', 8) ->setSummary(pht('Minimum password length.')) ->setDescription(