Changeset 12

Timestamp:

02/07/08 02:33:13 (5 years ago)

Author:

tobias382

Message:

Fixes #7
* Modified Quit plugin to use doQuit instead of disconnect
* Added a note in the base Driver class for plugins not to use disconnect

Location:

trunk/Phergie

Files:

• Client/Driver.php (modified) (1 diff)
• Event/Handler/Quit.php (modified) (1 diff)

trunk/Phergie/Client/Driver.php

@@ -38 +38 @@
 
     /**
-    * Terminates any existing connection to the server.
+    * Terminates any existing connection to the server. Note that this 
+    * should never be used by an event handler; doQuit should be called 
+    * instead.
     */
     public function disconnect();

trunk/Phergie/Event/Handler/Quit.php

@@ -11 +11 @@
     {
         if($event->getArgument(1)=='quit' && $this->isAdmin($event)) {
-                $this->disconnect();
+                $this->doQuit('by request of ' . $event->getNick());
         }
     }