Merge pull request #5 from TheNextLvl-net/create-group-overload

Add overloaded `createGroup` methods

Author: NonSwag

api/src/main/java/net/thenextlvl/perworlds/GroupProvider.java

@@ -106,6 +106,36 @@ public interface GroupProvider {
     @Contract(value = "_, _ -> new", mutates = "this")
     WorldGroup createGroup(String name, Collection<World> worlds) throws IllegalStateException;
 
+    /**
+     * Creates a new {@link WorldGroup} with the specified name, data, and a collection of worlds.
+     * The group must have a unique name and cannot conflict with already existing groups.
+     *
+     * @param name   the name of the group to be created.
+     * @param data   a {@link Consumer} to configure the {@link GroupData} for the new group.
+     * @param worlds an array of {@link World} instances that will be part of the group.
+     * @return the created {@link WorldGroup} instance.
+     * @throws IllegalStateException if a group with the specified name already exists,
+     *                               or if a given world is already part of another group.
+     * @since 1.0.2
+     */
+    @Contract(value = "_, _, _ -> new", mutates = "this")
+    WorldGroup createGroup(String name, Consumer<GroupData> data, World... worlds) throws IllegalStateException;
+
+    /**
+     * Creates a new {@link WorldGroup} with the specified name, data, and a collection of worlds.
+     * The group must have a unique name and cannot conflict with already existing groups.
+     *
+     * @param name   the name of the group to be created.
+     * @param data   a {@link Consumer} to configure the {@link GroupData} for the new group.
+     * @param worlds a collection of {@link World} instances that will be part of the group.
+     * @return the created {@link WorldGroup} instance.
+     * @throws IllegalStateException if a group with the specified name already exists,
+     *                               or if a given world is already part of another group.
+     * @since 1.0.2
+     */
+    @Contract(value = "_, _, _ -> new", mutates = "this")
+    WorldGroup createGroup(String name, Consumer<GroupData> data, Collection<World> worlds) throws IllegalStateException;
+
     /**
      * Creates a new {@link WorldGroup} with the specified name, data, settings, and a collection of worlds.
      * The group must have a unique name and cannot conflict with already existing groups.

build.gradle.kts

@@ -21,7 +21,7 @@ tasks.compileJava {
 }
 
 group = "net.thenextlvl.perworlds"
-version = "1.0.1"
+version = "1.0.2"
 
 repositories {
     mavenCentral()

src/main/java/net/thenextlvl/perworlds/group/PaperGroupProvider.java

@@ -184,7 +184,18 @@ public WorldGroup createGroup(String name, Consumer<GroupData> data, Consumer<Gr
     @Override
     public WorldGroup createGroup(String name, Collection<World> worlds) throws IllegalStateException {
         return createGroup(name, data -> {
-        }, settings -> {
+        }, worlds);
+    }
+
+    @Override
+    public WorldGroup createGroup(String name, Consumer<GroupData> data, World... worlds) throws IllegalStateException {
+        return createGroup(name, data, settings -> {
+        }, worlds);
+    }
+
+    @Override
+    public WorldGroup createGroup(String name, Consumer<GroupData> data, Collection<World> worlds) throws IllegalStateException {
+        return createGroup(name, data, settings -> {
         }, worlds);
     }
 
@@ -196,7 +207,6 @@ public WorldGroup createGroup(String name, Consumer<GroupData> data, Consumer<Gr
     @Override
     public WorldGroup createGroup(String name, World... worlds) throws IllegalStateException {
         return createGroup(name, data -> {
-        }, settings -> {
         }, worlds);
     }