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
Incompatible method declaration in Piwik_Log_APICall #821
Comments
Attachment: |
Hi, I just wrote a detailed bug report concerning my reopening of the bug, but when I tried to add the attachment, Trac unfortunately killed the text content. Sorry for the inconvenience. Here it is again: The method declaration mismatch still occurs, as you can see, in two lines of the piwik code:
and
Changing the method signatures would be a quick and working fix for that. See the attached file logger.patch. Anyhow, I did some research regarding the PHP version issue and found this bug addressing an inconsistency issue between the PHP's documentation and its actual behaviour when using set_error_handler. PHP now always executes the custom error handler function regardless of any error_reporting() settings. Because of that, and because of the two errors mentioned above, piwik's error handler now fails (see ErrorHandler.php, line 37) and as a result of that, it blindly exits the application. A fix for that could be to ignore non-fatal errors such as E_NOTICE, E_USER_NOTICE and E_STRICT before exiting (see attached file errorhandler.patch), but I suggest that you do some detailed testing on that and PHP 5.2.10 in general. :-) Benjamin |
Attachment: |
if it is a bug in PHP then we don't want to work around this, especially since the bug appears to have been fixed in PHP CVS. |
(In [1249]) fixes #821 - rename the log() method in subclasses of Piwik_Log because ref: http://ca.php.net/manual/en/language.oop5.overloading.php this change arises because PHP 5.2.10 escalated the log message |
(In [1250]) refs #821 - revised patch |
Hi,
I just upgraded to PHP 5.2.10 (from 5.2.9) and now I get the following error accessing piwik:
Here's the stack trace:
Benjamin
The text was updated successfully, but these errors were encountered: