frmContactEditor: Stop XAB crashing when pressing Modify/Delete

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

diff --git a/source/contacteditor/frmContactEditor-Business.cpp b/source/contacteditor/frmContactEditor-Business.cpp
index 5a19a7c..245b348 100644 (file)
--- a/source/contacteditor/frmContactEditor-Business.cpp
+++ b/source/contacteditor/frmContactEditor-Business.cpp
@@ -36,11 +36,10 @@
 void frmContactEditor::AddBusinessNickname( wxCommandEvent& event )
 {
 
-       int intResult = 0;
-    
+       // Bring up the window for adding a nickname.
+   
        frmContactEditorNickname *frameCENickname = new frmContactEditorNickname ( this );
        frameCENickname->SetEditorMode(FALSE, CE_WORK);
-       intResult = GetLastInt(&ContactEditorData.BusinessNicknamesList);
        frameCENickname->SetupPointers(&ContactEditorData.BusinessNicknamesList,
                &ContactEditorData.BusinessNicknamesListAltID,
                &ContactEditorData.BusinessNicknamesListPID,
@@ -59,10 +58,12 @@ void frmContactEditor::AddBusinessNickname( wxCommandEvent& event )
 void frmContactEditor::ModifyBusinessNickname( wxCommandEvent& event )
 {
 
+       // Bring up the window for modifying a nickname.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessNicknames,
+       if (lboBusinessNicknames->GetItemCount() == 0 || !GetSelectedItem(lboBusinessNicknames,
                &longSelected,
                &intSelectedData)){
                
@@ -90,10 +91,12 @@ void frmContactEditor::ModifyBusinessNickname( wxCommandEvent& event )
 void frmContactEditor::DeleteBusinessNickname( wxCommandEvent& event )
 {
 
+       // Bring up the window to delete the nickname.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessNicknames,
+       if (lboBusinessNicknames->GetItemCount() == 0 || !GetSelectedItem(lboBusinessNicknames,
                &longSelected,
                &intSelectedData)){
                
@@ -117,11 +120,10 @@ void frmContactEditor::DeleteBusinessNickname( wxCommandEvent& event )
 void frmContactEditor::AddBusinessAddress( wxCommandEvent& event )
 {
 
-       int intResult = 0;
+       // Bring up the window for adding an address.
     
        frmContactEditorAddress *frameCEAddress = new frmContactEditorAddress ( this );
        frameCEAddress->SetEditorMode(FALSE, CE_WORK);
-       intResult = GetLastInt(&ContactEditorData.BusinessAddressList);
        frameCEAddress->SetupPointers(&ContactEditorData.BusinessAddressList,
                &ContactEditorData.BusinessAddressListTown,
                &ContactEditorData.BusinessAddressListCounty,
@@ -148,10 +150,12 @@ void frmContactEditor::AddBusinessAddress( wxCommandEvent& event )
 void frmContactEditor::ModifyBusinessAddress( wxCommandEvent& event )
 {
 
+       // Bring up a window for editing an address.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessAddresses,
+       if (lboBusinessAddresses->GetItemCount() == 0 || !GetSelectedItem(lboBusinessAddresses,
                &longSelected,
                &intSelectedData)){
                
@@ -187,10 +191,12 @@ void frmContactEditor::ModifyBusinessAddress( wxCommandEvent& event )
 void frmContactEditor::DeleteBusinessAddress( wxCommandEvent& event )
 {
 
+       // Bring up the window for deleting the address.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessAddresses,
+       if (lboBusinessAddresses->GetItemCount() == 0 || !GetSelectedItem(lboBusinessAddresses,
                &longSelected,
                &intSelectedData)){
                
@@ -214,11 +220,10 @@ void frmContactEditor::DeleteBusinessAddress( wxCommandEvent& event )
 void frmContactEditor::AddBusinessEmail( wxCommandEvent& event )
 {
 
-       int intResult = 0;
+       // Bring up the window for adding an email address.
     
        frmContactEditorEmail *frameCEEmail = new frmContactEditorEmail ( this );
        frameCEEmail->SetEditorMode(FALSE, CE_WORK);
-       intResult = GetLastInt(&ContactEditorData.BusinessAddressList);
        frameCEEmail->SetupPointers(&ContactEditorData.BusinessEmailList,
                &ContactEditorData.BusinessEmailListAltID,
                &ContactEditorData.BusinessEmailListPID,
@@ -236,10 +241,12 @@ void frmContactEditor::AddBusinessEmail( wxCommandEvent& event )
 void frmContactEditor::ModifyBusinessEmail( wxCommandEvent& event )
 {
 
+       // Bring up the window for modifying the email address.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessEmail,
+       if (lboBusinessEmail->GetItemCount() == 0 || !GetSelectedItem(lboBusinessEmail,
                &longSelected,
                &intSelectedData)){
                
@@ -266,10 +273,12 @@ void frmContactEditor::ModifyBusinessEmail( wxCommandEvent& event )
 void frmContactEditor::DeleteBusinessEmail( wxCommandEvent& event )
 {
 
+       // Bring up the window to delete the email address.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessEmail,
+       if (lboBusinessEmail->GetItemCount() == 0 || !GetSelectedItem(lboBusinessEmail,
                          &longSelected,
                          &intSelectedData)){
                         
@@ -289,11 +298,10 @@ void frmContactEditor::DeleteBusinessEmail( wxCommandEvent& event )
 void frmContactEditor::AddBusinessIM( wxCommandEvent& event )
 {
 
-       int intResult = 0;
+       // Bring up the window for adding an IM address.
     
        frmContactEditorIM *frameCEIM = new frmContactEditorIM ( this );
        frameCEIM->SetEditorMode(FALSE, CE_WORK);
-       intResult = GetLastInt(&ContactEditorData.BusinessIMList);
        frameCEIM->SetupPointers(&ContactEditorData.BusinessIMList,
                &ContactEditorData.BusinessIMListAltID,
                &ContactEditorData.BusinessIMListPID,
@@ -313,10 +321,12 @@ void frmContactEditor::AddBusinessIM( wxCommandEvent& event )
 void frmContactEditor::ModifyBusinessIM( wxCommandEvent& event )
 {
 
+       // Bring up the window for editing an IM address.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessIM,
+       if (lboBusinessIM->GetItemCount() == 0 || !GetSelectedItem(lboBusinessIM,
                &longSelected,
                &intSelectedData)){
                
@@ -345,10 +355,12 @@ void frmContactEditor::ModifyBusinessIM( wxCommandEvent& event )
 void frmContactEditor::DeleteBusinessIM( wxCommandEvent& event )
 {
 
+       // Bring up the window for deleting an IM address.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessIM,
+       if (lboBusinessIM->GetItemCount() == 0 || !GetSelectedItem(lboBusinessIM,
                &longSelected,
                &intSelectedData)){
                
@@ -369,11 +381,10 @@ void frmContactEditor::DeleteBusinessIM( wxCommandEvent& event )
 void frmContactEditor::AddBusinessTelephone( wxCommandEvent& event )
 {
 
-       int intResult = 0;
+       // Bring up the window for adding a telephone number.
     
        frmContactEditorTelephone *frameCETel = new frmContactEditorTelephone ( this );
        frameCETel->SetEditorMode(FALSE, CE_WORK);
-       intResult = GetLastInt(&ContactEditorData.BusinessTelephoneList);
        frameCETel->SetupPointers(&ContactEditorData.BusinessTelephoneList,
                &ContactEditorData.BusinessTelephoneListAltID,
                &ContactEditorData.BusinessTelephoneListPID,
@@ -392,10 +403,12 @@ void frmContactEditor::AddBusinessTelephone( wxCommandEvent& event )
 void frmContactEditor::ModifyBusinessTelephone( wxCommandEvent& event )
 {
 
+       // Bring up the address for editing a telephone number.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessTelephone,
+       if (lboBusinessTelephone->GetItemCount() == 0 || !GetSelectedItem(lboBusinessTelephone,
                &longSelected,
                &intSelectedData)){
                
@@ -423,10 +436,12 @@ void frmContactEditor::ModifyBusinessTelephone( wxCommandEvent& event )
 void frmContactEditor::DeleteBusinessTelephone( wxCommandEvent& event )
 {
 
+       // Bring up the window for deleting a telephone number.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessTelephone,
+       if (lboBusinessTelephone->GetItemCount() == 0 || !GetSelectedItem(lboBusinessTelephone,
                &longSelected,
                &intSelectedData)){
                
@@ -446,11 +461,10 @@ void frmContactEditor::DeleteBusinessTelephone( wxCommandEvent& event )
 void frmContactEditor::AddBusinessLanguage( wxCommandEvent& event )
 {
 
-       int intResult = 0;
+       // Bring up the window for adding a language.
     
        frmContactEditorLanguages *frameCELang = new frmContactEditorLanguages ( this );
        frameCELang->SetEditorMode(FALSE, CE_WORK);
-       intResult = GetLastInt(&ContactEditorData.BusinessLanguageList);
        frameCELang->SetupPointers(&ContactEditorData.BusinessLanguageList,
                &ContactEditorData.BusinessLanguageListAltID,
                &ContactEditorData.BusinessLanguageListPID,
@@ -468,10 +482,12 @@ void frmContactEditor::AddBusinessLanguage( wxCommandEvent& event )
 void frmContactEditor::ModifyBusinessLanguage( wxCommandEvent& event )
 {
 
+       // Bring up the window for editing a language.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessLanguages,
+       if (lboBusinessLanguages->GetItemCount() == 0 || !GetSelectedItem(lboBusinessLanguages,
                &longSelected,
                &intSelectedData)){
                
@@ -498,10 +514,12 @@ void frmContactEditor::ModifyBusinessLanguage( wxCommandEvent& event )
 void frmContactEditor::DeleteBusinessLanguage( wxCommandEvent& event )
 {
 
+       // Bring up the window for deleting a language.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessLanguages,
+       if (lboBusinessLanguages->GetItemCount() == 0 || !GetSelectedItem(lboBusinessLanguages,
                &longSelected,
                &intSelectedData)){
                         
@@ -521,11 +539,10 @@ void frmContactEditor::DeleteBusinessLanguage( wxCommandEvent& event )
 void frmContactEditor::AddBusinessTimezone( wxCommandEvent& event )
 {
 
-       int intResult = 0;
+       // Bring up the window for adding a timezone.
     
        frmContactEditorTimezones *frameCETZ = new frmContactEditorTimezones ( this );
        frameCETZ->SetEditorMode(FALSE, CE_WORK);
-       intResult = GetLastInt(&ContactEditorData.BusinessTZList);
        frameCETZ->SetupPointers(&ContactEditorData.BusinessTZList,
                &ContactEditorData.BusinessTZListAltID,
                &ContactEditorData.BusinessTZListPID,
@@ -544,10 +561,12 @@ void frmContactEditor::AddBusinessTimezone( wxCommandEvent& event )
 void frmContactEditor::ModifyBusinessTimezone( wxCommandEvent& event )
 {
 
+       // Bring up the window for modifying a timezone.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessTimezones,
+       if (lboBusinessTimezones->GetItemCount() == 0 || !GetSelectedItem(lboBusinessTimezones,
                &longSelected,
                &intSelectedData)){
                
@@ -575,10 +594,12 @@ void frmContactEditor::ModifyBusinessTimezone( wxCommandEvent& event )
 void frmContactEditor::DeleteBusinessTimezone( wxCommandEvent& event )
 {
 
+       // Bring up the window for deleting a timezone.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessTimezones,
+       if (lboBusinessTimezones->GetItemCount() == 0 || !GetSelectedItem(lboBusinessTimezones,
                &longSelected,
                &intSelectedData)){
                
@@ -598,11 +619,10 @@ void frmContactEditor::DeleteBusinessTimezone( wxCommandEvent& event )
 void frmContactEditor::AddBusinessGeoposition( wxCommandEvent& event )
 {
 
-       int intResult = 0;
+       // Bring up the window for adding a geoposition location.
     
        frmContactEditorGeoposition *frameCEGeo = new frmContactEditorGeoposition ( this );
        frameCEGeo->SetEditorMode(FALSE, CE_WORK);
-       intResult = GetLastInt(&ContactEditorData.BusinessGeographyList);
        frameCEGeo->SetupPointers(&ContactEditorData.BusinessGeographyList,
                &ContactEditorData.BusinessGeographyListAltID,
                &ContactEditorData.BusinessGeographyListPID,
@@ -622,10 +642,12 @@ void frmContactEditor::AddBusinessGeoposition( wxCommandEvent& event )
 void frmContactEditor::ModifyBusinessGeoposition( wxCommandEvent& event )
 {
 
+       // Bring up the window for editing a geoposition location.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessGeoposition,
+       if (lboBusinessGeoposition->GetItemCount() == 0 || !GetSelectedItem(lboBusinessGeoposition,
                &longSelected,
                &intSelectedData)){
                
@@ -654,10 +676,12 @@ void frmContactEditor::ModifyBusinessGeoposition( wxCommandEvent& event )
 void frmContactEditor::DeleteBusinessGeoposition( wxCommandEvent& event )
 {
 
+       // Bring up the window for deleting a geoposition location.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessGeoposition,
+       if (lboBusinessGeoposition->GetItemCount() == 0 || !GetSelectedItem(lboBusinessGeoposition,
                &longSelected,
                &intSelectedData)){
                
@@ -678,11 +702,10 @@ void frmContactEditor::DeleteBusinessGeoposition( wxCommandEvent& event )
 void frmContactEditor::AddBusinessWebsite( wxCommandEvent& event )
 {
 
-       int intResult = 0;
+       // Bring up the window for adding a website.
     
        frmContactEditorWebsites *frameCEWeb = new frmContactEditorWebsites ( this );
        frameCEWeb->SetEditorMode(FALSE, CE_WORK);
-       intResult = GetLastInt(&ContactEditorData.BusinessWebsiteList);
        frameCEWeb->SetupPointers(&ContactEditorData.BusinessWebsiteList,
                &ContactEditorData.BusinessWebsiteListAltID,
                &ContactEditorData.BusinessWebsiteListPID,
@@ -701,10 +724,12 @@ void frmContactEditor::AddBusinessWebsite( wxCommandEvent& event )
 void frmContactEditor::ModifyBusinessWebsite( wxCommandEvent& event )
 {
 
+       // Bring up a window for editing a website.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessWebsites,
+       if (lboBusinessWebsites->GetItemCount() == 0 || !GetSelectedItem(lboBusinessWebsites,
                &longSelected,
                &intSelectedData)){
                
@@ -732,10 +757,12 @@ void frmContactEditor::ModifyBusinessWebsite( wxCommandEvent& event )
 void frmContactEditor::DeleteBusinessWebsite( wxCommandEvent& event )
 {
 
+       // Bring up the window for deleting a website.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessWebsites,
+       if (lboBusinessWebsites->GetItemCount() == 0 || !GetSelectedItem(lboBusinessWebsites,
                &longSelected,
                &intSelectedData)){
                
@@ -755,11 +782,10 @@ void frmContactEditor::DeleteBusinessWebsite( wxCommandEvent& event )
 void frmContactEditor::AddBusinessTitle( wxCommandEvent& event )
 {
 
-       int intResult = 0;
+       // Bring up the window for adding a title.
     
        frmContactEditorTitles *frameCETitle = new frmContactEditorTitles ( this );
        frameCETitle->SetEditorMode(FALSE, CE_WORK);
-       intResult = GetLastInt(&ContactEditorData.BusinessTitleList);
        frameCETitle->SetupPointers(&ContactEditorData.BusinessTitleList,
                &ContactEditorData.BusinessTitleListLanguage,
                &ContactEditorData.BusinessTitleListAltID,
@@ -778,10 +804,12 @@ void frmContactEditor::AddBusinessTitle( wxCommandEvent& event )
 void frmContactEditor::ModifyBusinessTitle( wxCommandEvent& event )
 {
 
+       // Bring up the window for editing a title.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessTitles,
+       if (lboBusinessTitles->GetItemCount() == 0 || !GetSelectedItem(lboBusinessTitles,
                &longSelected,
                &intSelectedData)){
                         
@@ -809,10 +837,12 @@ void frmContactEditor::ModifyBusinessTitle( wxCommandEvent& event )
 void frmContactEditor::DeleteBusinessTitle( wxCommandEvent& event )
 {
 
+       // Bring up the window to delete the title.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessTitles,
+       if (lboBusinessTitles->GetItemCount() == 0 || !GetSelectedItem(lboBusinessTitles,
                &longSelected,
                &intSelectedData)){
                
@@ -820,7 +850,7 @@ void frmContactEditor::DeleteBusinessTitle( wxCommandEvent& event )
     
        }
     
-       lboTitles->DeleteItem(longSelected);
+       lboBusinessTitles->DeleteItem(longSelected);
     
        DeleteMapData(intSelectedData, &ContactEditorData.BusinessTitleList,
                &ContactEditorData.BusinessTitleListLanguage, &ContactEditorData.BusinessTitleListAltID,
@@ -832,11 +862,10 @@ void frmContactEditor::DeleteBusinessTitle( wxCommandEvent& event )
 void frmContactEditor::AddBusinessRole( wxCommandEvent& event )
 {
 
-       int intResult = 0;
+       // Bring up the window to add a role.
     
        frmContactEditorRoles *frameCERole = new frmContactEditorRoles ( this );
        frameCERole->SetEditorMode(FALSE, CE_WORK);
-       intResult = GetLastInt(&ContactEditorData.BusinessRoleList);
        frameCERole->SetupPointers(&ContactEditorData.BusinessRoleList,
                &ContactEditorData.BusinessRoleListLanguage,
                &ContactEditorData.BusinessRoleListAltID,
@@ -855,10 +884,12 @@ void frmContactEditor::AddBusinessRole( wxCommandEvent& event )
 void frmContactEditor::ModifyBusinessRole( wxCommandEvent& event )
 {
 
+       // Bring up the window for editing a role.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessRoles,
+       if (lboBusinessRoles->GetItemCount() == 0 || !GetSelectedItem(lboBusinessRoles,
                &longSelected,
                &intSelectedData)){
                
@@ -886,10 +917,12 @@ void frmContactEditor::ModifyBusinessRole( wxCommandEvent& event )
 void frmContactEditor::DeleteBusinessRole( wxCommandEvent& event )
 {
 
+       // Bring up the window to delete a role.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessRoles,
+       if (lboBusinessRoles->GetItemCount() == 0 || !GetSelectedItem(lboBusinessRoles,
                &longSelected,
                &intSelectedData)){
                
@@ -909,11 +942,10 @@ void frmContactEditor::DeleteBusinessRole( wxCommandEvent& event )
 void frmContactEditor::AddBusinessOrganisation( wxCommandEvent& event )
 {
 
-       int intResult = 0;
-    
+       // Bring up the window to add an organisation.
+
        frmContactEditorOrganisations *frameCEOrg = new frmContactEditorOrganisations ( this );
        frameCEOrg->SetEditorMode(FALSE, CE_WORK);
-       intResult = GetLastInt(&ContactEditorData.BusinessOrganisationsList);
        frameCEOrg->SetupPointers(&ContactEditorData.BusinessOrganisationsList,
                &ContactEditorData.BusinessOrganisationsListLanguage,
                &ContactEditorData.BusinessOrganisationsListSortAs,
@@ -933,10 +965,12 @@ void frmContactEditor::AddBusinessOrganisation( wxCommandEvent& event )
 void frmContactEditor::ModifyBusinessOrganisation( wxCommandEvent& event )
 {
 
+       // Bring up the window to edit an organisation.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessOrganisations,
+       if (lboBusinessOrganisations->GetItemCount() == 0 || !GetSelectedItem(lboBusinessOrganisations,
                &longSelected,
                &intSelectedData)){
                
@@ -965,10 +999,12 @@ void frmContactEditor::ModifyBusinessOrganisation( wxCommandEvent& event )
 void frmContactEditor::DeleteBusinessOrganisation( wxCommandEvent& event )
 {
 
+       // Bring up the window to delete a organisation.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessOrganisations,
+       if (lboBusinessOrganisations->GetItemCount() == 0 || !GetSelectedItem(lboBusinessOrganisations,
                &longSelected,
                &intSelectedData)){
                
@@ -989,11 +1025,10 @@ void frmContactEditor::DeleteBusinessOrganisation( wxCommandEvent& event )
 void frmContactEditor::AddBusinessNote( wxCommandEvent& event )
 {
 
-       int intResult = 0;
+       // Bring up the window to add a note.
     
        frmContactEditorNotes *frameCENote = new frmContactEditorNotes ( this );
        frameCENote->SetEditorMode(FALSE, CE_WORK);
-       intResult = GetLastInt(&ContactEditorData.BusinessNoteList);
        frameCENote->SetupPointers(&ContactEditorData.BusinessNoteList,
                &ContactEditorData.BusinessNoteListLanguage,
                &ContactEditorData.BusinessNoteListAltID,
@@ -1012,10 +1047,12 @@ void frmContactEditor::AddBusinessNote( wxCommandEvent& event )
 void frmContactEditor::ModifyBusinessNote( wxCommandEvent& event )
 {
 
+       // Bring up the window to edit a note.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessNotes,
+       if (lboBusinessNotes->GetItemCount() == 0 || !GetSelectedItem(lboBusinessNotes,
                &longSelected,
                &intSelectedData)){
                
@@ -1043,10 +1080,12 @@ void frmContactEditor::ModifyBusinessNote( wxCommandEvent& event )
 void frmContactEditor::DeleteBusinessNote( wxCommandEvent& event )
 {
 
+       // Bring up the window to delete a note.
+
        long longSelected = -1;
        int intSelectedData = 0;
     
-       if (!GetSelectedItem(lboBusinessNotes,
+       if (lboBusinessNotes->GetItemCount() == 0 || !GetSelectedItem(lboBusinessNotes,
                &longSelected,
                &intSelectedData)){