Make NPC dialogs instance instead of global

This change allows players to talk to multiple NPCs at a time (if the server agrees). Manaserv's netcode allows multiple commerce instances too. eAthena's is limited to one commerce instance, due to protocol limitations.

1 files changed, 32 insertions, 8 deletions

diff --git a/src/gui/npcpostdialog.cpp b/src/gui/npcpostdialog.cpp
index a05008ec..cb6660c6 100644
--- a/src/gui/npcpostdialog.cpp
+++ b/src/gui/npcpostdialog.cpp
@@ -35,8 +35,11 @@
 
 #include "utils/gettext.h"
 
-NpcPostDialog::NpcPostDialog():
-    Window(_("NPC"))
+NpcPostDialog::DialogList NpcPostDialog::instances;
+
+NpcPostDialog::NpcPostDialog(int npcId):
+    Window(_("NPC")),
+    mNpcId(npcId)
 {
     setContentSize(400, 180);
 
@@ -74,6 +77,14 @@ NpcPostDialog::NpcPostDialog():
     add(cancelButton);
 
     setLocationRelativeTo(getParent());
+
+    instances.push_back(this);
+    setVisible(true);
+}
+
+NpcPostDialog::~NpcPostDialog()
+{
+    instances.remove(this);
 }
 
 void NpcPostDialog::action(const gcn::ActionEvent &event)
@@ -87,21 +98,34 @@ void NpcPostDialog::action(const gcn::ActionEvent &event)
         }
         else
         {
-            Net::getNpcHandler()->sendLetter(current_npc, mSender->getText(),
+            Net::getNpcHandler()->sendLetter(mNpcId, mSender->getText(),
                                              mText->getText());
         }
         setVisible(false);
-        clear();
     }
     else if (event.getId() == "cancel")
     {
         setVisible(false);
-        clear();
     }
 }
 
-void NpcPostDialog::clear()
+void NpcPostDialog::setVisible(bool visible)
+{
+    Window::setVisible(visible);
+
+    if (!visible)
+    {
+        scheduleDelete();
+    }
+}
+
+void NpcPostDialog::closeAll()
 {
-    mSender->setText("");
-    mText->setText("");
+    DialogList::iterator it = instances.begin();
+    DialogList::iterator it_end = instances.end();
+
+    for (; it != it_end; it++)
+    {
+        (*it)->close();
+    }
 }