Skip to content
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

missing error when a prepared statement with placeholders is executed with an empty bind #259

Open
druud opened this issue Aug 30, 2018 · 5 comments

Comments

@druud
Copy link

druud commented Aug 30, 2018

See: https://rt.cpan.org/Public/Bug/Display.html?id=125517

@pali
Copy link
Member

pali commented Aug 30, 2018

Hi! This problem should be already fixed in DBD::MariaDB 1.00 which is on cpan:
https://metacpan.org/pod/DBD::MariaDB

@druud
Copy link
Author

druud commented Aug 30, 2018

Hi pali, how does that apply to DBD::mysql?

@pali
Copy link
Member

pali commented Aug 30, 2018

I guess this cannot be fixed in DBD::mysql due to legacy issues and existing code which depends on this behavior which you descried in the title of the issue (see #117). Therefore if you does not like current behavior of DBD::mysql like you described, you can just switch to the DBD::MariaDB fork.

@burak
Copy link

burak commented Aug 30, 2018

@pali well this is not a complete behavior change and rather a stability issue to report misuse of placeholders. It can still be silent and can be made noisy with a flag, it can even be something to be enabled while building the module, so unless a switch is passed to Makefile.PL to decide between silent / emit warning (even die).

@CaptTofu
Copy link
Member

Burak - thank you. We are working on a fix for this for the next release of DBD::mysql

@dveeden dveeden added this to the 4.051 milestone Jan 15, 2019
@dveeden dveeden modified the milestones: 4.051, 4.052 Oct 4, 2023
@dveeden dveeden modified the milestones: 4.052, 4.053 Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants