🐛 Fix ListOption.Builder not having listener methods

Author: isXander

src/client/java/dev/isxander/yacl/api/ListOption.java

@@ -7,6 +7,7 @@
 import org.jetbrains.annotations.NotNull;
 
 import java.util.*;
+import java.util.function.BiConsumer;
 import java.util.function.Consumer;
 import java.util.function.Function;
 import java.util.function.Supplier;
@@ -132,6 +133,20 @@ interface Builder<T> {
          */
         Builder<T> collapsed(boolean collapsible);
 
+        /**
+         * Adds a listener to the option. Invoked upon changing any of the list's entries.
+         *
+         * @see Option#addListener(BiConsumer)
+         */
+        ListOption.Builder<T> listener(@NotNull BiConsumer<Option<List<T>>, List<T>> listener);
+
+        /**
+         * Adds multiple listeners to the option. Invoked upon changing of any of the list's entries.
+         *
+         * @see Option#addListener(BiConsumer)
+         */
+        ListOption.Builder<T> listeners(@NotNull Collection<BiConsumer<Option<List<T>>, List<T>>> listeners);
+
         ListOption<T> build();
     }
 }

src/client/java/dev/isxander/yacl/impl/ListOptionImpl.java

@@ -31,7 +31,7 @@ public final class ListOptionImpl<T> implements ListOption<T> {
     private final List<BiConsumer<Option<List<T>>, List<T>>> listeners;
     private final List<Runnable> refreshListeners;
 
-    public ListOptionImpl(@NotNull Component name, @NotNull Component tooltip, @NotNull Binding<List<T>> binding, @NotNull T initialValue, @NotNull Class<T> typeClass, @NotNull Function<ListOptionEntry<T>, Controller<T>> controllerFunction, ImmutableSet<OptionFlag> flags, boolean collapsed, boolean available) {
+    public ListOptionImpl(@NotNull Component name, @NotNull Component tooltip, @NotNull Binding<List<T>> binding, @NotNull T initialValue, @NotNull Class<T> typeClass, @NotNull Function<ListOptionEntry<T>, Controller<T>> controllerFunction, ImmutableSet<OptionFlag> flags, boolean collapsed, boolean available, Collection<BiConsumer<Option<List<T>>, List<T>>> listeners) {
         this.name = name;
         this.tooltip = tooltip;
         this.binding = binding;
@@ -43,6 +43,7 @@ public ListOptionImpl(@NotNull Component name, @NotNull Component tooltip, @NotN
         this.flags = flags;
         this.available = available;
         this.listeners = new ArrayList<>();
+        this.listeners.addAll(listeners);
         this.refreshListeners = new ArrayList<>();
         callListeners();
     }
@@ -219,6 +220,7 @@ public static final class BuilderImpl<T> implements ListOption.Builder<T> {
         private T initialValue;
         private boolean collapsed = false;
         private boolean available = true;
+        private final List<BiConsumer<Option<List<T>>, List<T>>> listeners = new ArrayList<>();
         private final Class<T> typeClass;
 
         public BuilderImpl(Class<T> typeClass) {
@@ -303,6 +305,18 @@ public ListOption.Builder<T> collapsed(boolean collapsible) {
             return this;
         }
 
+        @Override
+        public ListOption.Builder<T> listener(@NotNull BiConsumer<Option<List<T>>, List<T>> listener) {
+            this.listeners.add(listener);
+            return this;
+        }
+
+        @Override
+        public ListOption.Builder<T> listeners(@NotNull Collection<BiConsumer<Option<List<T>>, List<T>>> listeners) {
+            this.listeners.addAll(listeners);
+            return this;
+        }
+
         @Override
         public ListOption<T> build() {
             Validate.notNull(controllerFunction, "`controller` must not be null");
@@ -318,7 +332,7 @@ public ListOption<T> build() {
                 concatenatedTooltip.append(line);
             }
 
-            return new ListOptionImpl<>(name, concatenatedTooltip, binding, initialValue, typeClass, controllerFunction, ImmutableSet.copyOf(flags), collapsed, available);
+            return new ListOptionImpl<>(name, concatenatedTooltip, binding, initialValue, typeClass, controllerFunction, ImmutableSet.copyOf(flags), collapsed, available, listeners);
         }
     }
 }