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

Notice Uninitialized string offset in DataTypes.php #25

Closed
marcioAlmada opened this issue Mar 14, 2017 · 2 comments
Closed

Notice Uninitialized string offset in DataTypes.php #25

marcioAlmada opened this issue Mar 14, 2017 · 2 comments

Comments

@marcioAlmada
Copy link
Contributor

marcioAlmada commented Mar 14, 2017

This notice happens when querying insert statements or calling $pool->ping(). Looks like a minor bug as database interactions still works as expected but it's a bummer when using a more strict error handler that transforms all levels of errors into exceptions:

set_error_handler(function($errno, $errstr, $file, $line) {
   throw new \Exception("{$errstr} in {$file}:{$line}", $errno);
});

Error is:

notice Uninitialized string offset: 7 in /.../vendor/amphp/mysql/lib/DataTypes.php on line 202
bwoebi added a commit that referenced this issue Mar 14, 2017
WTF MySQL protocol???
bwoebi added a commit that referenced this issue Mar 14, 2017
@bwoebi
Copy link
Member

bwoebi commented Mar 14, 2017

Should be fixed ;o)

@bwoebi bwoebi closed this as completed Mar 14, 2017
@marcioAlmada
Copy link
Contributor Author

marcioAlmada commented Mar 14, 2017

Ty! Looks fine now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants