8000 [Validator][Email] - Strict validation and soft dependency by egulias · Pull Request #9140 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[Validator][Email] - Strict validation and soft dependency #9140

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 16 commits into from
Closed

[Validator][Email] - Strict validation and soft dependency #9140

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
885996b
#1581 - Strict in Email constraint and use of Egulias\EmailValidator
egulias Sep 19, 2013
b377295
#1581 - strict_emal configuration value (rebase)
egulias Jan 12, 2014
c3241ec
#1581 - strict_emal config into Email constraint configuration
egulias Sep 26, 2013
ab3787a
#1581 - Soft dependency with version
egulias Sep 27, 2013
fb36789
#1581 - EmailValidator hanldes global strict configuration
egulias Oct 13, 2013
6574d5f
#1581 - CS & false as default
egulias Nov 18, 2013
1052f3b
#1581 - Recover MX and Host validation
egulias Nov 18, 2013
33cb5c5
#1581 - Comment removed
egulias Nov 18, 2013
facef8e
#1581 - Test updated for configuration and validator
egulias Nov 22, 2013
7ce2528
#1581 - CS
egulias Jan 12, 2014
58e99b5
#1581 - BC break documentation and version update
egulias Jan 12, 2014
fadea1f
#1581 - composer list missing comma
egulias Jan 13, 2014
5791e36
#1581 - CS
egulias Jan 27, 2014
a7ec6ab
#1581 - Validator component changelog
egulias Jan 28, 2014
89ca713
8000 Deaults strict to false
egulias Mar 20, 2014
cf11a03
#1581 - Removed container parameter
egulias Mar 26, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Deaults strict to false
  • Loading branch information
@egulias
egulias committed Mar 20, 2014
commit 89ca7137845748e9860b7f14bcf294d82d1464a3
6 changes: 3 additions & 3 deletions src/Symfony/Component/Validator/Constraints/EmailValidator.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ class EmailValidator extends ConstraintValidator
*/
private $isStrict;

public function __construct($strict)
public function __construct($strict = false)
{
$this->isStrict = $strict;
}
Expand All @@ -49,11 +49,11 @@ public function validate($value, Constraint $constraint)
}

$value = (string) $value;
if ($constraint->strict === null) {
if (null === $constraint->strict) {
$constraint->strict = $this->isStrict;
}

if (true === $constraint->strict && class_exists('\Egulias\EmailValidator\EmailValidator')) {
if ($constraint->strict && class_exists('\Egulias\EmailValidator\EmailValidator')) {
$strictValidator = new StrictEmailValidator();
$valid = $strictValidator->isValid($value, $constraint->checkMX);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should probably disable MX checks here as it's going to be redone below anyway.

} elseif ($constraint->strict === true) {
Expand Down
0