Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Adding failing test for DDC-2214 #693

Merged
merged 1 commit into from Jun 12, 2013

Conversation

Projects
None yet
3 participants
Owner

Ocramius commented Jun 12, 2013

Parameters being bound to an SQL query should have the same type as
the identifier of the objects being bound to the placeholders of a
DQL query - this is currently broken with proxies, as this test
demonstrates.

This is a demonstration of DDC-2214

@ghost ghost assigned Ocramius Jun 12, 2013

Owner

Ocramius commented Jun 12, 2013

Tests pass once rebased on top of #690

Owner

guilhermeblanco commented Jun 12, 2013

@Ocramius merged related PR. can you rebase, so Travis provides some input for us?

@Ocramius Ocramius Adding failing test for DDC-2214
Parameters being bound to an SQL query should have the same type as
the identifier of the objects being bound to the placeholders of a
DQL query - this is currently broken with proxies, as this test
demonstrates.
a1355d0
Owner

Ocramius commented Jun 12, 2013

@guilhermeblanco build fixed

@ghost ghost assigned guilhermeblanco Jun 12, 2013

@guilhermeblanco guilhermeblanco added a commit that referenced this pull request Jun 12, 2013

@guilhermeblanco guilhermeblanco Merge pull request #693 from doctrine/hotfix/DDC-2214
Adding failing test for DDC-2214
0a90279

@guilhermeblanco guilhermeblanco merged commit 0a90279 into master Jun 12, 2013

1 check passed

default The Travis CI build passed
Details

@Ocramius Ocramius deleted the hotfix/DDC-2214 branch Jun 12, 2013

Owner

FabioBatSilva commented Jun 12, 2013

\o/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment