matomo-org / matomo

Empowering People Ethically with the leading open source alternative to Google Analytics that gives you full control over your data. Matomo lets you easily collect data from websites & apps and visualise this data and extract insights. Privacy is built-in. Liberating Web Analytics. Star us on Github? +1. And we love Pull Requests!

Home Page:https://matomo.org/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[Bug] Charts for Conversions by Entry Pages & Page Titles Broken

9joshua opened this issue · comments

What happened?

The 'Visits' chart plot does not have a title and once another chart plot type is selected, the visits type cannot be reselcted.
image

Entry Page conversion charts are partly broken for both:

  • Conversions -> Overview (or specific Goal) -> Goals by Pages
  • Ecommerce -> Sales -> Sales by Pages
    Ecommerce report - entry page ev chart
WARNING: /plugins/API/RowEvolution.php(67): Warning - Undefined array key "nb_visits" - Matomo 5.1.0-b3 - Please report this message in the Matomo forums: https://forum.matomo.org (please do a search first as it might have been reported already) (Module: CoreHome, Action: getRowEvolutionPopover, Method: API.getRowEvolution, In CLI mode: false)

What should happen?

The chart plot category 'Visits' should function properly.

How can this be reproduced?

View a evolution chart for Ecommerce -> Sales -> Sales by Pages -> Entry Pages -> Click the chart button for any page URL

Matomo version

5.1.0-b3 (Cloud)

PHP version

8.1

Server operating system

Linux

What browsers are you seeing the problem on?

Chrome

Computer operating system

Ubuntu

Relevant log output

WARNING: /plugins/API/RowEvolution.php(67): Warning - Undefined array key "nb_visits" - Matomo 5.1.0-b3 - Please report this message in the Matomo forums: https://forum.matomo.org (please do a search first as it might have been reported already) (Module: CoreHome, Action: getRowEvolutionPopover, Method: API.getRowEvolution, In CLI mode: false)

Browser console error when clicking the 'Visits' chart plot type:

index.php?module=Proxy&action=getCoreJs&cb=b8dc07360884fb8559939d48d1bf5db7:774 Uncaught TypeError: Cannot read properties of undefined (reading 'tickOptions')
    at exports.JqplotEvolutionGraphDataTable.setYTicksForAxis (index.php?module=Proxy&action=getCoreJs&cb=b8dc07360884fb8559939d48d1bf5db7:774:9)
    at exports.JqplotEvolutionGraphDataTable.setYTicks (index.php?module=Proxy&action=getCoreJs&cb=b8dc07360884fb8559939d48d1bf5db7:768:655)
    at RowEvolutionSeriesToggle.replot (index.php?module=Proxy&action=getCoreJs&cb=b8dc07360884fb8559939d48d1bf5db7:787:99)
    at RowEvolutionSeriesToggle.showSeries (index.php?module=Proxy&action=getCoreJs&cb=b8dc07360884fb8559939d48d1bf5db7:782:588)
    at HTMLTableRowElement.<anonymous> (index.php?module=Proxy&action=getCoreJs&cb=b8dc07360884fb8559939d48d1bf5db7:790:243)
    at HTMLTableRowElement.dispatch (index.php?module=Proxy&action=getCoreJs&cb=b8dc07360884fb8559939d48d1bf5db7:4:43184)
    at y.handle (index.php?module=Proxy&action=getCoreJs&cb=b8dc07360884fb8559939d48d1bf5db7:4:41168)

Validations

Thanks for the detailed report @9joshua
I was able to reproduce that one, and directly found a proper fix for it. Will prepare a pull request, so that gets fixed in the next release.