Bläddra i källkod

[Validator] fixed possible warning

Fabien Potencier 14 år sedan
förälder
incheckning
d2525c9492
1 ändrade filer med 11 tillägg och 8 borttagningar
  1. 11 8
      src/Symfony/Component/Validator/Constraints/EmailValidator.php

+ 11 - 8
src/Symfony/Component/Validator/Constraints/EmailValidator.php

@@ -30,15 +30,18 @@ class EmailValidator extends ConstraintValidator
         $value = (string) $value;
         $valid = filter_var($value, FILTER_VALIDATE_EMAIL);
 
-        $host = substr($value, strpos($value, '@') + 1);
-        if ($valid && version_compare(PHP_VERSION, '5.3.3', '<') && strpos($host, '.') === false) {
-            // Likely not a FQDN, bug in PHP FILTER_VALIDATE_EMAIL prior to PHP 5.3.3
-            $valid = false;
-        }
+        if ($valid) {
+            $host = substr($value, strpos($value, '@') + 1);
+
+            if (version_compare(PHP_VERSION, '5.3.3', '<') && strpos($host, '.') === false) {
+                // Likely not a FQDN, bug in PHP FILTER_VALIDATE_EMAIL prior to PHP 5.3.3
+                $valid = false;
+            }
 
-        // Check MX records
-        if ($valid && $constraint->checkMX) {
-            $valid = $this->checkMX($host);
+            // Check MX records
+            if ($valid && $constraint->checkMX) {
+                $valid = $this->checkMX($host);
+            }
         }
 
         if (!$valid) {