From 9b8891402f192de3b3351933e67f1f09dc551e28 Mon Sep 17 00:00:00 2001 From: Rohan Date: Fri, 11 Jul 2014 11:20:32 +0530 Subject: [PATCH] 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 --- includes/specials/SpecialConfirmemail.php | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) 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 { -- 2.20.1