frmContactEditor: Check save was successful before closing

diff --git a/source/contacteditor/frmContactEditor-Save.cpp b/source/contacteditor/frmContactEditor-Save.cpp
index baeeae5..dca32d0 100644 (file)
--- a/source/contacteditor/frmContactEditor-Save.cpp
+++ b/source/contacteditor/frmContactEditor-Save.cpp
@@ -33,7 +33,8 @@ void frmContactEditor::SaveContact( wxCommandEvent& event )
        
        if (boolUnsupportedAccount == true){
        
-               wxMessageBox(_("Cannot make changes to a contact from an unsupported account type."), _("Unsupported account"), wxICON_ERROR);          
+               wxMessageBox(_("Cannot make changes to a contact from an unsupported account type."), _("Unsupported account"), wxICON_ERROR);
+               saveSuccess = false;
                return;
                
        }
@@ -46,6 +47,7 @@ void frmContactEditor::SaveContact( wxCommandEvent& event )
        if (cmbDisplayAsValue.IsEmpty()){
         
                wxMessageBox(_("Display As value cannot be left blank."), _("Display As value empty"), wxICON_ERROR);
+               saveSuccess = false;
                return;
         
        }
@@ -566,6 +568,8 @@ void frmContactEditor::SaveContact( wxCommandEvent& event )
                event2.SetClientData(ucd);
                wxPostEvent(MainPtr, event2);
        }
+       
+       saveSuccess = true;     
     
 }
 
@@ -576,6 +580,9 @@ void frmContactEditor::SaveCloseContact( wxCommandEvent& event )
     
        wxCommandEvent NullEvent;
        this->SaveContact(NullEvent);
-       this->Close();
+       if (saveSuccess)
+       {
+               this->Close();
+       }
     
 }
\ No newline at end of file

diff --git a/source/contacteditor/frmContactEditor.h b/source/contacteditor/frmContactEditor.h
index 49dd826..dbad355 100644 (file)
--- a/source/contacteditor/frmContactEditor.h
+++ b/source/contacteditor/frmContactEditor.h
@@ -744,6 +744,7 @@ class frmContactEditor : public frmContactEditorADT
                bool EditMode = FALSE;
                bool StartupEditMode = FALSE;
                bool DialogOpen = FALSE;
+               bool saveSuccess = false;
                
                /*
                std::string base64dec;