@mattab opened this Issue on December 12th 2018 Member

just updated demo.matomo.org to beta4 and now getting heaps of warning like: WARNING: Trying to add two strings in DataTable\Row::sumRowArray: 0 + -58.4% for row # ['nb_visits' => 84389, 'nb_actions' => 302230, 'Actions_nb_pageviews' => 132551, 'Goal_revenue' => 1462.6, 'Goal_nb_conversions' => 5628, 'Goal_0_nb_conversions' => 0, 'Goal_0_revenue' => 0, 'label' => -2, 'visits_evolution' => 0, 'actions_evolution' => 0, 'pageviews_evolution' => 0, 'revenue_evolution' => 0, 'nb_conversions_evolution' => 0, 'orders_evolution' => 0, 'ecommerce_revenue_evolution' => 0, 'nb_pageviews' => 98667, 'revenue' => 307868.6, 'nb_conversions' => 3849, 'orders' => 1557, 'ecommerce_revenue' => 305945.6] [] [idsubtable = ]<br />

can be visible at: https://demo.matomo.org/index.php?module=TagManager&action=manageContainers&idSite=62&period=day&date=yesterday

it looks like this:
screenshot from 2018-12-13 10-23-10

@tsteur commented on December 19th 2018 Member

@diosmosis I've debugged locally and looked into code and then temporarily disabled proxysite on demo and then the error occurred. It can be reproduced on https://demo.matomo.org/index.php?module=MultiSites&action=index&idSite=1&period=day&date=yesterday when ProxySite is enabled. I think we have fixed this issue in https://github.com/matomo-org/matomo/pull/13853 already? The patch is now applied on the demo instance but still seems to occur?

@diosmosis commented on December 19th 2018 Member

@tsteur would need to be applied on demo-proxy and demo, since the target instance would need to generate the totals row. Then demo would see it and not calculate it again. If it's on both, then would have to look some more.

@tsteur commented on December 19th 2018 Member

Just to be sure as I think you mentioned it before in chat. Should I apply the whole Reports Total calculator patch on the demo so the totals row will be returned and won't be processed again?

@tsteur commented on December 20th 2018 Member

Applied the patch and it is now fixed. Can reload https://demo.matomo.org/index.php?module=MultiSites&action=index&idSite=62&period=day&date=yesterday as often as I want but does not appear anymore.

@mattab commented on January 9th 2019 Member

I'm still seeing the warnings on the demo which is running 3.8.0-rc2:

From 2019-01-07 to 2019-01-13
WARNING: Trying to add two strings in DataTable\Row::sumRowArray: 0 + 33% for row # ['label' => -2, 'nb_visits' => 6093, 'nb_hits' => 11605, 'sum_time_spent' => 768459, 'nb_hits_following_search' => 35, 'nb_hits_with_time_generation' => 10819, 'min_time_generation' => 0.003, 'max_time_generation' => 55.676, 'entry_nb_visits' => 1874, 'entry_nb_actions' => 8126, 'entry_sum_visit_length' => 455983, 'entry_bounce_count' => 648, 'exit_nb_visits' => 2212, 'sum_daily_nb_uniq_visitors' => 5323, 'sum_daily_entry_nb_uniq_visitors' => 1748, 'sum_daily_exit_nb_uniq_visitors' => 2059, 'avg_time_on_page' => 0, 'bounce_rate' => 0, 'exit_rate' => 0, 'avg_time_generation' => 0] [] [idsubtable = ]<br />

screenshot_2019-01-09 all websites dashboard - from 2019-01-07 to 2019-01-13 - web analytics reports -

@mattab commented on February 12th 2019 Member

Still seeing the error on demo.matomo.org today, any idea @diosmosis ?

screenshot from 2019-02-12 15-46-56

@diosmosis commented on February 12th 2019 Member

Updating demo to 3.8.1 from a rc, should fix this.

This Issue was closed on February 12th 2019
Powered by GitHub Issue Mirror