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

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

Closed
4 tasks done
9joshua opened this issue May 19, 2024 · 1 comment · Fixed by #22239
Closed
4 tasks done

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

9joshua opened this issue May 19, 2024 · 1 comment · Fixed by #22239
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@9joshua
Copy link
Contributor

9joshua commented May 19, 2024

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

@9joshua 9joshua added Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. To Triage An issue awaiting triage by a Matomo core team member labels May 19, 2024
@sgiehl sgiehl added Bug For errors / faults / flaws / inconsistencies etc. and removed Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. To Triage An issue awaiting triage by a Matomo core team member labels May 21, 2024
@sgiehl
Copy link
Member

sgiehl commented May 21, 2024

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.

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.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants