From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Sun, 20 Aug 2006 10:54:50 +0000 (+0000)
Subject: connect the affirmative/cancel button handlers to new ids, not old ones
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/fabd7a7fa1b379fbd589b1bbac2d226dae4f2fbe

connect the affirmative/cancel button handlers to new ids, not old ones


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

diff --git a/src/common/dlgcmn.cpp b/src/common/dlgcmn.cpp
index ec34631b67..730b2c300f 100644
--- a/src/common/dlgcmn.cpp
+++ b/src/common/dlgcmn.cpp
@@ -458,16 +458,16 @@ void wxDialogBase::SetAffirmativeId(int affirmativeId)
     }
     //else: wxID_OK is always handled
 
+    m_affirmativeId = affirmativeId;
+
     // connect the handler to the new button
-    if ( affirmativeId != wxID_NONE )
+    if ( m_affirmativeId != wxID_NONE )
     {
         Connect(m_affirmativeId,
                 wxEVT_COMMAND_BUTTON_CLICKED,
                 wxCommandEventHandler(wxDialogBase::OnAffirmativeButton));
     }
     //else: no affirmative button
-
-    m_affirmativeId = affirmativeId;
 }
 
 void wxDialogBase::SetEscapeId(int escapeId)
@@ -491,15 +491,15 @@ void wxDialogBase::SetEscapeId(int escapeId)
     }
     //else: wxID_CANCEL is always handled
 
+    m_escapeId = escapeId;
+
     // connect the handler to the new button
-    if ( escapeId != wxID_NONE )
+    if ( m_escapeId != wxID_NONE )
     {
         Connect(m_escapeId,
                 wxEVT_COMMAND_BUTTON_CLICKED,
                 wxCommandEventHandler(wxDialogBase::OnCancelButton));
     }
-
-    m_escapeId = escapeId;
 }
 
 // ----------------------------------------------------------------------------