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

MAYA-128211 block command when layers are muted #2934

Merged
merged 3 commits into from
Mar 15, 2023

Conversation

pierrebai-adsk
Copy link
Collaborator

When there is at least one muted layers, block the delete, rename, group and reparent commands. These are commands that re-organize the hierarchy and must thus affect all layers with opinions. When some of these layers are muted, they would not be able to be updated with the new prim path, thus we prevent the operation.

Add unit tests for the commands. Note that reparent and group are composite commands using the same underlying command (insert child) and thus require only one test.

When there is at least one muted layers, block the delete, rename, group and reparent commands. These are commands that re-organize the hierarchy and must thus affect all layers with opinions. When some of these layers are muted, they would not be able to be updated with the new prim path, thus we prevent the operation.

Add unit tests for the commands. Note that reparent and group are composite commands using the same underlying command (insert child) and thus require only one test.
@pierrebai-adsk pierrebai-adsk added bug Something isn't working adsk Related to Autodesk plugin labels Mar 10, 2023
@pierrebai-adsk
Copy link
Collaborator Author

Note PF only failed in one OSX instance which timeout, but that particular platform passed in a previous attempt, so it was only a glitch.

Copy link
Collaborator

@vlasovi vlasovi left a comment

Choose a reason for hiding this comment

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

Looks good

@pierrebai-adsk pierrebai-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Mar 14, 2023
@seando-adsk seando-adsk added ufe-usd Related to UFE-USD plugin in Maya-Usd and removed adsk Related to Autodesk plugin labels Mar 15, 2023
@seando-adsk seando-adsk merged commit e87e10d into dev Mar 15, 2023
@seando-adsk seando-adsk deleted the bailp/MAYA-128211/muted-layers-block-commands branch March 15, 2023 12:37
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 ufe-usd Related to UFE-USD plugin in Maya-Usd
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants