wxDocument::SaveAs sets m_savedYet flag correctly (patch 494838)

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@13247 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

diff --git a/src/common/docview.cpp b/src/common/docview.cpp
index 7588bcdd81fcaaad3138ddbd5c3d44adf0f926f5..95b9f93b460ad1244cd01235811084ab31a175c7 100644 (file)
--- a/src/common/docview.cpp
+++ b/src/common/docview.cpp
@@ -227,16 +227,13 @@ bool wxDocument::OnNewDocument()
 
 bool wxDocument::Save()
 {
 
 bool wxDocument::Save()
 {

-    bool ret = FALSE;
+    if (!IsModified() && m_savedYet)
+        return TRUE;

 
 

-    if (!IsModified() && m_savedYet) return TRUE;
-    if (m_documentFile == wxT("") || !m_savedYet)
-        ret = SaveAs();
-    else
-        ret = OnSaveDocument(m_documentFile);
-    if ( ret )
-        SetDocumentSaved(TRUE);
-    return ret;
+    if ( m_documentFile.empty() || !m_savedYet )
+        return SaveAs();
+
+    return OnSaveDocument(m_documentFile);

 }
 
 bool wxDocument::SaveAs()
 }
 
 bool wxDocument::SaveAs()


@@ -316,6 +313,7 @@ bool wxDocument::OnSaveDocument(const wxString& file)
     }
     Modify(FALSE);
     SetFilename(file);
     }
     Modify(FALSE);
     SetFilename(file);

+    SetDocumentSaved(TRUE);

     return TRUE;
 }
 
     return TRUE;
 }