-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Added mysqli driver. #72
Conversation
@@ -0,0 +1,56 @@ | |||
<?php | |||
/* | |||
* $Id$ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this line is a leftt-over from the SVN legacy. You should not include it in new files.
$values = $this->_fetch(); | ||
if (null === $values) { | ||
return null; | ||
} elseif (false === $values) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you should use if
instead of elseif
here. and you should probably use $this->fetch(\PDO::FETCH_NUM)
instead of $this->_fetch()
to avoid duplicating the logic creating the exception
…. Added MysqlConnection::getWrappedResourceHandle(). MysqliStatement::fetchAll() now decide fetch style if none is set, instead of letting fetch() do it on every call.
… exception if a error had a occured.
Completes Dbal and ORM tests.
Some primitive "time" cmd of tests is slower then pdo_mysql, all suggestions are welcome :)