Fix for operators @@, @> and <@. #135

Merged
merged 2 commits into from Dec 24, 2013

Projects

None yet

2 participants

@glenebob
Contributor

This PR fixes the query rewrite bug, revealed by PR #116, that would butcher the @@, @> and <@ operators.

@jjchiw, if you would, please test this to make sure it fixes the problems you were having.

-Glen

@glenebob glenebob NpgsqlCommand.AppendCommandReplacingParameterValues() and operators @…
…@, @> and <@.

Fix bug that would turn operators @@, @> and <@ in ::, :>, and <: respectively.
Add tests for said operator butchery.
19876b2
@jjchiw
jjchiw commented Dec 22, 2013

@glenebob

Great, It works with my EF tests....

@glenebob glenebob NpgsqlCommand.AppendCommandReplacingParameterValues()
Remove a couple bits of redundant and ineffective code in the scanner.
79e1ebc
@glenebob glenebob merged commit 22d840c into npgsql:master Dec 24, 2013
@franciscojunior franciscojunior added a commit that referenced this pull request Mar 28, 2014
@franciscojunior franciscojunior Fix #193.
This change breaks support added in #135. This is a temporary fix for
regression with NHibernate use. Glen Parker is working on a permanent fix.
86bd3f5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment