give error if user tries to export without filename

- also only close export options window if export succeeds

Author: vkovec

Assets/FbxExporters/Editor/ConvertToPrefabEditorWindow.cs

@@ -50,8 +50,18 @@ protected override void OnEnable ()
                 m_prefabExtLabelWidth = m_fbxExtLabelStyle.CalcSize (new GUIContent (".prefab")).x;
             }
 
-            protected override void Export ()
+            protected override bool Export ()
             {
+                if (string.IsNullOrEmpty (m_exportFileName)) {
+                    Debug.LogError ("FbxExporter: Please specify an fbx filename");
+                    return false;
+                }
+
+                if (string.IsNullOrEmpty (m_prefabFileName)) {
+                    Debug.LogError ("FbxExporter: Please specify a prefab filename");
+                    return false;
+                }
+
                 var fbxDirPath = ExportSettings.GetFbxAbsoluteSavePath ();
                 var fbxPath = System.IO.Path.Combine (fbxDirPath, m_exportFileName + ".fbx");
 
@@ -60,26 +70,27 @@ protected override void Export ()
 
                 // check if file already exists, give a warning if it does
                 if (!OverwriteExistingFile (fbxPath) || !OverwriteExistingFile (prefabPath)) {
-                    return;
+                    return false;
                 }
 
                 if (m_toConvert == null) {
                     Debug.LogError ("FbxExporter: missing object for conversion");
-                    return;
+                    return false;
                 }
 
                 if (m_toConvert.Length == 1) {
                     ConvertToModel.Convert (
                         m_toConvert[0], fbxFullPath: fbxPath, prefabFullPath: prefabPath, exportOptions: ExportSettings.instance.convertToPrefabSettings.info
                     );
-                    return;
+                    return true;
                 }
 
                 foreach (var go in m_toConvert) {
                     ConvertToModel.Convert (
                         go, fbxDirectoryFullPath: fbxDirPath, prefabDirectoryFullPath: prefabDirPath, exportOptions: ExportSettings.instance.convertToPrefabSettings.info
                     );
                 }
+                return true;
             }
 
             protected override bool DisableNameSelection ()

Assets/FbxExporters/Editor/ExportModelEditorWindow.cs

@@ -105,7 +105,7 @@ public void OnPresetSelectionChanged()
                 this.Repaint ();
             }
 
-            protected abstract void Export ();
+            protected abstract bool Export ();
 
             /// <summary>
             /// Function to be used by derived classes to add custom UI between the file path selector and export options.
@@ -228,8 +228,9 @@ protected void OnGUI ()
                 }
 
                 if (GUILayout.Button (ExportButtonName, GUILayout.Width(ExportButtonWidth))) {
-                    Export ();
-                    this.Close ();
+                    if (Export ()) {
+                        this.Close ();
+                    }
                 }
                 GUILayout.EndHorizontal ();
 
@@ -347,12 +348,17 @@ protected override bool DisableNameSelection ()
                 return m_isPlayableDirector;
             }
 
-            protected override void Export(){
+            protected override bool Export(){
+                if (string.IsNullOrEmpty (m_exportFileName)) {
+                    Debug.LogError ("FbxExporter: Please specify an fbx filename");
+                    return false;
+                }
+
                 var folderPath = ExportSettings.GetFbxAbsoluteSavePath ();
                 var filePath = System.IO.Path.Combine (folderPath, m_exportFileName + ".fbx");
 
                 if (!OverwriteExistingFile (filePath)) {
-                    return;
+                    return false;
                 }
 
                 if (m_isPlayableDirector) {
@@ -366,14 +372,15 @@ protected override void Export(){
                     // refresh the asset database so that the file appears in the
                     // asset folder view.
                     AssetDatabase.Refresh ();
-                    return;
+                    return true;
                 }
 
                 if (ModelExporter.ExportObjects (filePath, m_toExport, ExportSettings.instance.exportModelSettings.info, timelineAnim: m_isTimelineAnim) != null) {
                     // refresh the asset database so that the file appears in the
                     // asset folder view.
                     AssetDatabase.Refresh ();
                 }
+                return true;
             }
 
             #if UNITY_2018_1_OR_NEWER