Skip to content

Fix: Prevent FriendsListToggle null reference after scene changes - #1297

Merged
Pong-Epic merged 9 commits into
developmentfrom
bug/eosu-1391-friends-list-toggle-null-ref
Jul 27, 2026
Merged

Fix: Prevent FriendsListToggle null reference after scene changes#1297
Pong-Epic merged 9 commits into
developmentfrom
bug/eosu-1391-friends-list-toggle-null-ref

Conversation

@JessTello

Copy link
Copy Markdown
Contributor
  • Clear static Friends tab UI callbacks on scene unload to avoid retaining old listeners.
  • Safely invoke Friends tab expand/collapse callbacks, skipping destroyed Unity targets.
  • Guard optional UIFriendInteractionSource usage to prevent calls into missing or destroyed references during UI refresh.

- Clear static Friends tab UI callbacks on scene unload to avoid retaining old listeners.
- Safely invoke Friends tab expand/collapse callbacks, skipping destroyed Unity targets.
- Guard optional UIFriendInteractionSource usage to prevent calls into missing or destroyed references during UI refresh.
@JessTello
JessTello requested review from a team and Pong-Epic June 11, 2026 21:35
Comment thread Assets/Scripts/StandardSamples/UI/Lobbies/UIActions.cs Outdated
Comment thread Assets/Scripts/StandardSamples/UI/Lobbies/UIActions.cs Outdated
Comment thread Assets/Scripts/StandardSamples/UI/Lobbies/UIActions.cs Outdated
Comment thread Assets/Scripts/StandardSamples/UI/Lobbies/UIActions.cs Outdated
Comment thread Assets/Scripts/StandardSamples/UI/Lobbies/UIActions.cs Outdated
Comment thread Assets/Scripts/StandardSamples/UI/Lobbies/UIActions.cs Outdated
Comment thread Assets/Scripts/StandardSamples/UI/Lobbies/UIActions.cs Outdated

@ricardoamores-epic ricardoamores-epic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

matt-clarke
matt-clarke previously approved these changes Jul 22, 2026
Comment thread Assets/Scripts/StandardSamples/UI/Lobbies/UIActions.cs Outdated
@Pong-Epic

Copy link
Copy Markdown

Finially fixed the add/remove listener in correct pair in OnEnable/OnDisable

Comment thread Assets/Scripts/StandardSamples/UI/Lobbies/UILobbiesMenu.cs Outdated
matt-clarke
matt-clarke previously approved these changes Jul 24, 2026
@Pong-Epic
Pong-Epic removed their request for review July 27, 2026 10:00
@Pong-Epic
Pong-Epic merged commit ff79ed2 into development Jul 27, 2026
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.

4 participants