MAYA-111095 - [GitHub #1340] Mute layer doesn't work when a Layer is …

[spinell-adsk]

…loaded through Load Sublayer

-Update layerEditorCommand to use the layer identifier when adding a subLayerPath

[spinell-adsk]

When loading a layer from the layer editor "Load sublayers..." menu, we are using the filesystem path as an argument for
"mayaUsdLayerEditor -edit -insertSubPath" and this command will call Sdf::InsertSubLayerPath() with the filesystem path.
On Windows, if the path is absolute, the driver letter is in uppercase.

When USD need to build the layer stack in (PcpLayerStack::_BuildLayerStack()), it's check inside a cache if a sublayer is muted or not by using the subLayerPath. The cache is filled by function like USDCache::MuteLayer() which take the layer identifier as parameter. So, when USD check inside the cache to know which layer are mutes, it do a string comparation between the sublayerPath and the layer identifier which can be different on Windows. On Windows, If a layer's path is absolute, the layer's identifier will be the path but the driver letter will always be in lower case.

To fix this issue I think there is 2 solutions.
We can force the layer filesytem path to have a lowercase driver letter or we can send the layer identifier Sdf::InsertSubLayerPath().

I checked inside USD codebase (Python/c++ tests) and they are always using identifier with Sdf::InsertSubLayerPath() so I choose this way.

[ppt-adsk]

Is there a way to add an automated test for this?

[spinell-adsk]

@ppt-adsk I have added a test for the mute command.

[ppt-adsk]

Nice test!