Merge pull request #26 from Unity-Technologies/UNI-20152-reliable-naming

Uni 20152 reliable naming

Author: inwoods

Assets/FbxExporters/Editor/ConvertToModel.cs

@@ -21,6 +21,9 @@ public class ConvertToModel : System.IDisposable
             const string MenuItemName1 = "Assets/Convert To Model";
             const string MenuItemName2 = "GameObject/Convert To Model";
 
+            const string RegexCharStart = "[";
+            const string RegexCharEnd = "]";
+
             /// <summary>
             /// Clean up this class on garbage collection
             /// </summary>
@@ -76,7 +79,8 @@ private static List<GameObject> OnConvertInPlace (GameObject [] unityActiveGOs)
                 string dirPath = Path.Combine (Application.dataPath, "Objects");
 
                 for(int n = 0; n < gosToExport.Length; n++){
-                    filePaths[n] = Path.Combine (dirPath, gosToExport[n].name + ".fbx");
+                    string filename = ConvertToValidFilename (gosToExport [n].name + ".fbx");
+                    filePaths[n] = Path.Combine (dirPath, filename);
                 }
 
                 string[] fbxFileNames = new string[filePaths.Length];
@@ -136,6 +140,12 @@ private static List<GameObject> OnConvertInPlace (GameObject [] unityActiveGOs)
                 return result;
             }
 
+            private static string ConvertToValidFilename(string filename)
+            {
+                return System.Text.RegularExpressions.Regex.Replace (filename, 
+                    RegexCharStart + new string(Path.GetInvalidFileNameChars()) + RegexCharEnd, "_");
+            }
+
             private static void SetupImportedGameObject(GameObject orig, GameObject imported)
             {
                 Transform importedTransform = imported.transform;

Assets/FbxExporters/Editor/FbxExportSettings.cs

@@ -11,8 +11,21 @@ public class ExportSettingsEditor : UnityEditor.Editor {
         public override void OnInspectorGUI() {
             ExportSettings exportSettings = (ExportSettings)target;
 
+            // Increasing the label width so that none of the text gets cut off
+            EditorGUIUtility.labelWidth = 300;
+
             exportSettings.weldVertices = EditorGUILayout.Toggle ("Weld Vertices:", exportSettings.weldVertices);
             exportSettings.embedTextures = EditorGUILayout.Toggle ("Embed Textures:", exportSettings.embedTextures);
+            exportSettings.mayaCompatibleNames = EditorGUILayout.Toggle (
+                new GUIContent("Convert to Maya Compatible Naming:",
+                    "In Maya some symbols such as spaces and accents get replaced when importing an FBX " +
+                    "(e.g. \"foo bar\" becomes \"fooFBXASC032bar\"). " +
+                    "On export, convert the names of GameObjects so they are Maya compatible." +
+                    (exportSettings.mayaCompatibleNames? "" : 
+                        "\n\nWARNING: Disabling this feature may result in lost material connections," +
+                        " and unexpected character replacements in Maya.")
+                ),
+                exportSettings.mayaCompatibleNames);
 
             if (GUI.changed) {
                 EditorUtility.SetDirty (exportSettings);
@@ -26,6 +39,7 @@ public class ExportSettings : FbxExporters.EditorTools.ScriptableSingleton<Expor
     {
         public bool weldVertices = true;
         public bool embedTextures = false;
+        public bool mayaCompatibleNames = true;
 
         [MenuItem("Edit/Project Settings/Fbx Export", priority = 300)]
         static void ShowManager()

Assets/FbxExporters/Editor/FbxExporter.cs

@@ -36,6 +36,10 @@ public class ModelExporter : System.IDisposable
 
             const string ProgressBarTitle = "Fbx Export";
 
+            const char InvalidCharReplacement = '_';
+
+            const char MayaNamespaceSeparator = ':';
+
             /// <summary>
             /// Create instance of example
             /// </summary>
@@ -469,6 +473,10 @@ protected int ExportComponents (
             {
                 int numObjectsExported = exportProgress;
 
+                if (FbxExporters.EditorTools.ExportSettings.instance.mayaCompatibleNames) {
+                    unityGo.name = ConvertToMayaCompatibleName (unityGo.name);
+                }
+
                 // create an FbxNode and add it as a child of parent
                 FbxNode fbxNode = FbxNode.Create (fbxScene, unityGo.name);
                 NumNodes++;
@@ -838,6 +846,11 @@ public Material Material {
                         if (!renderer) {
                             return null;
                         }
+
+                        if (FbxExporters.EditorTools.ExportSettings.instance.mayaCompatibleNames) {
+                            renderer.sharedMaterial.name = ConvertToMayaCompatibleName (renderer.sharedMaterial.name);
+                        }
+
                         // .material instantiates a new material, which is bad
                         // most of the time.
                         return renderer.sharedMaterial;
@@ -1011,6 +1024,48 @@ private static void EnsureDirectory (string path)
                     Directory.CreateDirectory (fileInfo.Directory.FullName);
                 }
             }
+
+            /// <summary>
+            /// Removes the diacritics (i.e. accents) from letters.
+            /// e.g. é becomes e
+            /// </summary>
+            /// <returns>Text with accents removed.</returns>
+            /// <param name="text">Text.</param>
+            private static string RemoveDiacritics(string text) 
+            {
+                var normalizedString = text.Normalize(System.Text.NormalizationForm.FormD);
+                var stringBuilder = new System.Text.StringBuilder();
+
+                foreach (var c in normalizedString)
+                {
+                    var unicodeCategory = System.Globalization.CharUnicodeInfo.GetUnicodeCategory(c);
+                    if (unicodeCategory != System.Globalization.UnicodeCategory.NonSpacingMark)
+                    {
+                        stringBuilder.Append(c);
+                    }
+                }
+
+                return stringBuilder.ToString().Normalize(System.Text.NormalizationForm.FormC);
+            }
+
+            private static string ConvertToMayaCompatibleName(string name)
+            {
+                string newName = RemoveDiacritics (name);
+
+                if (char.IsDigit (newName [0])) {
+                    newName = newName.Insert (0, InvalidCharReplacement.ToString());
+                }
+
+                for (int i = 0; i < newName.Length; i++) {
+                    if (!char.IsLetterOrDigit (newName, i)) {
+                        if (i < newName.Length-1 && newName [i] == MayaNamespaceSeparator) {
+                            continue;
+                        }
+                        newName = newName.Replace (newName [i], InvalidCharReplacement);
+                    }
+                }
+                return newName;
+            }
         }
     }
 }