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

PHP Fatal error: Call to a member function toString() on a non-object in /home/www/piwik/plugins/CoreHome/DataTableRowAction/RowEvolution.php on line 90 #3363

Closed
anonymous-matomo-user opened this issue Sep 6, 2012 · 5 comments
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc. duplicate For issues that already existed in our issue tracker and were reported previously. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Milestone

Comments

@anonymous-matomo-user
Copy link

each time we try to output an evolution graph we got an error on the top most windows and the div in popover stay on the front for ever ...

The error in the php logs is : PHP Fatal error: Call to a member function toString() on a non-object in /home/www/piwik/plugins/CoreHome/DataTableRowAction/RowEvolution.php on line 90

we use piwik 1.8.3 on a dedicated VM with UBUNTU 10.04.LTS and PHP 5.3.2-1ubuntu4.17 with Suhosin-Patch (standard for this ubuntu).

I haven't seen any ticket on this, even if I am sure we are not the only one concerned ...

Thanks for your help !

Stphane
Keywords: error php non-object RowEvolution.php

@mattab
Copy link
Member

mattab commented Sep 7, 2012

Thanks for the report. Do you see the same problem on demo.piwik.org or only in your code?

can you please paste in pastebin.com the full error message with backtrace? thanks!

@anonymous-matomo-user
Copy link
Author

Hi Matt,

Replying to matt:

I am using piwik 1.8.3 not 1.8.4 (I just use the automatic update link when it show off).

Thanks for the report. Do you see the same problem on demo.piwik.org or only in your code?

no, it's working fine on demo.

can you please paste in pastebin.com the full error message with backtrace? thanks!

I uploaded the screen shot : http://demo.ovh.com/fr/94f5f2783510ab33a35570c49fd52990/
and the php-error.log : http://pastebin.com/NtUcyugE

I don't know what you mean by backtrace, there is nothing more here ...

I took a look at the source code and the error match the line :

$end = $date->toString();

and date is a parameter of the function call :

        /**
     * The constructor
     * Initialize some local variables from the request
     * @param int $idSite
     * @param Piwik_Date $date ($this->date from controller)
     * @throws Exception
     */
    public function __construct($idSite, $date)
    {

as we are using it in French, maybe it's a conversion date problem.

on the side of the error, I think that you should try to show the error (oops) INSIDE the popup, when the http call is coming from it, like this the popup don't stay in front waiting for nothing except for the user to close it

Thanks for your help AND good work (I love that software).

Stphane

@anonymous-matomo-user
Copy link
Author

In fact, I did solved the problem by extension the max execution time variable in the php.ini file.

max_execution_time = 120
to
max_execution_time = 240

but this mean that I take way too long to make the calculation ... I will try to add cpu to the VM

Thanks for your help !

Stphane

@diosmosis
Copy link
Member

I can't reproduce this. Stphane, can you tell me what date, period & report you were viewing (the screenshot is no longer available)? Thanks!

@mattab
Copy link
Member

mattab commented Oct 22, 2012

We will work on Row Evolution performance as part of: #3464

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. duplicate For issues that already existed in our issue tracker and were reported previously. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Projects
None yet
Development

No branches or pull requests

3 participants