From f817c233bdf58ae4003b77c8fe028f366869a09e Mon Sep 17 00:00:00 2001
From: Apehum <apehumchik@gmail.com>
Date: Sun, 19 May 2024 04:43:17 +0800
Subject: [PATCH] feat: add methods to remove and clear ServerActivation
 listeners

---
 .../audio/capture/ServerActivation.java       | 57 ++++++++++++++++++-
 .../audio/capture/VoiceServerActivation.kt    | 21 +++++++
 2 files changed, 75 insertions(+), 3 deletions(-)

diff --git a/api/server-proxy-common/src/main/java/su/plo/voice/api/server/audio/capture/ServerActivation.java b/api/server-proxy-common/src/main/java/su/plo/voice/api/server/audio/capture/ServerActivation.java
index fd3bf126..1700968c 100644
--- a/api/server-proxy-common/src/main/java/su/plo/voice/api/server/audio/capture/ServerActivation.java
+++ b/api/server-proxy-common/src/main/java/su/plo/voice/api/server/audio/capture/ServerActivation.java
@@ -119,7 +119,7 @@ default boolean checkPermissions(@NotNull VoicePlayer player) {
     void setProximity(boolean proximity);
 
     /**
-     * Invoked when a player uses the activation and meets all requirements:
+     * Adds a new listener that will be invoked when a player uses the activation and meets all requirements:
      * <ul>
      *     <li>{@link #checkPermissions(VoicePlayer)}</li>
      *     <li>{@link #checkDistance(int)}</li>
@@ -131,7 +131,21 @@ default boolean checkPermissions(@NotNull VoicePlayer player) {
     void onPlayerActivation(@NotNull PlayerActivationListener activationListener);
 
     /**
-     * Invoked when a player starts using the activation and meets all requirements:
+     * Removes a player activation listener.
+     *
+     * @param activationListener The listener.
+     *
+     * @return {@code true} if the listener was removed.
+     */
+    boolean removePlayerActivationListener(@NotNull PlayerActivationListener activationListener);
+
+    /**
+     * Clears all player activation listeners.
+     */
+    void clearPlayerActivationListeners();
+
+    /**
+     * Adds a new listener that will be invoked when a player starts using the activation and meets all requirements:
      * <ul>
      *     <li>{@link #checkPermissions(VoicePlayer)}</li>
      *     <li>{@link #checkDistance(int)}</li>
@@ -143,7 +157,21 @@ default boolean checkPermissions(@NotNull VoicePlayer player) {
     void onPlayerActivationStart(@NotNull ServerActivation.PlayerActivationStartListener activationStartListener);
 
     /**
-     * Invoked when a player sends {@link PlayerAudioEndPacket} and meets all requirements:
+     * Removes a player activation start listener.
+     *
+     * @param activationListener The listener.
+     *
+     * @return {@code true} if the listener was removed.
+     */
+    boolean removePlayerActivationStartListener(@NotNull ServerActivation.PlayerActivationStartListener activationListener);
+
+    /**
+     * Clears all player activation start listeners.
+     */
+    void clearPlayerActivationStartListeners();
+
+    /**
+     * Adds a new listener that will be invoked when a player sends {@link PlayerAudioEndPacket} and meets all requirements:
      * <ul>
      *     <li>{@link #checkPermissions(VoicePlayer)}</li>
      *     <li>{@link #checkDistance(int)}</li>
@@ -155,6 +183,29 @@ default boolean checkPermissions(@NotNull VoicePlayer player) {
      */
     void onPlayerActivationEnd(@NotNull ServerActivation.PlayerActivationEndListener activationEndListener);
 
+    /**
+     * Removes a player activation end listener.
+     *
+     * @param activationListener The listener.
+     *
+     * @return {@code true} if the listener was removed.
+     */
+    boolean removePlayerActivationEndListener(@NotNull ServerActivation.PlayerActivationEndListener activationListener);
+
+    /**
+     * Clears all player activation end listeners.
+     */
+    void clearPlayerActivationEndListeners();
+
+    /**
+     * Clears all activation listeners.
+     */
+    default void clearListeners() {
+        clearPlayerActivationListeners();
+        clearPlayerActivationStartListeners();
+        clearPlayerActivationEndListeners();
+    }
+
     @FunctionalInterface
     interface PlayerActivationListener {
 
diff --git a/server-proxy-common/src/main/kotlin/su/plo/voice/server/audio/capture/VoiceServerActivation.kt b/server-proxy-common/src/main/kotlin/su/plo/voice/server/audio/capture/VoiceServerActivation.kt
index 3fb981cf..04b4157b 100644
--- a/server-proxy-common/src/main/kotlin/su/plo/voice/server/audio/capture/VoiceServerActivation.kt
+++ b/server-proxy-common/src/main/kotlin/su/plo/voice/server/audio/capture/VoiceServerActivation.kt
@@ -98,14 +98,35 @@ class VoiceServerActivation(
         activationListeners.add(activationListener)
     }
 
+    override fun removePlayerActivationListener(activationListener: ServerActivation.PlayerActivationListener): Boolean =
+        activationListeners.remove(activationListener)
+
+    override fun clearPlayerActivationListeners() {
+        activationListeners.clear()
+    }
+
     override fun onPlayerActivationStart(activationStartListener: ServerActivation.PlayerActivationStartListener) {
         activationStartListeners.add(activationStartListener)
     }
 
+    override fun removePlayerActivationStartListener(activationListener: ServerActivation.PlayerActivationStartListener): Boolean =
+        activationStartListeners.remove(activationListener)
+
+    override fun clearPlayerActivationStartListeners() {
+        activationStartListeners.clear()
+    }
+
     override fun onPlayerActivationEnd(activationEndListener: ServerActivation.PlayerActivationEndListener) {
         activationEndListeners.add(activationEndListener)
     }
 
+    override fun removePlayerActivationEndListener(activationListener: ServerActivation.PlayerActivationEndListener): Boolean =
+        activationEndListeners.remove(activationListener)
+
+    override fun clearPlayerActivationEndListeners() {
+        activationEndListeners.clear()
+    }
+
     override fun equals(other: Any?): Boolean {
         return other === this || super.equals(other)
     }