From 6add849446209d8c33b6f9da59bb7370107c3652 Mon Sep 17 00:00:00 2001
From: Alan Stanzione <alan@toolchefs.com>
Date: Tue, 2 Apr 2019 11:56:34 +0100
Subject: [PATCH] fixed children with name containg separation character

---
 src/AtomsGaffer/AtomsVariationReader.cpp | 31 ++++++++++++++++++++----
 1 file changed, 26 insertions(+), 5 deletions(-)

diff --git a/src/AtomsGaffer/AtomsVariationReader.cpp b/src/AtomsGaffer/AtomsVariationReader.cpp
index be79b81..44a1b27 100644
--- a/src/AtomsGaffer/AtomsVariationReader.cpp
+++ b/src/AtomsGaffer/AtomsVariationReader.cpp
@@ -317,6 +317,7 @@ public :
 
             const auto& agentTypeName = agentTypeNames[aTypeId];
             auto& agentTypeRoot = m_root.children[agentTypeName];
+            agentTypeRoot.name = agentTypeName;
             auto agentTypePtr = m_variations.getAgentTypeVariationPtr( agentTypeName);
             if ( !agentTypePtr )
                 continue;
@@ -358,7 +359,10 @@ public :
                     currentRoot->variations.insert(variationName);
                     for( const auto& objName: objectNames )
                     {
+                        if ( objName == "" )
+                            continue;
                         currentRoot = &currentRoot->children[objName];
+                        currentRoot->name = objName;
                         currentRoot->variations.insert( variationName );
                         auto objIt = currentMap->getTypedEntry<AtomsCore::MapMetadata>( objName );
                         if ( !objIt )
@@ -405,7 +409,10 @@ public :
                         AtomsDagNode* currentRoot = &agentTypeRoot;
                         currentRoot->variations.insert( variationName + ":" + lodName );
                         for ( const auto &objName: lodObjectNames ) {
+                            if ( objName == "" )
+                                continue;
                             currentRoot = &currentRoot->children[objName];
+                            currentRoot->name = objName;
                             currentRoot->variations.insert( variationName + ":" + lodName );
                             auto objIt = currentMap->getTypedEntry<AtomsCore::MapMetadata>(objName);
                             if (!objIt) {
@@ -538,8 +545,14 @@ public :
 
             AtomsDagNode* childRootNode = nullptr;
             if ( rootNode ) {
-                childRootNode = &rootNode->children[it->first];
-                childRootNode->data = meshGroup;
+                std::vector<std::string> objectNames;
+                AtomsUtils::splitString( it->first, '|', objectNames );
+
+                if ( objectNames.size() > 0)
+                {
+                    childRootNode = &rootNode->children[objectNames.back()];
+                    childRootNode->data = meshGroup;
+                }
             }
             flatMeshHierarchy( meshGroup.get(), outMap, childRootNode );
         }
@@ -836,13 +849,15 @@ public :
             const std::string& currentPath ) const
     {
         if ( node->data ) {
+
             auto sets = node->data->getTypedEntry<const AtomsCore::StringArrayMetadata>( "sets" );
             if ( sets ) {
                 auto& setVec = sets->get();
                 for ( auto& currentSet: setVec ) {
                     if ( currentSet == setName ) {
-                        for ( auto& variation: node->variations )
-                            paths.push_back( variation + "/" + currentPath );
+                        for ( auto& variation: node->variations ) {
+                            paths.push_back(variation + "/" + currentPath);
+                        }
                         break;
                     }
                 }
@@ -850,7 +865,12 @@ public :
         }
 
         for ( auto it = node->children.cbegin(); it != node->children.cend(); ++it ) {
-            collectAllPathFromSetName( setName, &it->second, paths, currentPath + "/" + it->first );
+            if (currentPath.empty() || currentPath == "/")
+            {
+                collectAllPathFromSetName(setName, &it->second, paths, it->first);
+            } else {
+                collectAllPathFromSetName(setName, &it->second, paths, currentPath + "/" + it->first);
+            }
         }
     }
 
@@ -1576,6 +1596,7 @@ IECore::ConstPathMatcherDataPtr AtomsVariationReader::computeSet( const IECore::
     }
     auto &result = resultData->writable();
 
+
     if ( !engineData->isDefaultSet( setName ) )  {
         const EngineData::AtomsDagNode *rootNode = &engineData->root();
         for ( auto it = rootNode->children.cbegin(); it != rootNode->children.cend(); ++it ) {