Skip to content

fix(webpush): Fix "Undefined array key -… at lib/Push.php#297" - #3314

Merged
nickvergessen merged 2 commits into
masterfrom
bugfix/noid/fix-undefined-index-when-skipping-token
Aug 28, 2026
Merged

fix(webpush): Fix "Undefined array key -… at lib/Push.php#297"#3314
nickvergessen merged 2 commits into
masterfrom
bugfix/noid/fix-undefined-index-when-skipping-token

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

πŸ€– AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@github-actions

Copy link
Copy Markdown
Contributor

🐒 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 9152 was 9041 (+1.22%)
Please check your code again. If you added a new test this can be expected and the base value in tests/Integration/base-query-count.txt can be increased.

Signed-off-by: Joas Schilling <coding@schilljs.com>
Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessen force-pushed the bugfix/noid/fix-undefined-index-when-skipping-token branch from cb898fa to f61f07d Compare August 28, 2026 07:30
@github-actions

Copy link
Copy Markdown
Contributor

🐒 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 9152 was 9041 (+1.22%)
Please check your code again. If you added a new test this can be expected and the base value in tests/Integration/base-query-count.txt can be increased.

@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable35

@nickvergessen
nickvergessen merged commit afe84a8 into master Aug 28, 2026
46 checks passed
@nickvergessen
nickvergessen deleted the bugfix/noid/fix-undefined-index-when-skipping-token branch August 28, 2026 07:47
@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable34

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants