Skip to content

[stable5.4] fix(analytics): count answers per question and handle forms without answers - #3876

Merged
Chartman123 merged 2 commits into
stable5.4from
backport/3869/stable5.4
Nov 11, 2026
Merged

Chartman123 merged 2 commits into
stable5.4from
backport/3869/stable5.4

Conversation

@backportbot

@backportbot backportbot Bot commented Nov 11, 2026

Copy link
Copy Markdown

Backport of PR #3869

The data of a form without answers was never initialized, so sorting it
failed. Answers of deleted questions were counted for a question without
text.

Assisted-by: Claude Code:claude-opus-5-5
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
The Analytics data source counted answers under the key "question
text|answer text" and split that key at "|" again. A "|" in a question
title or an answer cut the texts apart, and questions with the same
title shared one count.

Count the answers per question id and answer text instead.

Assisted-by: Claude Code:claude-opus-5-5
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
@backportbot backportbot Bot added bug Something isn't working 3. to review Waiting for reviews labels Nov 11, 2026
@codecov

codecov Bot commented Nov 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
lib/Analytics/AnalyticsDatasource.php 0.00% 7 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Chartman123
Chartman123 merged commit bf11d61 into stable5.4 Nov 11, 2026
50 of 51 checks passed
@Chartman123
Chartman123 deleted the backport/3869/stable5.4 branch November 11, 2026 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants