Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Validator] Test that validateProperty() works if no constraint is defined #11707

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.

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

Merged
merged 1 commit into from
Aug 31, 2014

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Aug 19, 2014

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes (2.3 has to be merged into 2.5 first)
Fixed tickets #11604, #11614
License MIT
Doc PR

Adds a test case for #11604 to avoid regressions. The actual issue has been fixed in Symfony 2.3 with the merge of #11615.

Adds a test case for symfony#11604 to avoid regressions. The actual issue
has been fixed in Symfony 2.3 with the merge of symfony#11615.
@webmozart webmozart changed the title [Validator] add test for #11604 [Validator] Test that validateProperty() works if no constraint is defined Aug 20, 2014
@fabpot
Copy link
Member

fabpot commented Aug 28, 2014

👍

1 similar comment
@stof
Copy link
Member

stof commented Aug 28, 2014

👍

@fabpot
Copy link
Member

fabpot commented Aug 31, 2014

Thank you @xabbuh.

@fabpot fabpot merged commit a47a884 into symfony:2.5 Aug 31, 2014
fabpot added a commit that referenced this pull request Aug 31, 2014
…straint is defined (xabbuh)

This PR was merged into the 2.5 branch.

Discussion
----------

[Validator] Test that validateProperty() works if no constraint is defined

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes (2.3 has to be merged into 2.5 first)
| Fixed tickets | #11604, #11614
| License       | MIT
| Doc PR        |

Adds a test case for #11604 to avoid regressions. The actual issue has been fixed in Symfony 2.3 with the merge of #11615.

Commits
-------

a47a884 add test for #11604
@xabbuh xabbuh deleted the issue-11604 branch August 31, 2014 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants