Skip to content

Prepare for Matomo 6 - #93

Merged
sgiehl merged 10 commits into
6.x-devfrom
prepare6x
Aug 10, 2026
Merged

Prepare for Matomo 6#93
sgiehl merged 10 commits into
6.x-devfrom
prepare6x

Conversation

@sgiehl

@sgiehl sgiehl commented Aug 9, 2026

Copy link
Copy Markdown
Member

Description

Prepares TreemapVisualization for Matomo 6.

  • plugin.json: version 6.0.0, requires matomo >=6.0.0-b1,<7.0.0-b1. The -b1 lower bound matters — a plain >=6.0.0 sorts above 6.0.0-b1 and would disable the plugin against a beta core.
  • CHANGELOG.md: 6.0.0 entry.
  • .github/workflows/matomo-tests.yml: PHP versions resolve through the shared matomo6_min_php / matomo6_max_php aliases instead of hardcoded literals; PluginTests runs a MySQL 8.0 + MariaDB 10.6 matrix; the UI job runs on Node 24 with an explicit MySQL 8.0.
  • phpstan.neon: phpVersion raised to the new PHP 8.1 floor. Matomo 6 bundles PHPStan 2, which fails hard on an excludePaths entry matching nothing, so the vendor/ entry is dropped (the plugin has no composer.json) and github-action-tests/ is marked optional — it only exists inside a CI run.

The Matomo 6 breaking-changes sweep found nothing to change: no removed PHP APIs, HTTP API methods, global functions, changed interfaces, dependency-upgrade fallout, removed Less variables or removed JS/Vue APIs are used. The plugin has no composer.json, no Updates/ and no vue/src, so those steps do not apply.

Issue No

Related to the Matomo 6 plugin preparation work.

Steps to Replicate the Issue

  1. Install the plugin on a Matomo 6 checkout.
  2. Expected: the plugin activates and the Treemap visualisation renders.
  3. Actual (before this PR): plugin.json capped Matomo at <6.0.0-b1, so the plugin was not loadable.

Checklist

  • [✔] Tested locally or on demo2/demo3?
  • [NA] New test case added/updated?
  • [NA] Are all newly added texts included via translation?
  • [NA] Are text sanitized properly? (Eg use of v-text v/s v-html for vue)
  • [✔] Version bumped?
  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules
  • [NA] Documentation updated?

AI Checklist

  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules

sgiehl and others added 9 commits August 9, 2026 16:06
Bump plugin version to 6.0.0 and require Matomo >=6.0.0-b1,<7.0.0-b1.
* Run plugin tests against MySQL 8.0 and MariaDB 10.6

The workflow inherited the shared action's MySQL 5.7 default, which is below
Matomo's new minimum. Pin the database-backed jobs to the supported floor:
PluginTests now runs a MySQL 8.0 + MariaDB 10.6 matrix, and the UI/JS jobs run
on MySQL 8.0.

* Only upload plugin test artifacts for the MySQL matrix leg

PluginTests runs a MySQL + MariaDB matrix; the upload-artifacts condition
matched both legs and uploaded twice. Restrict the upload to the MySQL leg.
… Puppeteer 24) (#89)

* Update expected UI screenshots for the new headless Chrome (Node 24 / Puppeteer 24)

* Run the plugin UI tests on Node 24

Match the Node 24 / Puppeteer 24 screenshot-testing stack; the UI job was still pinned to Node 16.
Teal brand screenshots on 6.x-dev plugin base; synced from matomo-org/matomo run 30405694397.
The DataTable search moved from the table footer into the shared ReportHeader
(matomo-org/matomo#24986), which changes every report screenshot.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The workflow pinned 8.1 / 8.5 as literals, which drift as soon as core
changes its supported range. Resolve them through the central action instead.
Raise phpVersion to the new PHP 8.1 floor. PHPStan 2 fails hard on an
excludePaths entry that matches nothing, so drop the vendor/ entry (the plugin
has no composer.json) and mark github-action-tests/ optional — it only exists
inside a CI run.
@sgiehl
sgiehl requested a review from a team August 9, 2026 14:11
…release]

Core now renders the report search as an icon in the shared ReportHeader
instead of an expanded search box, which removes a header row and shifts the
whole treemap up.
@sgiehl
sgiehl merged commit 947ad91 into 6.x-dev Aug 10, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants