frmContactEditor: Stop XAB crashing when pressing Modify/Delete

[xestiaab/.git] / source / contacteditor / frmContactEditor-Other.cpp

diff --git a/source/contacteditor/frmContactEditor-Other.cpp b/source/contacteditor/frmContactEditor-Other.cpp
index 97519e6..171b3ae 100644 (file)
--- a/source/contacteditor/frmContactEditor-Other.cpp
+++ b/source/contacteditor/frmContactEditor-Other.cpp
@@ -23,11 +23,10 @@
 void frmContactEditor::AddVendorNamespace( wxCommandEvent& event )
 {
        
-       int intResult = 0;
+       // Bring up the window for adding vendor namespace data.
     
        frmContactEditorVendor *frameCEVen = new frmContactEditorVendor ( this );
        frameCEVen->SetEditorMode(FALSE);
-       intResult = GetLastInt(&ContactEditorData.VendorList);
        frameCEVen->SetupPointers(&ContactEditorData.VendorList,
                &ContactEditorData.VendorListPEN,
                 &ContactEditorData.VendorListElement,
@@ -42,10 +41,12 @@ void frmContactEditor::AddVendorNamespace( wxCommandEvent& event )
 void frmContactEditor::ModifyVendorNamespace( wxCommandEvent& event )
 {
        
+       // Bring up the window for modifying vendor namespace data.
+       
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboVendorNamespace,
+       if (lboVendorNamespace->GetItemCount() == 0 || !GetSelectedItem(lboVendorNamespace,
                &longSelected,
                &intSelectedData)){
                return;
@@ -66,10 +67,13 @@ void frmContactEditor::ModifyVendorNamespace( wxCommandEvent& event )
 
 void frmContactEditor::DeleteVendorNamespace( wxCommandEvent& event )
 {
+       
+       // Bring up the window for deleting vendor namespace data.
+       
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboVendorNamespace,
+       if (lboVendorNamespace->GetItemCount() == 0 || !GetSelectedItem(lboVendorNamespace,
                &longSelected,
                 &intSelectedData)){
                return;
@@ -85,11 +89,10 @@ void frmContactEditor::DeleteVendorNamespace( wxCommandEvent& event )
 void frmContactEditor::AddXToken( wxCommandEvent& event )
 {
        
-       int intResult = 0;
-    
+       // Bring up the window for adding extended data.
+           
        frmContactEditorXToken *frameCEXT = new frmContactEditorXToken ( this );
        frameCEXT->SetEditorMode(FALSE);
-       intResult = GetLastInt(&ContactEditorData.XTokenList);
        frameCEXT->SetupPointers(&ContactEditorData.XTokenList,
                &ContactEditorData.XTokenListTokens,
                 lboXToken,
@@ -103,10 +106,12 @@ void frmContactEditor::AddXToken( wxCommandEvent& event )
 void frmContactEditor::ModifyXToken( wxCommandEvent& event )
 {
        
+       // Bring up the window for modifying extended data.
+       
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboXToken,
+       if (lboXToken->GetItemCount() == 0 || !GetSelectedItem(lboXToken,
        
                &longSelected,
                 &intSelectedData)){
@@ -129,10 +134,12 @@ void frmContactEditor::ModifyXToken( wxCommandEvent& event )
 void frmContactEditor::DeleteXToken( wxCommandEvent& event )
 {
        
+       // Bring up the window for deleting extended data.
+       
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboXToken,
+       if (lboXToken->GetItemCount() == 0 || !GetSelectedItem(lboXToken,
                &longSelected,
                 &intSelectedData)){
                return;