8000 Fix the reuse of a statement on sqlsrv with explicit bindValue() by morozov · Pull Request #2646 · doctrine/dbal · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix the reuse of a statement on sqlsrv with explicit bindValue() #2646

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 8, 2017

Conversation

morozov
Copy link
Member
@morozov morozov commented Feb 8, 2017

If a value is bound with bindValue() between statement executions, the value used on a subsequent execution is ignored. The same happens with bindParam(). See the tests.

@morozov
Copy link
Member Author
morozov commented Feb 8, 2017

The proposed solution is more of a POC as it duplicates code and doesn't solve the second issue. I believe the proper one will not have these downsides. Fixed the 2nd issue, however some code is still duplicated.

@morozov morozov force-pushed the sqlsrv-reuse-rebind branch from 8b8b55e to d6a7079 Compare February 8, 2017 00:25
@deeky666
Copy link
Member
deeky666 commented Feb 8, 2017

@morozov would you be interested in joining the Doctrine core team? You did some very good quality additions to DBAL lately and we are still looking for support here :)

@morozov
Copy link
Member Author
morozov commented Feb 8, 2017

@deeky666 it'd be my honor. Hopefully, my further contributions will not be that damaging :-).

@deeky666
Copy link
Member
deeky666 commented Feb 8, 2017

@morozov hehe no worries, we're all humans ;) Can you join on IRC #doctrine-dev (freenode network) and throw a private message at me? I will tell you more then.

@Ocramius
Copy link
Member
Ocramius commented Feb 8, 2017

@morozov welcome to the team! Meanwhile, this is being merged :-)

@Ocramius Ocramius self-assigned this Feb 8, 2017
@Ocramius Ocramius added this to the 2.5.12 milestone Feb 8, 2017
@Ocramius Ocramius merged commit 319466f into doctrine:master Feb 8, 2017
Ocramius added a commit that referenced this pull request Feb 8, 2017
…er-subsequent-calls' into 2.5

Backport #2646 to 2.5.x
@morozov morozov deleted the sqlsrv-reuse-rebind branch May 17, 2017 01:11
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 18, 2022
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
0