Merge remote-tracking branch 'refs/remotes/origin/master' into UNI-30583-hide-disabled-gameobjects-in-fbx

# Conflicts:
#	Assets/FbxExporters/Editor/UnitTests/ModelExporterTest.cs

Author: vkovec

Assets/FbxExporters/Editor/FbxExportSettings.cs

@@ -10,9 +10,11 @@ namespace FbxExporters.EditorTools {
     [CustomEditor(typeof(ExportSettings))]
     public class ExportSettingsEditor : UnityEditor.Editor {
         Vector2 scrollPos = Vector2.zero;
-        const float LabelWidth = 130;
+        const float LabelWidth = 144;
         const float SelectableLabelMinWidth = 90;
         const float BrowseButtonWidth = 25;
+        const float FieldOffset = 18;
+        const float BrowseButtonOffset = 5;
 
         public override void OnInspectorGUI() {
             ExportSettings exportSettings = (ExportSettings)target;
@@ -27,6 +29,8 @@ public override void OnInspectorGUI() {
                 GUILayout.Label ("Version: " + version, EditorStyles.centeredGreyMiniLabel);
                 EditorGUILayout.Space ();
             }
+            EditorGUILayout.LabelField("Export Options", EditorStyles.boldLabel);
+            EditorGUI.indentLevel++;
             GUILayout.BeginVertical();
             exportSettings.mayaCompatibleNames = EditorGUILayout.Toggle (
                 new GUIContent ("Compatible Naming:",
@@ -45,47 +49,53 @@ public override void OnInspectorGUI() {
                 exportSettings.centerObjects
             );
 
-            EditorGUILayout.Space();
-
             GUILayout.BeginHorizontal();
-            GUILayout.Label(new GUIContent("Export Format:", "Export the FBX file in the standard binary format." +
-                " Select ASCII to export the FBX file in ASCII format."), GUILayout.Width(LabelWidth - 3));
+            EditorGUILayout.LabelField(new GUIContent("Export Format:", "Export the FBX file in the standard binary format." +
+                " Select ASCII to export the FBX file in ASCII format."), GUILayout.Width(LabelWidth - FieldOffset));
             exportSettings.ExportFormatSelection = EditorGUILayout.Popup(exportSettings.ExportFormatSelection, new string[]{"Binary", "ASCII"});
             GUILayout.EndHorizontal();
 
-            EditorGUILayout.Space();
-
-            GUILayout.BeginHorizontal ();
-            GUILayout.Label (new GUIContent (
+            GUILayout.BeginHorizontal();
+            EditorGUILayout.LabelField(new GUIContent(
                 "Export Path:",
-                "Relative path for saving Model Prefabs."), GUILayout.Width(LabelWidth - 3));
+                "Relative path for saving Model Prefabs."), GUILayout.Width(LabelWidth - FieldOffset));
 
             var pathLabel = ExportSettings.GetRelativeSavePath();
             if (pathLabel == ".") { pathLabel = "(Assets root)"; }
             EditorGUILayout.SelectableLabel(pathLabel,
                 EditorStyles.textField,
                 GUILayout.MinWidth(SelectableLabelMinWidth),
                 GUILayout.Height(EditorGUIUtility.singleLineHeight));
+            GUILayout.EndHorizontal();
+            GUILayout.BeginHorizontal();
+
+            GUILayout.Space(LabelWidth + BrowseButtonOffset);
 
-            if (GUILayout.Button (new GUIContent("...", "Browse to a new location for saving model prefabs"), EditorStyles.miniButton, GUILayout.Width (BrowseButtonWidth))) {
+            if (GUILayout.Button(new GUIContent("...", "Browse to a new location for saving model prefabs"), EditorStyles.miniButton, GUILayout.Width(BrowseButtonWidth)))
+            {
                 string initialPath = ExportSettings.GetAbsoluteSavePath();
 
                 // if the directory doesn't exist, set it to the default save path
                 // so we don't open somewhere unexpected
-                if (!System.IO.Directory.Exists (initialPath)) {
+                if (!System.IO.Directory.Exists(initialPath))
+                {
                     initialPath = Application.dataPath;
                 }
 
-                string fullPath = EditorUtility.OpenFolderPanel (
+                string fullPath = EditorUtility.OpenFolderPanel(
                         "Select Model Prefabs Path", initialPath, null
                         );
 
                 // Unless the user canceled, make sure they chose something in the Assets folder.
-                if (!string.IsNullOrEmpty (fullPath)) {
+                if (!string.IsNullOrEmpty(fullPath))
+                {
                     var relativePath = ExportSettings.ConvertToAssetRelativePath(fullPath);
-                    if (string.IsNullOrEmpty(relativePath)) {
-                        Debug.LogWarning ("Please select a location in the Assets folder");
-                    } else {
+                    if (string.IsNullOrEmpty(relativePath))
+                    {
+                        Debug.LogWarning("Please select a location in the Assets folder");
+                    }
+                    else
+                    {
                         ExportSettings.SetRelativeSavePath(relativePath);
 
                         // Make sure focus is removed from the selectable label
@@ -95,18 +105,28 @@ public override void OnInspectorGUI() {
                     }
                 }
             }
-            GUILayout.EndHorizontal ();
+            GUILayout.EndHorizontal();
+
+            EditorGUILayout.Space();
+            EditorGUILayout.Space();
+            EditorGUI.indentLevel--;
+            EditorGUILayout.LabelField("Integration", EditorStyles.boldLabel);
+            EditorGUI.indentLevel++;
 
             GUILayout.BeginHorizontal ();
-            GUILayout.Label (new GUIContent (
+            EditorGUILayout.LabelField(new GUIContent (
                 "Integrations Path:",
-                "Installation path for 3D application integrations."), GUILayout.Width(LabelWidth - 3));
+                "Installation path for 3D application integrations."), GUILayout.Width(LabelWidth - FieldOffset));
 
             var IntegrationsPathLabel = ExportSettings.GetIntegrationSavePath();
             EditorGUILayout.SelectableLabel(IntegrationsPathLabel,
                 EditorStyles.textField,
                 GUILayout.MinWidth(SelectableLabelMinWidth),
                 GUILayout.Height(EditorGUIUtility.singleLineHeight));
+            GUILayout.EndHorizontal();
+            GUILayout.BeginHorizontal();
+
+            GUILayout.Space(LabelWidth + BrowseButtonOffset);
 
             if (GUILayout.Button(new GUIContent("...", "Browse to a new installation path for 3D application integrations"), EditorStyles.miniButton, GUILayout.Width(BrowseButtonWidth)))
             {
@@ -129,17 +149,21 @@ public override void OnInspectorGUI() {
 
             GUILayout.EndHorizontal();
 
-            EditorGUILayout.Space ();
-
             GUILayout.BeginHorizontal ();
-            GUILayout.Label (new GUIContent (
+            EditorGUILayout.LabelField(new GUIContent (
                 "3D Application:",
-                "Select the 3D Application for which you would like to install the Unity integration."), GUILayout.Width(LabelWidth - 3));
+                "Select the 3D Application for which you would like to install the Unity integration."), GUILayout.Width(LabelWidth - FieldOffset));
 
             // dropdown to select Maya version to use
             var options = ExportSettings.GetDCCOptions();
 
             exportSettings.selectedDCCApp = EditorGUILayout.Popup(exportSettings.selectedDCCApp, options);
+
+            GUILayout.EndHorizontal();
+            GUILayout.BeginHorizontal();
+
+            GUILayout.Space(LabelWidth + BrowseButtonOffset);
+
             if (GUILayout.Button(new GUIContent("...", "Browse to a 3D application in a non-default location"), EditorStyles.miniButton, GUILayout.Width(BrowseButtonWidth))) {
                 var ext = "";
                 switch (Application.platform) {
@@ -176,19 +200,28 @@ public override void OnInspectorGUI() {
 
             EditorGUILayout.Space();
 
+            exportSettings.launchAfterInstallation = EditorGUILayout.Toggle(
+                new GUIContent("Keep open:",
+                    "Keep the selected 3D application open after Unity integration install has completed."),
+                exportSettings.launchAfterInstallation
+            );
+
+            exportSettings.HideSendToUnityMenu = EditorGUILayout.Toggle(
+                new GUIContent("Hide native menu:",
+                    "Replace Maya's native 'Send to Unity' menu with the Unity Integration's menu"),
+                exportSettings.HideSendToUnityMenu
+            );
+
+            EditorGUILayout.Space();
+
+
             var installIntegrationContent = new GUIContent(
                     "Install Unity Integration",
                     "Install and configure the Unity integration for the selected 3D application so that you can import and export directly with this project.");
             if (GUILayout.Button (installIntegrationContent)) {
                 FbxExporters.Editor.IntegrationsUI.InstallDCCIntegration ();
             }
 
-            exportSettings.launchAfterInstallation = EditorGUILayout.Toggle(
-                new GUIContent("Keep 3D Application opened:",
-                    "Keep the selected 3D application open after Unity integration install has completed."),
-                exportSettings.launchAfterInstallation
-            );
-
             GUILayout.FlexibleSpace ();
             GUILayout.EndScrollView ();
             GUILayout.EndVertical();
@@ -300,6 +333,7 @@ public static string[] DCCVendorLocations {
         public bool mayaCompatibleNames;
         public bool centerObjects;
         public bool launchAfterInstallation;
+        public bool HideSendToUnityMenu;
         public int ExportFormatSelection;
 
         public string IntegrationSavePath;
@@ -331,6 +365,7 @@ protected override void LoadDefaults()
             mayaCompatibleNames = true;
             centerObjects = true;
             launchAfterInstallation = true;
+            HideSendToUnityMenu = true;
             ExportFormatSelection = 0;
             convertToModelSavePath = kDefaultSavePath;
             IntegrationSavePath = DefaultIntegrationSavePath;
@@ -601,15 +636,15 @@ public static GUIContent[] GetDCCOptions(){
                 FindDCCInstalls ();
             }
 
+            // store the selected app
+            var prevSelection = instance.dccOptionPaths[instance.selectedDCCApp];
+
             // remove options that no longer exist
             List<string> pathsToDelete = new List<string>();
             List<string> namesToDelete = new List<string>();
             for(int i = 0; i < instance.dccOptionPaths.Count; i++) {
                 var dccPath = instance.dccOptionPaths [i];
                 if (!File.Exists (dccPath)) {
-                    if (i == instance.selectedDCCApp) {
-                        instance.selectedDCCApp = instance.GetPreferredDCCApp();
-                    }
                     namesToDelete.Add (instance.dccOptionNames [i]);
                     pathsToDelete.Add (dccPath);
                 }
@@ -621,6 +656,13 @@ public static GUIContent[] GetDCCOptions(){
                 instance.dccOptionNames.Remove (str);
             }
 
+            // set the selected DCC app to the previous selection
+            instance.selectedDCCApp = instance.dccOptionPaths.IndexOf (prevSelection);
+            if (instance.selectedDCCApp < 0) {
+                // find preferred app if previous selection no longer exists
+                instance.selectedDCCApp = instance.GetPreferredDCCApp ();
+            }
+
             if (instance.dccOptionPaths.Count <= 0) {
                 instance.selectedDCCApp = 0;
                 return new GUIContent[]{
@@ -635,7 +677,6 @@ public static GUIContent[] GetDCCOptions(){
                     instance.dccOptionPaths[i]
                 );
             }
-
             return optionArray;
         }
 
@@ -762,7 +803,8 @@ public static void SetRelativeSavePath(string newPath) {
         public static string GetIntegrationSavePath()
         {
             //If the save path gets messed up and ends up not being valid, just use the project folder as the default
-            if (string.IsNullOrEmpty(instance.IntegrationSavePath.Trim()) || !Directory.Exists(instance.IntegrationSavePath))
+            if (string.IsNullOrEmpty(instance.IntegrationSavePath) ||
+                !Directory.Exists(instance.IntegrationSavePath))
             {
                 //The project folder, above the asset folder
                 instance.IntegrationSavePath = DefaultIntegrationSavePath;