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 error when clearing the srcNode property in audio and video texture target components. #175

Merged
merged 1 commit into from
Dec 23, 2022

Conversation

Exairnous
Copy link
Contributor

Clearing the srcNode property triggers a reset of the bone EnumProperty, but this failed because of an unguarded check for the mode of the srcNode object when accessing the bone EnumProperty to reset it. This PR adds a check for whether the srcNode has an object before checking whether the srcNode object is in edit mode so that resetting the bone EnumProperty to the blank id doesn't cause an error when getting the list of bones.

…re target components.

Clearing the srcNode property triggers a reset of the bone EnumProperty, but this failed
because of an unguarded check for the mode of the srcNode object when accessing the bone
EnumProperty to reset it.  Add a check for whether the srcNode has an object before
checking whether the srcNode object is in edit mode so that resetting the bone
EnumProperty to the blank id doesn't cause an error when getting the list of bones.
Copy link
Contributor

@keianhzo keianhzo left a comment

Choose a reason for hiding this comment

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

LGTM

@keianhzo keianhzo merged commit ce6937f into Hubs-Foundation:master Dec 23, 2022
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.

2 participants