Actions
Bug #3518
closedemail verification always optional
Start date:
08/07/2014
Due date:
% Done:
0%
Estimated time:
Description
Hi,
I could not make my AuthWidget to require email verification with current version from git. It always is optional.
The problem is probably in this method:
RegistrationModel::RegistrationModel(...){
...
if (baseAuth.emailVerificationEnabled())
emailPolicy_ = EmailOptional;
else if (baseAuth.emailVerificationRequired())
emailPolicy_ = EmailMandatory;
else
emailPolicy_ = EmailDisabled;
...
}
But unfortunatly AuthService::setEmailVerificationRequired emailVerification_ is set to true if verification is required.
void AuthService::setEmailVerificationRequired(bool enabled)
{
emailVerificationReq_ = enabled;
if (enabled)
emailVerification_ = true;
}
So the order of ifs in RegistrationModel::RegistrationModel is wrong.
Updated by Koen Deforche over 10 years ago
- Status changed from New to InProgress
- Assignee set to Korneel Dumon
- Target version set to 3.3.3
Korneel I believe the analysis correct?
Updated by Korneel Dumon over 10 years ago
- Status changed from InProgress to Resolved
Updated by Koen Deforche about 10 years ago
- Status changed from Resolved to Closed
Actions