Extension talk:ConfirmAccount/archive 1

Download link is busted. &#91;user:jldupont&#124;Jean-Lou Dupont] 21:44, 12 July 2007 (UTC)
 * I haven't yet uploaded it. It's only 1/3 done :) Aaron 21:49, 12 July 2007 (UTC)
 * Up now. Aaron 07:42, 15 July 2007 (UTC)

Permission denied when non-user tries to queue a new account
Maybe I am understanding this incorrectly, but I think your extension allows non-users to submit sign up details, such that the admin gets to approve which submissions become users.

On my installation, the sign up form requires that the user is already authenticated.

Have I misconfigured or do I have the wrong idea about what your plugin does?

Thanks, Martin.
 * I'm note quite sure what you mean. Anyone can sign up. Aaron 18:57, 18 July 2007 (UTC)
 * Thanks for your reply. I had the same problem as the report below.

No signup possible when extension is active
This extension sounds to be exactly what I was looking for. My problem is, that I don't get it working. In the moment, when I include the extension, the signup link is removed from the Special:Userlogin page. That seems to be the case because in SpecialConfirmAccount.php I read: $wgGroupPermissions['*']['createaccount'] = false; Okay, if I change this to true after including the extension, the extension is disabled, because everybody can create his account without confirmation. But what's the trick to get to some confirmation page? Do I have to modify SpecialUserlogin.php? --Rrosenfeld 16:35, 18 July 2007 (UTC)
 * The "create account" link is replaced with a "request one" link. See this test site for example. Aaron 19:00, 18 July 2007 (UTC)
 * That's what I expected, when I read the documentation and the code. My problem is, that the "request one" link does not appear and when I manually enter Special:RequestAccount this tells me that authentication is required.  I already added "Special:RequestAccount" and  "Spezial:RequestAccount" to $wgWhitelistRead, but this doesn't help. I see what you mean on your demo site, but cannot get this behavior on my one. --Rrosenfeld 20:01, 18 July 2007 (UTC)
 * What do you mean by "authentication is required". Is that what it actually says? Aaron 20:08, 18 July 2007 (UTC)
 * My fault was to try to enter "Special:RequestAccount" via the wiki search. This gives me "Login Required.  You must log in to view other pages.".  But if I enter Special:RequestAccount directly to the URL, I get the intended form and can register a new user.  One problem persists: The Special:RequestAccount link on Special:Userlogin is still missing. --Rrosenfeld 20:29, 18 July 2007 (UTC)
 * Ah, now I found a workaround: If I replace 'header' with 'link' in SpecialConfirmAccount.php function efAddRequestLoginText, the link to Special:RequestAccount is shown on my system. BTW: Is someone working on a German translation of this extension?  If not, I'll try one in the next days... --Rrosenfeld 20:55, 18 July 2007 (UTC)
 * Not that I know of. Submit a patch to bugzilla as a bug for de translations (and any else you can/want to do). Anyways, setting the header seems to work on the test site, maybe it only works on MW 1.10+. I may need some extra backwards-compatibility checking. Aaron 22:12, 18 July 2007 (UTC)
 * For the records: I'm using 1.10.1 --Rrosenfeld 07:02, 19 July 2007 (UTC)
 * Right, I checked, and headers for the login template were there too, and the hook my extension uses to set it. Not sure why it's not working for you. Aaron 07:51, 19 July 2007 (UTC)
 * In my case (MW 1.10.0), I found that the 'header' hook for plugins is missing from the userloginForm in template/Userlogin.php. Adding it fixes the problem, as does the workaround of changing 'header' to 'link' in SpecialConfirmAccount.php. --Jennis 13:03, 29 July 2007 (UTC)

Fatal error: Call to undefined method User::isPingLimitable extensions/ConfirmAccount/ConfirmAccount_body.php on line 216
I'm using MW 1.9.2 - cheers, Martin.
 * Hmmm, that functions was added in 1.10, I forgot. I'll have to find some way to do an existence check for 1.8-1.9 support. Aaron 19:14, 18 July 2007 (UTC)
 * OK, fixed. Aaron 19:21, 18 July 2007 (UTC)

German Translation
You'll find a first draft of a German translation at http://www.spinnaker.de/tmp/ConfirmAccount.i18n.php.txt (sorry, no idea, how to corretly file a bug report for this) --Rrosenfeld 11:08, 25 July 2007 (UTC)
 * OK, let me know when it's done :) Aaron 04:24, 30 July 2007 (UTC)

Remove HTML code from confirmaccount-email-body[1-4]
I had some trouble with the accept/reject E-Mails, which contained HTML code (  and the like). To get rid of this, the following patch should help: --- ConfirmAccount_body.php.org	2007-07-25 10:48:00.362520000 +0200 +++ ConfirmAccount_body.php	2007-07-25 12:55:50.185854304 +0200 @@ -420,10 +420,10 @@ 			if( !$row->acr_rejected ) { if( $this->reason ) { $result = $u->sendMail( wfMsg( 'confirmaccount-email-subj' ), -						wfMsgExt( 'confirmaccount-email-body4', array('parseinline'), $u->getName, $this->reason ) ); +						wfMsg( 'confirmaccount-email-body4', $u->getName, $this->reason ) ); 				} else { 					$result = $u->sendMail( wfMsg( 'confirmaccount-email-subj' ), -						wfMsgExt( 'confirmaccount-email-body3', array('parseinline'), $u->getName ) ); +						wfMsg( 'confirmaccount-email-body3', $u->getName ) ); } 				if( WikiError::isError( $result ) ) { $error = wfMsg( 'mailerror', htmlspecialchars( $result->getMessage ) ); @@ -468,10 +468,10 @@ 			$user->setEmail( $row->acr_email ); if( $this->reason ) { $result = $user->sendMail( wfMsg( 'confirmaccount-email-subj' ), -					wfMsgExt( 'confirmaccount-email-body2', array('parseinline'), $user->getName, $p, $this->reason ) ); +					wfMsg( 'confirmaccount-email-body2', $user->getName, $p, $this->reason ) ); 			} else { 				$result = $user->sendMail( wfMsg( 'confirmaccount-email-subj' ), -					wfMsgExt( 'confirmaccount-email-body', array('parseinline'), $user->getName, $p ) ); +					wfMsg( 'confirmaccount-email-body', $user->getName, $p ) ); } 			if( WikiError::isError( $result ) ) { $error = wfMsg( 'mailerror', htmlspecialchars( $result->getMessage ) ); --Rrosenfeld 11:12, 25 July 2007 (UTC)
 * This fixed on SVN a few days ago after a user pointed it out. Also, I did no a schema change with acr_rejected, so make sure yours is char(14) and not a boolean. I'll be marking this as stable after final code review, which is when people should be using extensions, otherwise things might change a bit. Aaron 10:27, 26 July 2007 (UTC)

Wishlist
I'm looking for a mechanism to send an information mail to the bureaucrats (or a subset of them?), that a new user registered and confirmed his e-mail. I'm not fully sure how to implement this, but it would hurt if someone waits for a long time in the queue without noticing him. As a workaround I could set up a cron job that sends out a list of all registered and not rejected account requests to the bureaucreats, but a mail just in time could be nicer. --Rrosenfeld 14:57, 25 July 2007 (UTC)