Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix fileComplete reference error in Loader.loadVideoTag #743

Merged
merged 1 commit into from
Sep 28, 2024

Conversation

Akurn
Copy link
Contributor

@Akurn Akurn commented Sep 28, 2024

This PR is a bug fix (closes #742)

Please include a summary in Change Log: Unreleased and thank yourself.

Describe the changes below:
The game.load reference in the videoLoadEvent callback gets nulled when the game is destroyed. This uses the local reference for fileComplete instead.

The game.load reference in the videoLoadEvent callback gets nulled when the game is destroyed.
This uses the local reference for fileComplete instead.
@Akurn
Copy link
Contributor Author

Akurn commented Sep 28, 2024

Just checking if this the correct way to make a PR?
Also do I need to add to the CHANGELOG?
Thanks @samme 🙏

@samme samme merged commit f2508a8 into phaserjs:master Sep 28, 2024
3 checks passed
@samme
Copy link
Collaborator

samme commented Sep 28, 2024

I can do the CHANGELOG.

@Akurn Akurn deleted the fix/issue-742 branch September 28, 2024 14:18
samme added a commit that referenced this pull request Sep 28, 2024
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.

Undefined reference in Loader videoLoadEvent (after game destroy)
2 participants