Skip to content
This repository has been archived by the owner on Jan 21, 2020. It is now read-only.

pass field type to $queryBuilder->setParameter() #293

Merged
merged 2 commits into from
Jan 18, 2018
Merged

pass field type to $queryBuilder->setParameter() #293

merged 2 commits into from
Jan 18, 2018

Conversation

gegok
Copy link
Contributor

@gegok gegok commented Mar 29, 2017

If you use custom type for identifier (e.g. uuid_binary https://github.com/ramsey/uuid-doctrine) single entity won't be found by id

@coveralls
Copy link

Coverage Status

Coverage remained the same at 51.143% when pulling 2b72469 on AdMeGroup:specify-field-type into 689a9aa on zfcampus:master.

@gegok
Copy link
Contributor Author

gegok commented Apr 3, 2017

@webimpress

@michalbundyra
Copy link
Member

@gegok Can you add please test case for it? Thanks!

@michalbundyra michalbundyra added this to the 2.1.1 milestone Jan 18, 2018
@michalbundyra michalbundyra self-assigned this Jan 18, 2018
@michalbundyra michalbundyra merged commit 8cc7419 into zfcampus:master Jan 18, 2018
michalbundyra added a commit that referenced this pull request Jan 18, 2018
pass field type to $queryBuilder->setParameter()

# Conflicts:
#	test/src/Server/ORM/CRUD/CRUDTest.php
michalbundyra added a commit that referenced this pull request Jan 18, 2018
michalbundyra added a commit that referenced this pull request Jan 18, 2018
michalbundyra added a commit that referenced this pull request Jan 18, 2018
@michalbundyra
Copy link
Member

Thanks @gegok !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants