Skip to content
This repository was archived by the owner on Sep 8, 2024. It is now read-only.

Commit 10e4660

Browse files
committed
Add some safety to remove_all_listers
The same websocket client object is often used between many different parts of the software. (for example skills) This will reduce the risk that all listeners are removed for all events by mistake. It it still possible toi do this by calling `WebsocketClient.ws.remove_all_listers()` if there would be need for this mechanism.
1 parent d7d0dbe commit 10e4660

File tree

1 file changed

+8
-0
lines changed
  • mycroft/messagebus/client

1 file changed

+8
-0
lines changed

mycroft/messagebus/client/ws.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,14 @@ def remove(self, event_name, func):
101101
self.emitter.remove_listener(event_name, func)
102102

103103
def remove_all_listeners(self, event_name):
104+
'''
105+
Remove all listeners connected to event_name.
106+
107+
Args:
108+
event_name: event from which to remove listeners
109+
'''
110+
if event_name is None:
111+
raise ValueError
104112
self.emitter.remove_all_listeners(event_name)
105113

106114
def run_forever(self):

0 commit comments

Comments
 (0)