summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat (limited to 'UserMerge/includes/SpecialUserMerge.php')
-rw-r--r--UserMerge/includes/SpecialUserMerge.php13
1 files changed, 3 insertions, 10 deletions
diff --git a/UserMerge/includes/SpecialUserMerge.php b/UserMerge/includes/SpecialUserMerge.php
index 76f2b96c..a4c171bf 100644
--- a/UserMerge/includes/SpecialUserMerge.php
+++ b/UserMerge/includes/SpecialUserMerge.php
@@ -26,15 +26,11 @@ class SpecialUserMerge extends FormSpecialPage {
protected function getFormFields() {
return [
'olduser' => [
- 'type' => 'text',
+ 'type' => 'user',
+ 'exists' => true,
'label-message' => 'usermerge-olduser',
'required' => true,
'validation-callback' => function ( $val ) {
- // only pass strings to User::newFromName
- if ( !is_string( $val ) ) {
- return true;
- }
-
$key = $this->validateOldUser( $val );
if ( is_string( $key ) || is_array( $key ) ) {
return $this->msg( $key )->escaped();
@@ -43,7 +39,7 @@ class SpecialUserMerge extends FormSpecialPage {
},
],
'newuser' => [
- 'type' => 'text',
+ 'type' => 'user',
'required' => true,
'label-message' => 'usermerge-newuser',
'validation-callback' => function ( $val ) {
@@ -73,9 +69,6 @@ class SpecialUserMerge extends FormSpecialPage {
*/
public function validateOldUser( $val ) {
$oldUser = User::newFromName( $val );
- if ( !$oldUser || $oldUser->getId() === 0 ) {
- return 'usermerge-badolduser';
- }
if ( $this->getUser()->getId() === $oldUser->getId() ) {
return [ 'usermerge-noselfdelete', $this->getUser()->getName() ];
}