noccy/mercureact
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bugfix topicmanager

Browse Source 
* Unsubscribing wouldn't always remove the subscriber.
This commit is contained in:
Chris 2024-03-11 01:50:34 +01:00
parent 2cbc41d29e
commit d310060309
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Broker/TopicManager.php
View File

@ -33,14 +33,15 @@ class TopicManager
public function subscribe(SubscriberInterface $subscriber, array $topics): void public function subscribe(SubscriberInterface $subscriber, array $topics): void
{ {
$this->subscribers->attach($subscriber);
foreach ($topics as $topic) { foreach ($topics as $topic) {
$this->getTopic($topic)->addSubscriber($subscriber); $this->getTopic($topic)->addSubscriber($subscriber);
} }
$this->subscribers->attach($subscriber);
} }
public function unsubscribe(SubscriberInterface $subscriber, ?array $topics=null): void public function unsubscribe(SubscriberInterface $subscriber, ?array $topics=null): void
{ {
$this->subscribers->detach($subscriber);
if (!$topics) { if (!$topics) {
foreach ($this->topics as $topic) { foreach ($this->topics as $topic) {
$topic->removeSubscriber($subscriber); $topic->removeSubscriber($subscriber);
@ -50,7 +51,6 @@ class TopicManager
foreach ($topics as $topic) { foreach ($topics as $topic) {
$this->getTopic($topic)->removeSubscriber($subscriber); $this->getTopic($topic)->removeSubscriber($subscriber);
} }
$this->subscribers->detach($subscriber);
} }
public function getTopicCount(): int public function getTopicCount(): int