From 1ccef789c4fffe156cd2cb84a4dab8444cf25a7e Mon Sep 17 00:00:00 2001
From: ousttrue <ousttrue@gmail.com>
Date: Sat, 26 Jan 2019 16:12:10 +0900
Subject: [PATCH] Fix exporter logic that getting node.mesh index #140

---
 Assets/VRM/UniGLTF/Editor/UniGLTFTests.cs     | 14 ++++++++++++--
 Assets/VRM/UniGLTF/Scripts/IO/gltfExporter.cs | 14 +++++++-------
 2 files changed, 19 insertions(+), 9 deletions(-)

diff --git a/Assets/VRM/UniGLTF/Editor/UniGLTFTests.cs b/Assets/VRM/UniGLTF/Editor/UniGLTFTests.cs
index 3ed2ce06f..a2b9de381 100644
--- a/Assets/VRM/UniGLTF/Editor/UniGLTFTests.cs
+++ b/Assets/VRM/UniGLTF/Editor/UniGLTFTests.cs
@@ -587,6 +587,7 @@ namespace UniGLTF
                 {
                     cubeA.transform.SetParent(go.transform);
                     var material = new Material(shader);
+                    material.name = "red";
                     material.color = Color.red;
                     cubeA.GetComponent<Renderer>().sharedMaterial = material;
                 }
@@ -596,6 +597,7 @@ namespace UniGLTF
                     cubeB.transform.SetParent(go.transform);
                     var material = new Material(shader);
                     material.color = Color.blue;
+                    material.name = "blue";
                     cubeB.GetComponent<Renderer>().sharedMaterial = material;
 
                     Assert.AreEqual(cubeB.GetComponent<MeshFilter>().sharedMesh, cubeA.GetComponent<MeshFilter>().sharedMesh);
@@ -620,6 +622,10 @@ namespace UniGLTF
                 var blue = gltf.materials[gltf.meshes[1].primitives[0].material];
                 Assert.AreEqual(new float[] { 0, 0, 1, 1 }, blue.pbrMetallicRoughness.baseColorFactor);
 
+                Assert.AreEqual(2, gltf.nodes.Count);
+
+                Assert.AreNotEqual(gltf.nodes[0].mesh, gltf.nodes[1].mesh);
+
                 // import
                 var context = new ImporterContext();
                 context.ParseJson(json, new SimpleStorage(new ArraySegment<byte>(new byte[1024 * 1024])));
@@ -627,10 +633,14 @@ namespace UniGLTF
                 context.Load();
 
                 var importedRed = context.Root.transform.GetChild(0);
-                Assert.AreEqual(Color.red, importedRed.GetComponent<Renderer>().sharedMaterial.color);
+                var importedRedMaterial = importedRed.GetComponent<Renderer>().sharedMaterial;
+                Assert.AreEqual("red", importedRedMaterial.name);
+                Assert.AreEqual(Color.red, importedRedMaterial.color);
 
                 var importedBlue = context.Root.transform.GetChild(1);
-                Assert.AreEqual(Color.blue, importedBlue.GetComponent<Renderer>().sharedMaterial.color);
+                var importedBlueMaterial = importedBlue.GetComponent<Renderer>().sharedMaterial;
+                Assert.AreEqual("blue", importedBlueMaterial.name);
+                Assert.AreEqual(Color.blue, importedBlueMaterial.color);
             }
             finally
             {
diff --git a/Assets/VRM/UniGLTF/Scripts/IO/gltfExporter.cs b/Assets/VRM/UniGLTF/Scripts/IO/gltfExporter.cs
index ed6d0f396..9a955dd81 100644
--- a/Assets/VRM/UniGLTF/Scripts/IO/gltfExporter.cs
+++ b/Assets/VRM/UniGLTF/Scripts/IO/gltfExporter.cs
@@ -151,7 +151,7 @@ namespace UniGLTF
         }
 
         #region Export
-        static glTFNode ExportNode(Transform x, List<Transform> nodes, List<Mesh> meshes, List<SkinnedMeshRenderer> skins)
+        static glTFNode ExportNode(Transform x, List<Transform> nodes, List<Renderer> renderers, List<SkinnedMeshRenderer> skins)
         {
             var node = new glTFNode
             {
@@ -164,16 +164,16 @@ namespace UniGLTF
 
             if (x.gameObject.activeInHierarchy)
             {
-                var meshFilter = x.GetComponent<MeshFilter>();
-                if (meshFilter != null)
+                var meshRenderer = x.GetComponent<MeshRenderer>();
+                if (meshRenderer != null)
                 {
-                    node.mesh = meshes.IndexOf(meshFilter.sharedMesh);
+                    node.mesh = renderers.IndexOf(meshRenderer);
                 }
 
                 var skinnredMeshRenderer = x.GetComponent<SkinnedMeshRenderer>();
                 if (skinnredMeshRenderer != null)
                 {
-                    node.mesh = meshes.IndexOf(skinnredMeshRenderer.sharedMesh);
+                    node.mesh = renderers.IndexOf(skinnredMeshRenderer);
                     node.skin = skins.IndexOf(skinnredMeshRenderer);
                 }
             }
@@ -244,14 +244,14 @@ namespace UniGLTF
                 Meshes = unityMeshes.Select(x => x.Mesh).ToList();
                 #endregion
 
-                #region Skins
+                #region Nodes and Skins
                 var unitySkins = Nodes
                     .Select(x => x.GetComponent<SkinnedMeshRenderer>()).Where(x =>
                         x != null
                         && x.bones != null
                         && x.bones.Length > 0)
                     .ToList();
-                gltf.nodes = Nodes.Select(x => ExportNode(x, Nodes, unityMeshes.Select(y => y.Mesh).ToList(), unitySkins)).ToList();
+                gltf.nodes = Nodes.Select(x => ExportNode(x, Nodes, unityMeshes.Select(y => y.Rendererer).ToList(), unitySkins)).ToList();
                 gltf.scenes = new List<gltfScene>
                 {
                     new gltfScene