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] Tracking Code requires Overlay plugin to be enabled #21891

Closed
4 tasks done
mikkeschiren opened this issue Feb 7, 2024 · 2 comments · Fixed by #21896
Closed
4 tasks done

[Bug] Tracking Code requires Overlay plugin to be enabled #21891

mikkeschiren opened this issue Feb 7, 2024 · 2 comments · Fixed by #21896
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@mikkeschiren
Copy link
Contributor

What happened?

With Overlay plugin deactivated, and you go to

Measureables -> Tracking Code

You get a warning:

The plugin Overlay is not enabled. You can activate the plugin on Settings > Plugins page in Matomo.

It also makes the area to get the Tracking Code for JavaScript non working (empty).

What should happen?

You should get the Tracking Code, and no warning.

How can this be reproduced?

Deactivate Overlay plugin if it is enabled.

Then you go to:
Measureables -> Tracking Code

You get the warning and nothing for the Tracking Code.

Matomo version

5.0.2

PHP version

No response

Server operating system

No response

What browsers are you seeing the problem on?

No response

Computer operating system

No response

Relevant log output

Uncaught exception in API: /var/www/html/core/Plugin/Manager.php(303): The plugin Overlay is not enabled. You can activate the plugin on Settings > Plugins page in Matomo. [Query: ?date=yesterday&module=API&format=json&method=Overlay.getExcludedQueryParameters&idSite=1&filter_limit=-1&segment=&period=day, CLI mode: 0]

Validations

@mikkeschiren mikkeschiren 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 Feb 7, 2024
@mikkeschiren
Copy link
Contributor Author

mikkeschiren commented Feb 7, 2024

Suggested solution would be to add public function getExcludedQueryParameters($idSite) to CoreAdminCore, that would cause code duplication, but it also makes core functionality work without the need of a none required plugin, and will not break anything. I can create a pull request for this.

@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 Feb 8, 2024
@sgiehl sgiehl added this to the 5.1.0 milestone Feb 8, 2024
@sgiehl
Copy link
Member

sgiehl commented Feb 8, 2024

Thanks for the report. This indeed shouldn't be that case. I'll prepare a fix for that.

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