sentora-core icon indicating copy to clipboard operation
sentora-core copied to clipboard

Update controller.ext.php

Open evandone opened this issue 7 years ago • 3 comments

Allow email addresses that include a plus sign for tagging/local filtering.

evandone avatar Jul 07 '17 03:07 evandone

I have the feeling that it would be more efficient and reliable to use:

static function IsValidEmail($email) { return filter_var($email, FILTER_VALIDATE_EMAIL) !== false; }

or even to remove completely the function IsValidEmail($email) and to call each time directly filter_var() instead.

Can someone else more accustomed to filter_var() confirm or deny?

5050 avatar Jul 07 '17 07:07 5050

Agreed, It should be:

static function IsValidEmail($email) {
    return !!filter_var($email, FILTER_VALIDATE_EMAIL)
}

Or keep as Is, there are bunch of the issues with email verification. And caveats in each solution but at least this is de facto one.

Rakhmanov avatar Jan 30 '18 04:01 Rakhmanov

@Dukecitysolutions - was this ever resolved?

TGates71 avatar Feb 01 '24 02:02 TGates71