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

EMSUSD-987 undo edit prototype #3593

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

pierrebai-adsk
Copy link
Collaborator

Editing as Maya an instanced object and then undoing the edit-as-Maya would crash. That was because when editing an instance, Maya nodes were briefly created for the prototype, registered as newly-created nodes, then immediately deleted. When undoing, the undo was trying to delete those non-existing nodes.

Fix by removing the deleted node from the map of created Maya nodes.

Editing as Maya an instanced object and then undoing the edit-as-Maya would
crash. That was because when editing an instance, Maya nodes were briefly
created for the prototype, registered as newly-created nodes, then immediately
deleted. When undoing, the undo was trying to delete those non-existing nodes.

Fix by removing the deleted node from the map of created Maya nodes.
@pierrebai-adsk pierrebai-adsk force-pushed the bailp/EMSUSD-987/undo-edit-prototype-crash branch from 717998d to b1a164f Compare February 5, 2024 15:48
@pierrebai-adsk pierrebai-adsk self-assigned this Feb 5, 2024
@pierrebai-adsk pierrebai-adsk added bug Something isn't working adsk Related to Autodesk plugin labels Feb 5, 2024
@pierrebai-adsk pierrebai-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Feb 5, 2024
@seando-adsk seando-adsk added workflows Related to in-context workflows and removed adsk Related to Autodesk plugin labels Feb 6, 2024
@seando-adsk seando-adsk merged commit 622bd40 into dev Feb 6, 2024
11 checks passed
@seando-adsk seando-adsk deleted the bailp/EMSUSD-987/undo-edit-prototype-crash branch February 6, 2024 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ready-for-merge Development process is finished, PR is ready for merge workflows Related to in-context workflows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants