Skip to content

Fix crash when clicking panels during recycling#36457

Merged
bdach merged 4 commits intoppy:masterfrom
eeliogata:fix-panel-click-null-reference
Mar 6, 2026
Merged

Fix crash when clicking panels during recycling#36457
bdach merged 4 commits intoppy:masterfrom
eeliogata:fix-panel-click-null-reference

Conversation

@eeliogata
Copy link
Contributor

Closes #36246

This PR overrides HandlePositionalInput to reject all positional input when Item is null.

@peppy
Copy link
Member

peppy commented Jan 25, 2026

Test coverage?

@peppy peppy moved this from Inbox to Pending Review in osu! team task tracker Mar 6, 2026
@peppy peppy self-requested a review March 6, 2026 06:54
@peppy peppy force-pushed the fix-panel-click-null-reference branch from 9b9793f to 03f187a Compare March 6, 2026 07:16
@pull-request-size pull-request-size bot added size/S and removed size/XS labels Mar 6, 2026
@peppy
Copy link
Member

peppy commented Mar 6, 2026

Added test coverage and adjusted code a bit.

@bdach 5 minutes please.

@peppy peppy requested review from bdach and peppy March 6, 2026 07:17
@bdach bdach merged commit e911217 into ppy:master Mar 6, 2026
7 of 9 checks passed
@github-project-automation github-project-automation bot moved this from Pending Review to Done in osu! team task tracker Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Unhandled error when clicking between difficulties quickly

3 participants