Skip to content

fix: prevent exception in Boss Room with NetworkAnimator #2140

Merged
jeffreyrainy merged 2 commits intodevelopfrom
fix/boss-room-network-animator-crash
Aug 22, 2022
Merged

fix: prevent exception in Boss Room with NetworkAnimator #2140
jeffreyrainy merged 2 commits intodevelopfrom
fix/boss-room-network-animator-crash

Conversation

@jeffreyrainy
Copy link
Copy Markdown
Contributor

Prevents an exception in Boss Room with NetworkAnimator due to m_NetworkBehaviour being null

Following a previous change for performance, a null exception is raised in Boss Room. This works around it.

@jeffreyrainy jeffreyrainy requested a review from a team as a code owner August 22, 2022 18:38
@jeffreyrainy
Copy link
Copy Markdown
Contributor Author

@ashwinimurt This will come up in Boss Room testing if not merged in. We still need some work to understand why it is needed, but at least it removes the exception.

@jeffreyrainy jeffreyrainy enabled auto-merge (squash) August 22, 2022 23:16
Copy link
Copy Markdown
Contributor

@JesseOlmer JesseOlmer left a comment

Choose a reason for hiding this comment

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

lgtm. Please make sure there's a JIRA issue to track the long term fix/investigation

@jeffreyrainy jeffreyrainy merged commit 08f6f99 into develop Aug 22, 2022
@jeffreyrainy jeffreyrainy deleted the fix/boss-room-network-animator-crash branch August 22, 2022 23:46
jakobbbb pushed a commit to GooseGirlGames/com.unity.netcode.gameobjects that referenced this pull request Feb 22, 2023
…ologies#2140)

fix: prevent exception in Boss Room with NetworkAnimator due to m_NetworkBehaviour being null
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.

3 participants