Add mysqli exception mode#349
Conversation
|
thanks, maybe we can remove the warning suppression (@) now too? other question is can we get the state before we set it and set it back after connecting? It also seems that |
Thanks for pointing this out. I removed the error suppression.
Yes, that would be true. Hence, why the current code needs to be changed because it doesn't work as intended. I still like to set the error mode explicitly though. |
richard67
left a comment
There was a problem hiding this comment.
Code review ok, and I've also done some tests on 5.4-dev of the CMS.
|
Thanks @kamil-tekiela for this fix, and @HLeithner and @rdeutz for reviews. |
This PR changes the
MysqliDriverto use exception mode just like PDO does. I tried not to make too many changes and stick to the previous style. There was some dead code inexecuteUnpreparedQuerywhich I removed to make the change easier.