Merge pull request #265 from php-school/remove-listener-feature

Remove listeners feature

Author: AydinHassan

src/Event/EventDispatcher.php

@@ -78,6 +78,22 @@ private function attachListener(string $eventName, callable $callback): void
         }
     }
 
+    public function removeListener(string $eventName, callable $callback): void
+    {
+        foreach ($this->listeners[$eventName] ?? [] as $key => $listener) {
+            if ($listener === $callback) {
+                unset($this->listeners[$eventName][$key]);
+                $this->listeners[$eventName] = array_values($this->listeners[$eventName]);
+
+                if (empty($this->listeners[$eventName])) {
+                    unset($this->listeners[$eventName]);
+                }
+
+                break;
+            }
+        }
+    }
+
     /**
      * Insert a verifier callback which will execute at the given event name much like normal listeners.
      * A verifier should return an object which implements `PhpSchool\PhpWorkshop\Result\FailureInterface`

src/Listener/OutputRunInfoListener.php

@@ -28,10 +28,6 @@ public function __construct(OutputInterface $output, RequestRenderer $requestRen
 
     public function __invoke(Event $event): void
     {
-        if (PHP_SAPI !== "cli") {
-            return;
-        }
-
         switch (get_class($event)) {
             case CliExecuteEvent::class:
                 $args = $event->getArgs();

test/Event/EventDispatcherTest.php

@@ -139,4 +139,26 @@ public function testListenersAndVerifiersAreCalledInOrderOfAttachment(): void
 
         $this->eventDispatcher->dispatch($e1);
     }
+
+    public function testRemoveListener(): void
+    {
+        $listener = function () {
+        };
+
+        $listener2 = function () {
+        };
+
+        $this->eventDispatcher->listen('some-event', $listener);
+        $this->eventDispatcher->listen('some-event', $listener2);
+
+        $this->assertEquals(['some-event' => [$listener, $listener2]], $this->eventDispatcher->getListeners());
+
+        $this->eventDispatcher->removeListener('some-event', $listener);
+
+        $this->assertEquals(['some-event' => [$listener2]], $this->eventDispatcher->getListeners());
+
+        $this->eventDispatcher->removeListener('some-event', $listener2);
+
+        $this->assertEquals([], $this->eventDispatcher->getListeners());
+    }
 }