From: Rohan Date: Fri, 11 Jul 2014 05:50:32 +0000 (+0530) Subject: Do not require login on Special:Confirmemail to confirm the email X-Git-Tag: 1.31.0-rc.0~7375 X-Git-Url: http://git.cyclocoop.org//%22javascript:ModifierStyle%28%27%22.%24id.%22%27%29/%22?a=commitdiff_plain;h=9b8891402f192de3b3351933e67f1f09dc551e28;p=lhc%2Fweb%2Fwiklou.git Do not require login on Special:Confirmemail to confirm the email The user doesn't need to be logged in to confirm his/her email. Particle revert of I43ceaddb370d09784021b3fc2d5d1ff6616fef1f Bug: T62434 Change-Id: Idf3f9827daaac87f5c57617f4e5d42ca9569839d --- diff --git a/includes/specials/SpecialConfirmemail.php b/includes/specials/SpecialConfirmemail.php index 914781c16a..a656c2ea11 100644 --- a/includes/specials/SpecialConfirmemail.php +++ b/includes/specials/SpecialConfirmemail.php @@ -56,8 +56,6 @@ class EmailConfirmation extends UnlistedSpecialPage { $this->checkReadOnly(); $this->checkPermissions(); - $this->requireLogin( 'confirmemail_needlogin' ); - // This could also let someone check the current email address, so // require both permissions. if ( !$this->getUser()->isAllowed( 'viewmyprivateinfo' ) ) { @@ -65,6 +63,7 @@ class EmailConfirmation extends UnlistedSpecialPage { } if ( $code === null || $code === '' ) { + $this->requireLogin( 'confirmemail_needlogin' ); if ( Sanitizer::validateEmail( $this->getUser()->getEmail() ) ) { $this->showRequestForm(); } else {