Navigation Menu

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

Option::set() always fail with mysqli #2011

Closed
anonymous-matomo-user opened this issue Jan 12, 2011 · 6 comments
Closed

Option::set() always fail with mysqli #2011

anonymous-matomo-user opened this issue Jan 12, 2011 · 6 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc. worksforme The issue cannot be reproduced and things work as intended.
Milestone

Comments

@anonymous-matomo-user
Copy link

core/Option.php
public function set($name, $value, $autoload = 0)
{
$autoload = (int)$autoload;

libs/Zend/Db/Statement/Mysqli.php

public function _execute(array $params = null)
......
......
if ($params) {
array_unshift($params, str_repeat('s', count($params)));


The problem is $autoload is always int and the types passed to bind_param() is "ssss".

@robocoder
Copy link
Contributor

Is that trunk or 1.1.1?

@robocoder
Copy link
Contributor

@robocoder
Copy link
Contributor

What's the error you get?

@robocoder
Copy link
Contributor

Also, what are these set at?

SELECT @@GLOBAL.sql_mode;
SELECT @@SESSION.sql_mode;

@anonymous-matomo-user
Copy link
Author

version is 1.1.1.

And today I found there is no issue actually.

Let me explain:

Yesterday, I came across this problem: http://forum.piwik.org/read.php?2,69793 and http://forum.piwik.org/read.php?2,55771

Then I thought there should be a bug in the mysqli driver wrapper. So I looked into the source of piwik. Then I found something I posted above and I thought it is a invalid time problem.

And also I tried something about mysqli::bind_param() today and found that "s" is ok for an int value.

So I think we can close this ticket. Sorry for the inconvenience. And sorry for the ticket number - #2011 should be used for a better ticket since this year is 2011.

@robocoder
Copy link
Contributor

Thanks. Those are old forum posts. In future, please try to report bugs that you're actually experiencing, can reproduce, or are able to troublehoot/diagnose/fix.

@anonymous-matomo-user anonymous-matomo-user added this to the Piwik 1.2 milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc. worksforme The issue cannot be reproduced and things work as intended.
Projects
None yet
Development

No branches or pull requests

3 participants