Page MenuHomePhabricator
Files F15394355

D20826.id49656.diff
No OneTemporary
Actions

D20826.id49656.diff
View Options

diff --git a/src/applications/people/controller/PhabricatorPeopleRenameController.php b/src/applications/people/controller/PhabricatorPeopleRenameController.php
--- a/src/applications/people/controller/PhabricatorPeopleRenameController.php
+++ b/src/applications/people/controller/PhabricatorPeopleRenameController.php
@@ -43,32 +43,25 @@
}
- $inst1 = pht(
- 'Be careful when renaming users!');
-
- $inst2 = pht(
- 'The old username will no longer be tied to the user, so anything '.
- 'which uses it (like old commit messages) will no longer associate '.
- 'correctly. (And, if you give a user a username which some other user '.
- 'used to have, username lookups will begin returning the wrong user.)');
-
- $inst3 = pht(
- 'It is generally safe to rename newly created users (and test users '.
- 'and so on), but less safe to rename established users and unsafe to '.
- 'reissue a username.');
-
- $inst4 = pht(
- 'Users who rely on password authentication will need to reset their '.
- 'password after their username is changed (their username is part of '.
- 'the salt in the password hash).');
-
- $inst5 = pht(
+ $instructions = array();
+
+ $instructions[] = pht(
+ 'If you rename this user, the old username will no longer be tied '.
+ 'to the user account. Anything which uses the old username in raw '.
+ 'text (like old commit messages) may no longer associate correctly.');
+
+ $instructions[] = pht(
+ 'It is generally safe to rename users, but changing usernames may '.
+ 'create occasional minor complications or confusion with text that '.
+ 'contains the old username.');
+
+ $instructions[] = pht(
'The user will receive an email notifying them that you changed their '.
- 'username, with instructions for logging in and resetting their '.
- 'password if necessary.');
+ 'username.');
+
+ $instructions[] = null;
$form = id(new AphrontFormView())
- ->setUser($viewer)
->appendChild(
id(new AphrontFormStaticControl())
->setLabel(pht('Old Username'))
@@ -79,19 +72,20 @@
->setValue($username)
->setName('username'));
- return $this->newDialog()
- ->setWidth(AphrontDialogView::WIDTH_FORM)
+ $dialog = $this->newDialog()
->setTitle(pht('Change Username'))
- ->setValidationException($validation_exception)
- ->appendParagraph($inst1)
- ->appendParagraph($inst2)
- ->appendParagraph($inst3)
- ->appendParagraph($inst4)
- ->appendParagraph($inst5)
- ->appendParagraph(null)
+ ->setValidationException($validation_exception);
+
+ foreach ($instructions as $instruction) {
+ $dialog->appendParagraph($instruction);
+ }
+
+ $dialog
->appendForm($form)
->addSubmitButton(pht('Rename User'))
->addCancelButton($done_uri);
+
+ return $dialog;
}
}

File Metadata

Mime Type
text/plain
Expires
Sun, Mar 16, 11:49 PM (6 d, 13 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7707868
Default Alt Text
D20826.id49656.diff (2 KB)

Event Timeline

Phabricator · Built by Phacility