Refactor SpinifyImpl to improve handling of server subscriptions in S…

…pinifySubscriptionMixin
PlugFox · Jun 13, 2024 · 09401a6 · 09401a6
1 parent 55145ea
commit 09401a6
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/lib/src/spinify_impl.dart b/lib/src/spinify_impl.dart
@@ -408,7 +408,7 @@ base mixin SpinifySubscriptionMixin on SpinifyBase, SpinifyCommandMixin {
       final event = reply.event;
       _eventController.add(event);
       if (event is SpinifySubscribe) {
-        // Add server subscription to the registry.
+        // Add server subscription to the registry on subscribe event.
         _serverSubscriptionRegistry.putIfAbsent(
             event.channel,
             () => SpinifyServerSubscriptionImpl(
@@ -424,10 +424,11 @@ base mixin SpinifySubscriptionMixin on SpinifyBase, SpinifyCommandMixin {
           ..onEvent(event)
           ..setState(SpinifySubscriptionState.subscribed());
       } else if (event is SpinifyUnsubscribe) {
-        // Remove server subscription from the registry.
+        // Remove server subscription from the registry on unsubscribe event.
         _serverSubscriptionRegistry.remove(event.channel)
           ?..onEvent(event)
           ..setState(SpinifySubscriptionState.unsubscribed());
+        // Unsubscribe client subscription on unsubscribe event.
         _clientSubscriptionRegistry[event.channel]
           ?..onEvent(event)
           ..setState(SpinifySubscriptionState.unsubscribed());