@anonymous-matomo-user opened this Issue on September 6th 2012

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 commented on September 7th 2012 Member

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 commented on September 7th 2012

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
     * <a class='mention' href='https://github.com/param'>@param</a> int $idSite
     * <a class='mention' href='https://github.com/param'>@param</a> Piwik_Date $date ($this->date from controller)
     * <a class='mention' href='https://github.com/throws'>@throws</a> 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 commented on September 7th 2012

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 commented on October 19th 2012 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 commented on October 22nd 2012 Member

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

This Issue was closed on October 22nd 2012
Powered by GitHub Issue Mirror