jenkins-bot has submitted this change and it was merged.

Change subject: Better error message for invalid user name in cleanupSpam.php
......................................................................


Better error message for invalid user name in cleanupSpam.php

So that one that gets this message does not have to look at
the code to see what is wrong and how to correct it.

Change-Id: I4012ce7db010b9c132e68da9577ba1d5512d13e7
---
M maintenance/cleanupSpam.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Alex Monk: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/maintenance/cleanupSpam.php b/maintenance/cleanupSpam.php
index 9838569..a41423a 100644
--- a/maintenance/cleanupSpam.php
+++ b/maintenance/cleanupSpam.php
@@ -44,7 +44,7 @@
                $username = wfMessage( 'spambot_username' )->text();
                $wgUser = User::newFromName( $username );
                if ( !$wgUser ) {
-                       $this->error( "Invalid username", true );
+                       $this->error( "Invalid username specified in 
'spambot_username' message: $username", true );
                }
                // Create the user if necessary
                if ( !$wgUser->getId() ) {

-- 
To view, visit https://gerrit.wikimedia.org/r/49008
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4012ce7db010b9c132e68da9577ba1d5512d13e7
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: IAlex <[email protected]>
Gerrit-Reviewer: Alex Monk <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to