Validators #8577

Merged
merged 1 commit into from Aug 8, 2013

3 participants

@EmmanuelVella
Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
License MIT

This PR avoid comparison validators to be executed if the compared value is null.

@stof
Symfony member

@fabpot 👍

@fabpot fabpot added a commit that referenced this pull request Aug 8, 2013
@fabpot fabpot merged branch EmmanuelVella/validators (PR #8577)
This PR was merged into the 2.3 branch.

Discussion
----------

Validators

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| License       | MIT

This PR avoid comparison validators to be executed if the compared value is null.

Commits
-------

48338fc Ignore null value in comparison validators
ffe5567
@fabpot fabpot merged commit 48338fc into symfony:2.3 Aug 8, 2013

1 check passed

Details default The Travis CI build passed
@EmmanuelVella EmmanuelVella deleted the EmmanuelVella:validators branch Aug 8, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment