Merge pull request #71 from Unity-Technologies/UNI-22860-bug-export-works-without-selection

UNI-22860 export happens even if nothing selected

Author: vkovec

Assets/FbxExporters/Editor/ConvertToModel.cs

@@ -33,9 +33,15 @@ public void Dispose () { }
             public static void OnMenuItem ()
             {
                 GameObject [] unityGameObjectsToConvert = Selection.GetFiltered<GameObject> (SelectionMode.Editable | SelectionMode.TopLevel);
+                if (unityGameObjectsToConvert.Length <= 0) {
+                    ModelExporter.DisplayNoSelectionDialog ();
+                    return;
+                }
+
                 Object[] result = CreateInstantiatedModelPrefab (unityGameObjectsToConvert);
-                if (result.Length>0)
+                if (result.Length > 0) {
                     Selection.objects = result;
+                }
             }
 
             /// <summary>

Assets/FbxExporters/Editor/FbxExporter.cs

@@ -1112,6 +1112,10 @@ private void ReplaceFile ()
             [MenuItem (MenuItemName, false)]
             public static void OnMenuItem ()
             {
+                if (Selection.objects.Length <= 0) {
+                    DisplayNoSelectionDialog ();
+                    return;
+                }
                 OnExport ();
             }
 
@@ -1131,9 +1135,21 @@ public static bool OnValidateMenuItem ()
             [MenuItem ("GameObject/Export Model...", false, 30)]
             static void OnContextItem (MenuCommand command)
             {
+                if (Selection.objects.Length <= 0) {
+                    DisplayNoSelectionDialog ();
+                    return;
+                }
                 OnExport ();
             }
 
+            public static void DisplayNoSelectionDialog()
+            {
+                UnityEditor.EditorUtility.DisplayDialog (
+                    "Fbx Exporter Warning", 
+                    "No GameObjects selected for export.", 
+                    "Ok");
+            }
+
             //
             // export mesh info from Unity
             //