From: Brion Vibber <brion@users.mediawiki.org>
Date: Tue, 19 Aug 2008 19:11:27 +0000 (+0000)
Subject: Revert r39625 "Run autocreated accounts through AbortNewAccount hooks"
X-Git-Tag: 1.31.0-rc.0~45790
X-Git-Url: http://git.cyclocoop.org/%7B%24www_url%7Dadmin/compta/operations/?a=commitdiff_plain;h=3d71f0c39ed59db6306efa1b916f6524f0b92af4;p=lhc%2Fweb%2Fwiklou.git

Revert r39625 "Run autocreated accounts through AbortNewAccount hooks"
This seems to be mixing UI and backend code.

Further I'm unsure we want to treat authplugin autocreations as "new accounts" for this purpose; since they *have* been previously created and perhaps approved, just blindly running it through the same abort hook again will be a huge pain in the butt, for instance for those who had to go through admin approval due to a machine-detected name spoof conflict or an IP blocking issue. If an abort is appropriate, it should probably be a distinct hook so it can be treated differently.
---

diff --git a/includes/specials/SpecialUserlogin.php b/includes/specials/SpecialUserlogin.php
index 551d9c06e2..00c93d6513 100644
--- a/includes/specials/SpecialUserlogin.php
+++ b/includes/specials/SpecialUserlogin.php
@@ -500,14 +500,6 @@ class LoginForm {
 			return self::CREATE_BLOCKED;
 		}
 
-		$abortError = '';
-		if( !wfRunHooks( 'AbortNewAccount', array( $user->getName(), &$abortError ) ) ) {
-			// Hook point to add extra creation throttles and blocks
-			wfDebug(  __METHOD__.": a hook blocked creation\n" );
-			$this->mainLoginForm( $abortError );
-			return self::ABORTED;
-		}
-
 		wfDebug( __METHOD__.": creating account\n" );
 		$user = $this->initUser( $user, true );
 		return self::SUCCESS;