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

Skip to content

[DoctrineBridge] Fixed validating custom doctrine type columns #21656

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
Feb 21, 2017

Conversation

dmaicher
Copy link
Contributor

@dmaicher dmaicher commented Feb 18, 2017

Q A
Branch? 3.1
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #21619
License MIT
Doc PR -

This fixes #21619 by not assuming the invalid $value is a Doctrine entity if its an object

@xabbuh
Copy link
Member

xabbuh commented Feb 20, 2017

Symfony 3.1 doesn't receive bug fixes any more. Can you rebase on the 3.2 branch and change the target branch accordingly?

@dmaicher
Copy link
Contributor Author

Ah sorry you are right. Will rebase it later today 😉

@dmaicher dmaicher force-pushed the fix-validation-custom-type branch from c642345 to 70d962c Compare February 20, 2017 10:37
@dmaicher dmaicher changed the base branch from 3.1 to 3.2 February 20, 2017 10:37
@dmaicher dmaicher force-pushed the fix-validation-custom-type branch from 70d962c to ad59370 Compare February 20, 2017 10:38
@dmaicher
Copy link
Contributor Author

@xabbuh done 😉

Copy link
Member

@xabbuh xabbuh left a comment

Choose a reason for hiding this comment

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

👍 We could use the class constants instead of strings, but that's not a blocker IMO.

Status: Reviewed

@fabpot
Copy link
Member

fabpot commented Feb 21, 2017

Thank you @dmaicher.

@fabpot fabpot merged commit ad59370 into symfony:3.2 Feb 21, 2017
fabpot added a commit that referenced this pull request Feb 21, 2017
…umns (dmaicher)

This PR was merged into the 3.2 branch.

Discussion
----------

[DoctrineBridge] Fixed validating custom doctrine type columns

| Q             | A
| ------------- | ---
| Branch?       | 3.1
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #21619
| License       | MIT
| Doc PR        | -

This fixes #21619 by not assuming the invalid `$value` is a Doctrine entity if its an object

Commits
-------

ad59370 [DoctrineBridge] Fixed validating custom doctrine type columns
@fabpot fabpot mentioned this pull request Mar 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants