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
Call to a member function on a non-object #1450
Comments
sounds like a bug in the modx plugin specifically? |
Also reported in http://forum.piwik.org/index.php?showtopic=11541 |
This is not invalid, I am having the same error and do not run modx. v0.5.4 was fine. Upgraded to 0.7.0 and this error started for some sites when trying to load a widget with visits summary & graph (VisitsSummary Index). Fatal error: Call to a member function getTimezone() on a non-object in /san/home/piwik0.7.0/core/Controller.php on line 212 |
DriverDan: can you try 0.8? There was no feedback on the Piwik forum from the original poster. On the modx forum, the OP says: Are your errors and configuration similar? |
We're running this across many live sites and have a lot of ones that are now inactive (>50%). I cannot delete them because we save the data for reference. I will try to upgrade to 0.8.0 as soon as I have time. I've made a lot of template changes and due to the way Piwik organizes its views/templates upgrades are very time consuming. |
Maybe it would be a good thing to work on theming for Piwik. see #5597 |
Please reopen if the issue is still here in 0.8, with steps to reproduce. |
Still happening in 0.9: This is trying to load the Visits summary overview with graph widget using token_auth. If I log into the account everything in the dashboard works. If I log out and try accessing the widget with token_auth I get the error. |
I haven't replicated yet, but leaving open as reminder |
(In [2982]) Fixes #1450
|
DriverDan, can you please try the patch and see if it fixes the issue for you? |
I finally had an opportunity to upgrade to 1.0.0. The previous error has been replaced by a new one. Looks like the Exception.php file in libs/HTML/QuickForm2/ isn't being included. When I changed it to a regular exception I got "Renderer type 'smarty' is already registered". |
(In [3040]) refs #1450 - some of the exception classes don't follow PEAR naming convention; use explicit require_once |
Thanks for fixing that issue. This ticket should still be open though, the "Renderer type 'smarty' is already registered" error isn't going to be fixed by including the exceptions. Here's a backtrace when the exception is thrown:
|
In [3041], fixes comment:ticket:1450:17 - redirect to Login when user doesn't have view access to that site's stats |
Applying this patch fixes the exception issues but shows a login page when trying to access widgets using the API key for the account I was using. Using the API key of a newly created account works as it should. It seems something in the database related to accounts and permissions wasn't updated on an upgrade (unknown version) which was causing my issue. I deleted the previous user and replaced it with a new one to solve the problem. |
Next time you encounter this, please check that the user has view/admin access to that site. When a site is added, existing users (except the superuser) have no access to the new site. |
Thanks for the help. I'm well aware of how permissions work. I tried removing access for the account, changing it to view or admin and auth key access still didn't work. This is probably an edge case. I consider this closed and not worth any more time unless other people complain. |
DriverDan, you suggest there is still a bug on "edge case" - can you please explain the steps to reproduce? |
Unfortunately I can't. I'm pretty certain it was caused by a database change missing from an upgrade at some point, but I've been running this installation of Piwik for almost 3 years now so I have no idea which update caused the problem. Unless a lot of other people complain I wouldn't worry about it. |
See: http://modxcms.com/forums/index.php?topic=32891.msg295437#msg295437
The text was updated successfully, but these errors were encountered: