LOOKDEVX-1912 - Append a '1' to new primitives names only if the name…

… does not end with a digit.
Autodesk · Sep 18, 2023 · a60e02e · a60e02e
1 parent 974b0a1
commit a60e02e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/lib/usdUfe/ufe/UsdUndoAddNewPrimCommand.cpp b/lib/usdUfe/ufe/UsdUndoAddNewPrimCommand.cpp
@@ -58,8 +58,14 @@ UsdUndoAddNewPrimCommand::UsdUndoAddNewPrimCommand(
     auto dagSegment = segments[0];
     _stage = usdSceneItem->prim().GetStage();
     if (_stage) {
+
+        std::string base, suffixStr;
+
         // Append the parent path and the requested name into a full ufe path.
-        _newUfePath = appendToPath(ufePath, name + '1');
+        // Append a '1' to new primitives names if the name does not end with a digit.
+        _newUfePath = splitNumericalSuffix(name, base, suffixStr)
+            ? appendToPath(ufePath, name)
+            : appendToPath(ufePath, name + '1');
 
         // Ensure the requested name is unique.
         auto newPrimName

diff --git a/test/lib/ufe/testContextOps.py b/test/lib/ufe/testContextOps.py
@@ -1141,7 +1141,7 @@ def testMaterialBindingWithNodeDefHandler(self):
 
         self.assertTrue(shaderAttrs.hasAttribute("info:id"))
         self.assertEqual(shaderAttrs.attribute("info:id").get(), shaderName)
-        self.assertEqual(ufe.PathString.string(shaderItem.path()), "|stage1|stageShape1,/Material1/Red11")
+        self.assertEqual(ufe.PathString.string(shaderItem.path()), "|stage1|stageShape1,/Material1/Red1")
         materialHier = ufe.Hierarchy.hierarchy(materialItem)
         self.assertTrue(materialHier.hasChildren())