Fix for [#32792] BVH exporter crashes on rigs with multiple roots.

In this situation, the script creates a dummy root bone... But code was buggy (and import of non-rotating bones was too).

1 files changed, 12 insertions, 4 deletions

diff --git a/io_anim_bvh/export_bvh.py b/io_anim_bvh/export_bvh.py
index fcddb63c..4d7f0b09 100644
--- a/io_anim_bvh/export_bvh.py
+++ b/io_anim_bvh/export_bvh.py
@@ -99,7 +99,7 @@ def write_armature(context,
 
         if my_children:
             # store the location for the children
-            # to het their relative offset
+            # to get their relative offset
 
             # Write children
             for child_bone in my_children:
@@ -124,15 +124,23 @@ def write_armature(context,
         write_recursive_nodes(key, indent)
 
     else:
-        # Write a dummy parent node
+        # Write a dummy parent node, with a dummy key name
+        # Just be sure it's not used by another bone!
+        i = 0
+        key = "__%d" % i
+        while key in children:
+            i += 1
+            key = "__%d" % i
         file.write("ROOT %s\n" % key)
         file.write("{\n")
         file.write("\tOFFSET 0.0 0.0 0.0\n")
         file.write("\tCHANNELS 0\n")  # Xposition Yposition Zposition Xrotation Yrotation Zrotation
-        key = None
         indent = 1
 
-        write_recursive_nodes(key, indent)
+        # Write children
+        for child_bone in children[None]:
+            serialized_names.append(child_bone)
+            write_recursive_nodes(child_bone, indent)
 
         file.write("}\n")