Project

General

Profile

Actions

Bug #3518

closed

email verification always optional

Added by Roman Korba over 9 years ago. Updated over 9 years ago.

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

Actions #1

Updated by Koen Deforche over 9 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?

Actions #2

Updated by Korneel Dumon over 9 years ago

  • Status changed from InProgress to Resolved
Actions #3

Updated by Koen Deforche over 9 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF