Made the OkDialog and ConfirmDialog classes proxies for their buttons' events. Removed the RequestTradeWindow class, replaced with a plain ConfirmDialog. Fixed a memory leak along the way.

1 files changed, 8 insertions, 5 deletions

diff --git a/src/gui/confirm_dialog.cpp b/src/gui/confirm_dialog.cpp
index 2babbf23..becb8859 100644
--- a/src/gui/confirm_dialog.cpp
+++ b/src/gui/confirm_dialog.cpp
@@ -29,7 +29,7 @@
 
 
 ConfirmDialog::ConfirmDialog(const std::string &title, const std::string &msg,
-        gcn::ActionListener *listener, Window *parent):
+        Window *parent):
     Window(title, true, parent)
 {
     gcn::Label *textLabel = new gcn::Label(msg);
@@ -57,10 +57,6 @@ ConfirmDialog::ConfirmDialog(const std::string &title, const std::string &msg,
     noButton->setEventId("no");
     yesButton->addActionListener(this);
     noButton->addActionListener(this);
-    if (listener) {
-        yesButton->addActionListener(listener);
-        noButton->addActionListener(listener);
-    }
 
     add(textLabel);
     add(yesButton);
@@ -75,6 +71,13 @@ ConfirmDialog::ConfirmDialog(const std::string &title, const std::string &msg,
 
 void ConfirmDialog::action(const std::string &eventId)
 {
+    // Proxy button events to our listeners
+    ActionListenerIterator i;
+    for (i = mActionListeners.begin(); i != mActionListeners.end(); ++i)
+    {
+        (*i)->action(eventId);
+    }
+
     // Can we receive anything else anyway?
     if (eventId == "yes" || eventId == "no") {
         scheduleDelete();