Project

General

Profile

Actions
Copy link

Bug #3518

closed
RK KD

email verification always optional

Bug #3518: email verification always optional

Added by Roman Korba about 12 years ago. Updated almost 12 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Korneel Dumon
Target version:
3.3.3
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.

KD Updated by Koen Deforche about 12 years ago Actions
Copy link
 #1

  • Status changed from New to InProgress
  • Assignee set to Korneel Dumon
  • Target version set to 3.3.3

Korneel I believe the analysis correct?

KD Updated by Korneel Dumon about 12 years ago Actions
Copy link
 #2

  • Status changed from InProgress to Resolved

KD Updated by Koen Deforche almost 12 years ago Actions
Copy link
 #3

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: PDF Atom