Fix: Email Validator vs IDN

[svyatoslav-kubakh]

Non-latin emails validation

[moderndeveloperllc]

I don't think you can use idn_to_ascii() in this class as Zend\Validator does not require the intl extension to be present. If you want to validate non-Latin emails, it might be better to write an Zend\I18n validator that uses that function. Thoughts @DASPRiD?

[weierophinney]

Two potential options:

• Have it use idn_to_ascii, but only if ext/intl is installed. This could be achieved via a method call:

$this->getLocalizedLocalPart();

private function getLocalizedLocalPart()
{
    if (extension_loaded('intl')) {
        return idn_to_ascii($this->localPart);
    }
    return $this->localPart;
}

• or move it into the Zend\I18n\Validator namespace.

Your pick. If you can do this by 9 March 2015, I can include it in 2.4.

[moderndeveloperllc]

@svyatoslav-kubakh I can do the change and move if you don't want to deal with it.

[svyatoslav-kubakh]

Hello, everyone. I apologize for my absence. Thank you for your help and advice. I will make a second attempt to fix within 1-2 days. If the patch is rejected again, you give in turn. :-)

[moderndeveloperllc]

@svyatoslav-kubakh We're your fixes made on another PR, or have you decided not to fix this?

[svyatoslav-kubakh]

That's correct. #7281