Skip to content

fix: remove submissionId from initial state - #3751

Merged
Chartman123 merged 1 commit into
mainfrom
fix/stale-submission-id
Oct 12, 2026
Merged

Chartman123 merged 1 commit into
mainfrom
fix/stale-submission-id

Conversation

@Chartman123

Copy link
Copy Markdown
Collaborator

This fixes a stale submission id causing to load submission data in submit view even if you wouldn't expect it.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (N/A)

@Chartman123 Chartman123 added bug Something isn't working 3. to review Waiting for reviews labels Oct 11, 2026
@Chartman123
Chartman123 requested a review from Koc October 11, 2026 20:54
@Chartman123
Chartman123 requested review from samin-z and susnux October 11, 2026 20:54
@Chartman123 Chartman123 self-assigned this Oct 11, 2026
@Chartman123
Chartman123 force-pushed the fix/stale-submission-id branch from fef6517 to e54f4cd Compare October 11, 2026 20:57
@codecov

codecov Bot commented Oct 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
lib/Controller/PageController.php 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Chartman123 Chartman123 changed the title fix: remove submissionId from initial state to fix stale ids in the frontend after editing a submission fix: remove submissionId from initial state Oct 11, 2026

@Koc Koc left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, interesting. Can you share reproducer steps?

@Chartman123

Copy link
Copy Markdown
Collaborator Author

Wow, interesting. Can you share reproducer steps?

Yes sure, I clicked on edit submission in the results view. Than changed one answer and went to some other form to test your grid pr for (re) storing the answers to/from local storage. Then I saw that the changes were gone again and the original values were loaded. When I then went to some other forms submit view I saw the call to get the submissions of the previous form in the network tab.

…rontend after editing a submission

Signed-off-by: Christian Hartmann <chris-hartmann@gmx.de>
@Chartman123
Chartman123 force-pushed the fix/stale-submission-id branch from e54f4cd to 88bd7c9 Compare October 12, 2026 09:51
@Chartman123
Chartman123 requested a review from Koc October 12, 2026 09:51
@Chartman123

Copy link
Copy Markdown
Collaborator Author

@Koc I found and fixed another bug. Now submissions loaded from the backend will be added to local storage. Until now only questions that have been updated got written there. Please review again :)

@Koc

Koc commented Oct 12, 2026

Copy link
Copy Markdown
Collaborator

Looks good from code perspective side. But I haven't tested it manually. But I trust you 🤝 😄

@Chartman123
Chartman123 merged commit df847d4 into main Oct 12, 2026
50 of 51 checks passed
@Chartman123
Chartman123 deleted the fix/stale-submission-id branch October 12, 2026 12:58
@Chartman123

Copy link
Copy Markdown
Collaborator Author

/backport to stable5.4

@backportbot backportbot Bot added the backport-request Pending backport by the backport-bot label Oct 12, 2026
@backportbot backportbot Bot removed the backport-request Pending backport by the backport-bot label Oct 12, 2026
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 feature: 📝 submitting responses

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants