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

Skip to content

Update ORMQueryBuilderLoader.php #4

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

Closed
wants to merge 1 commit into from
Closed

Conversation

sasedev
Copy link

@sasedev sasedev commented Dec 25, 2015

Type::GUID can be used as entity ID and is Connection::PARAM_STR_ARRAY

when submitting form with required set to FALSE and empty entity with Type::GUID as entity ID,
we still receive an SQLSTATE[22P02]: Invalid text representation: 7 ERROR: invalid input syntax for uuid: ""
in the PostgreSQL Platform for example.

symfony/symfony#14465 ???

Type::GUID can be used as entity ID and is Connection::PARAM_STR_ARRAY

when submitting form with required set to FALSE and empty entity with Type::GUID as entity ID,
we still receive an SQLSTATE[22P02]: Invalid text representation: 7 ERROR:  invalid input syntax for uuid: ""
in the PostgreSQL Platform for example.

symfony/symfony#14465 ???
@zhukovra
Copy link

@sasedev it is read-only repository. You should add tests to your fix and create new PR in https://github.com/symfony/symfony

@nicolas-grekas
Copy link
Member

Thanks for this PR. Yet I invite you to submit it on the main symfony/symfony repository: this is where merges happen. Note that you should add test cases while doing so. Cheers.

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