[patch] Backport of fix to http://bugzilla.audacityteam.org/show_bug.cgi?id=1315

FileExport.patch (text/plain), 2.64 KB, created by BobbyK on 2017-07-30 12:48:36 UTC

--- A/src/export/Export.cpp	2017-03-08 08:15:50.000000000 -0500
+++ B/src/export/Export.cpp	2017-07-26 20:08:55.399358895 -0400
@@ -517,6 +517,7 @@
       mFilterIndex = 0;
    }
    maskString.RemoveLast();
+   wxString defext = mPlugins[mFormat]->GetExtension(mSubFormat).Lower();
 
 //Bug 1304: Set a default path if none was given.  For Export.
 #ifdef __WIN32__
@@ -532,37 +533,44 @@
    mFilename.SetPath(gPrefs->Read(wxT("/Export/Path"), wxT("~/Documents")));
 #endif
    mFilename.SetName(mProject->GetName());
+   if (mFilename.GetName().empty())
+      mFilename.SetName(_("untitled"));
    while (true) {
       // Must reset each iteration
       mBook = NULL;
 
-      FileDialog fd(mProject,
-                    mFileDialogTitle,
-                    mFilename.GetPath(),
-                    mFilename.GetFullName(),
-                    maskString,
-                    wxFD_SAVE | wxRESIZE_BORDER);
-      mDialog = &fd;
-      mDialog->PushEventHandler(this);
-
-      fd.SetUserPaneCreator(CreateUserPaneCallback, (wxUIntPtr) this);
-      fd.SetFilterIndex(mFilterIndex);
+      {
+         auto useFileName = mFilename;
+         if (!useFileName.HasExt())
+            useFileName.SetExt(defext);
+         FileDialog fd(mProject,
+                       mFileDialogTitle,
+                       mFilename.GetPath(),
+                       useFileName.GetFullName(),
+                       maskString,
+                       wxFD_SAVE | wxRESIZE_BORDER);
+         mDialog = &fd;
+         mDialog->PushEventHandler(this);
+
+         fd.SetUserPaneCreator(CreateUserPaneCallback, (wxUIntPtr) this);
+         fd.SetFilterIndex(mFilterIndex);
 
-      int result = fd.ShowModal();
+         int result = fd.ShowModal();
 
-      mDialog->PopEventHandler();
+         mDialog->PopEventHandler();
 
-      if (result == wxID_CANCEL) {
-         return false;
-      }
+         if (result == wxID_CANCEL) {
+            return false;
+         }
 
-      mFilename = fd.GetPath();
-      if (mFilename == wxT("")) {
-         return false;
-      }
+         mFilename = fd.GetPath();
+         if (mFilename == wxT("")) {
+            return false;
+         }
 
-      mFormat = fd.GetFilterIndex();
-      mFilterIndex = fd.GetFilterIndex();
+         mFormat = fd.GetFilterIndex();
+         mFilterIndex = fd.GetFilterIndex();
+      }
 
       int c = 0;
       int i = -1;
@@ -581,7 +589,7 @@
       }
 
       wxString ext = mFilename.GetExt();
-      wxString defext = mPlugins[mFormat]->GetExtension(mSubFormat).Lower();
+      defext = mPlugins[mFormat]->GetExtension(mSubFormat).Lower();
 
       //
       // Check the extension - add the default if it's not there,