diff --git a/gradle.properties b/gradle.properties index 8cfc0d9..88d2485 100644 --- a/gradle.properties +++ b/gradle.properties @@ -1,16 +1,16 @@ org.gradle.jvmargs=-Xmx2G # BTA -bta_version=7.3 +bta_version=7.3_01 bta_channel=release # Loader & Mod Menu loader_version=0.15.6-bta.7 # ServerLibe -serverlibe_version=beta.2.7-7.3 +serverlibe_version=beta.2.7-7.3_01 # Mod -mod_version=1.0.0-7.3 +mod_version=1.0.1-7.3_01 mod_group=MelonModding mod_name=MelonUtilities diff --git a/src/main/java/MelonUtilities/command/commandlogic/CommandLogicRollback.java b/src/main/java/MelonUtilities/command/commandlogic/CommandLogicRollback.java index ee41e3f..f2540e0 100644 --- a/src/main/java/MelonUtilities/command/commandlogic/CommandLogicRollback.java +++ b/src/main/java/MelonUtilities/command/commandlogic/CommandLogicRollback.java @@ -26,6 +26,7 @@ import java.io.IOException; import java.nio.file.Files; import java.text.SimpleDateFormat; +import java.time.ZoneId; import java.util.HashMap; import java.util.Map; @@ -56,7 +57,7 @@ public static int rollback(PlayerServer sender) { { ItemStack snapshotIcon = Items.PAPER.getDefaultStack(); SimpleDateFormat sdf = new SimpleDateFormat("MMM/dd/yyyy HH:mm:ss"); - snapshotIcon.setCustomName("Snapshot: [" + sdf.format(capture.getKey()) + "]"); + snapshotIcon.setCustomName("Snapshot: [" + sdf.format(capture.getKey()) + "]" + " (" + ZoneId.systemDefault() + ")"); snapshotIcon.setCustomColor((byte) TextFormatting.LIGHT_BLUE.id); return new ServerSlotButton(snapshotIcon, inventory, finalI, () -> { for(Entity entity : sender.world.loadedEntityList){ @@ -82,7 +83,7 @@ public static int rollback(PlayerServer sender) { { ItemStack backupIcon = Items.BOOK.getDefaultStack(); SimpleDateFormat sdf = new SimpleDateFormat("MMM/dd/yyyy HH:mm:ss"); - backupIcon.setCustomName("Backup: [" + sdf.format(capture.getKey()) + "]"); + backupIcon.setCustomName("Backup: [" + sdf.format(capture.getKey()) + "]" + " (" + ZoneId.systemDefault() + ")"); backupIcon.setCustomColor((byte) TextFormatting.CYAN.id); return new ServerSlotButton(backupIcon, inventory, finalI, () -> { for(Entity entity : sender.world.loadedEntityList){ diff --git a/src/main/java/MelonUtilities/command/commandlogic/CommandLogicTPA.java b/src/main/java/MelonUtilities/command/commandlogic/CommandLogicTPA.java index 718d46b..0008368 100644 --- a/src/main/java/MelonUtilities/command/commandlogic/CommandLogicTPA.java +++ b/src/main/java/MelonUtilities/command/commandlogic/CommandLogicTPA.java @@ -4,13 +4,19 @@ import MelonUtilities.utility.feedback.FeedbackType; import MelonUtilities.utility.managers.TpaManager; import com.mojang.brigadier.Command; +import net.minecraft.server.MinecraftServer; import net.minecraft.server.entity.player.PlayerServer; +import net.minecraft.server.world.WorldServer; public class CommandLogicTPA { public static int tpa(PlayerServer sender, String targetUsername) { - assert sender.world != null; - PlayerServer target = (PlayerServer) sender.world.getPlayerEntityByName(targetUsername); + PlayerServer target = null; + for(WorldServer dimension : MinecraftServer.getInstance().dimensionWorlds.values()){ + if(dimension.getPlayerEntityByName(targetUsername) != null){ + target = (PlayerServer) dimension.getPlayerEntityByName(targetUsername); + } + } if (target != null) { TpaManager.addRequest(sender, target, false); diff --git a/src/main/java/MelonUtilities/command/commandlogic/CommandLogicTPAHere.java b/src/main/java/MelonUtilities/command/commandlogic/CommandLogicTPAHere.java index c3cc6a5..dcaa9b1 100644 --- a/src/main/java/MelonUtilities/command/commandlogic/CommandLogicTPAHere.java +++ b/src/main/java/MelonUtilities/command/commandlogic/CommandLogicTPAHere.java @@ -4,13 +4,19 @@ import MelonUtilities.utility.feedback.FeedbackType; import MelonUtilities.utility.managers.TpaManager; import com.mojang.brigadier.Command; +import net.minecraft.server.MinecraftServer; import net.minecraft.server.entity.player.PlayerServer; +import net.minecraft.server.world.WorldServer; public class CommandLogicTPAHere { public static int tpaHere(PlayerServer sender, String targetUsername){ - assert sender.world != null; - PlayerServer target = (PlayerServer) sender.world.getPlayerEntityByName(targetUsername); + PlayerServer target = null; + for(WorldServer dimension : MinecraftServer.getInstance().dimensionWorlds.values()){ + if(dimension.getPlayerEntityByName(targetUsername) != null){ + target = (PlayerServer) dimension.getPlayerEntityByName(targetUsername); + } + } if(target != null){ TpaManager.addRequest(sender, target, true); diff --git a/src/main/java/MelonUtilities/command/commands/CommandElevator.java b/src/main/java/MelonUtilities/command/commands/CommandElevator.java index 19433bc..bd3ff36 100644 --- a/src/main/java/MelonUtilities/command/commands/CommandElevator.java +++ b/src/main/java/MelonUtilities/command/commands/CommandElevator.java @@ -2,18 +2,18 @@ import MelonUtilities.command.commandlogic.CommandLogicElevator; import com.mojang.brigadier.CommandDispatcher; -import com.mojang.brigadier.arguments.IntegerArgumentType; +import com.mojang.brigadier.arguments.ArgumentTypeInteger; import com.mojang.brigadier.builder.ArgumentBuilder; -import com.mojang.brigadier.builder.LiteralArgumentBuilder; -import com.mojang.brigadier.builder.RequiredArgumentBuilder; +import com.mojang.brigadier.builder.ArgumentBuilderLiteral; +import com.mojang.brigadier.builder.ArgumentBuilderRequired; import net.minecraft.core.net.command.CommandManager; import net.minecraft.core.net.command.CommandSource; import net.minecraft.server.entity.player.PlayerServer; public class CommandElevator implements CommandManager.CommandRegistry{ - public static ArgumentBuilder> elevatorAllowObstructions(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("allowobstructions") + public static ArgumentBuilder> elevatorAllowObstructions(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("allowobstructions") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -24,9 +24,9 @@ public static ArgumentBuilder> elevatorCooldown(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("cooldown") - .then(RequiredArgumentBuilder.argument("cooldownvalue", IntegerArgumentType.integer(0, 256)) + public static ArgumentBuilder> elevatorCooldown(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("cooldown") + .then(ArgumentBuilderRequired.argument("cooldownvalue", ArgumentTypeInteger.integer(0, 256)) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -41,7 +41,7 @@ public static ArgumentBuilder dispatcher) { - LiteralArgumentBuilder builder = LiteralArgumentBuilder.literal("elevator").requires(CommandSource::hasAdmin); + ArgumentBuilderLiteral builder = ArgumentBuilderLiteral.literal("elevator").requires(CommandSource::hasAdmin); elevatorAllowObstructions(builder); elevatorCooldown(builder); diff --git a/src/main/java/MelonUtilities/command/commands/CommandHome.java b/src/main/java/MelonUtilities/command/commands/CommandHome.java index da757ec..698a4be 100644 --- a/src/main/java/MelonUtilities/command/commands/CommandHome.java +++ b/src/main/java/MelonUtilities/command/commands/CommandHome.java @@ -5,18 +5,18 @@ import MelonUtilities.config.Data; import MelonUtilities.config.datatypes.data.Home; import com.mojang.brigadier.CommandDispatcher; -import com.mojang.brigadier.arguments.StringArgumentType; +import com.mojang.brigadier.arguments.ArgumentTypeString; import com.mojang.brigadier.builder.ArgumentBuilder; -import com.mojang.brigadier.builder.LiteralArgumentBuilder; -import com.mojang.brigadier.builder.RequiredArgumentBuilder; +import com.mojang.brigadier.builder.ArgumentBuilderLiteral; +import com.mojang.brigadier.builder.ArgumentBuilderRequired; import net.minecraft.core.net.command.CommandManager; import net.minecraft.core.net.command.CommandSource; import net.minecraft.server.entity.player.PlayerServer; public class CommandHome implements CommandManager.CommandRegistry { - public static ArgumentBuilder> homeTP(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("tp") - .then(RequiredArgumentBuilder.argument("home", ArgumentTypeHome.home()) + public static ArgumentBuilder> homeTP(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("tp") + .then(ArgumentBuilderRequired.argument("home", ArgumentTypeHome.home()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -34,9 +34,9 @@ public static ArgumentBuilder> homeDelete(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("delete") - .then(RequiredArgumentBuilder.argument("home", ArgumentTypeHome.home()) + public static ArgumentBuilder> homeDelete(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("delete") + .then(ArgumentBuilderRequired.argument("home", ArgumentTypeHome.home()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -54,9 +54,9 @@ public static ArgumentBuilder> homeCreate(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("create") - .then(RequiredArgumentBuilder.argument("name", StringArgumentType.string()) + public static ArgumentBuilder> homeCreate(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("create") + .then(ArgumentBuilderRequired.argument("name", ArgumentTypeString.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -69,10 +69,10 @@ public static ArgumentBuilder> homeRename(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("rename") - .then(RequiredArgumentBuilder.argument("home", ArgumentTypeHome.home()) - .then(RequiredArgumentBuilder.argument("name", StringArgumentType.string()) + public static ArgumentBuilder> homeRename(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("rename") + .then(ArgumentBuilderRequired.argument("home", ArgumentTypeHome.home()) + .then(ArgumentBuilderRequired.argument("name", ArgumentTypeString.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -92,8 +92,8 @@ public static ArgumentBuilder> homeList(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("list") + public static ArgumentBuilder> homeList(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("list") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -106,7 +106,7 @@ public static ArgumentBuilder dispatcher) { - LiteralArgumentBuilder builder = LiteralArgumentBuilder.literal("home"); + ArgumentBuilderLiteral builder = ArgumentBuilderLiteral.literal("home"); homeTP(builder); homeDelete(builder); diff --git a/src/main/java/MelonUtilities/command/commands/CommandLock.java b/src/main/java/MelonUtilities/command/commands/CommandLock.java index 72ee6e5..b6c790d 100644 --- a/src/main/java/MelonUtilities/command/commands/CommandLock.java +++ b/src/main/java/MelonUtilities/command/commands/CommandLock.java @@ -4,16 +4,16 @@ import MelonUtilities.command.commandlogic.CommandLogicLock; import com.mojang.brigadier.CommandDispatcher; import com.mojang.brigadier.builder.ArgumentBuilder; -import com.mojang.brigadier.builder.LiteralArgumentBuilder; -import com.mojang.brigadier.builder.RequiredArgumentBuilder; +import com.mojang.brigadier.builder.ArgumentBuilderLiteral; +import com.mojang.brigadier.builder.ArgumentBuilderRequired; import net.minecraft.core.net.command.CommandManager; import net.minecraft.core.net.command.CommandSource; import net.minecraft.server.entity.player.PlayerServer; @SuppressWarnings("UnusedReturnValue") public class CommandLock implements CommandManager.CommandRegistry{ - public static ArgumentBuilder> lockOnBlockPlaced(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("onblockplaced") + public static ArgumentBuilder> lockOnBlockPlaced(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("onblockplaced") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -24,8 +24,8 @@ public static ArgumentBuilder> lockOnBlockPunched(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("onblockpunched") + public static ArgumentBuilder> lockOnBlockPunched(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("onblockpunched") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -36,9 +36,9 @@ public static ArgumentBuilder> lockTrust(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("trust") - .then(RequiredArgumentBuilder.argument("username", ArgumentTypeUsername.string()) + public static ArgumentBuilder> lockTrust(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("trust") + .then(ArgumentBuilderRequired.argument("username", ArgumentTypeUsername.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -51,9 +51,9 @@ public static ArgumentBuilder> lockTrustAll(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("trustall") - .then(RequiredArgumentBuilder.argument("username", ArgumentTypeUsername.string()) + public static ArgumentBuilder> lockTrustAll(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("trustall") + .then(ArgumentBuilderRequired.argument("username", ArgumentTypeUsername.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -66,8 +66,8 @@ public static ArgumentBuilder> lockTrustCommunity(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("trustcommunity") + public static ArgumentBuilder> lockTrustCommunity(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("trustcommunity") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -78,9 +78,9 @@ public static ArgumentBuilder> lockUntrust(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("untrust") - .then(RequiredArgumentBuilder.argument("username", ArgumentTypeUsername.string()) + public static ArgumentBuilder> lockUntrust(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("untrust") + .then(ArgumentBuilderRequired.argument("username", ArgumentTypeUsername.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -93,9 +93,9 @@ public static ArgumentBuilder> lockUntrustAll(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("untrustall") - .then(RequiredArgumentBuilder.argument("username", ArgumentTypeUsername.string()) + public static ArgumentBuilder> lockUntrustAll(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("untrustall") + .then(ArgumentBuilderRequired.argument("username", ArgumentTypeUsername.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -108,8 +108,8 @@ public static ArgumentBuilder> lockUntrustCommunity(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("untrustcommunity") + public static ArgumentBuilder> lockUntrustCommunity(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("untrustcommunity") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -120,8 +120,8 @@ public static ArgumentBuilder> lockBypass(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("bypass").requires(CommandSource::hasAdmin) + public static ArgumentBuilder> lockBypass(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("bypass").requires(CommandSource::hasAdmin) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -132,8 +132,8 @@ public static ArgumentBuilder> lockInfo(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("info") + public static ArgumentBuilder> lockInfo(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("info") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -144,7 +144,7 @@ public static ArgumentBuilder> lock(ArgumentBuilder> builder) { + public static ArgumentBuilder> lock(ArgumentBuilder> builder) { builder.executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -157,7 +157,7 @@ public static ArgumentBuilder dispatcher) { - LiteralArgumentBuilder builder = LiteralArgumentBuilder.literal("lock"); + ArgumentBuilderLiteral builder = ArgumentBuilderLiteral.literal("lock"); lock(builder); lockOnBlockPlaced(builder); diff --git a/src/main/java/MelonUtilities/command/commands/CommandMelonUtilities.java b/src/main/java/MelonUtilities/command/commands/CommandMelonUtilities.java index bc13f86..d6af502 100644 --- a/src/main/java/MelonUtilities/command/commands/CommandMelonUtilities.java +++ b/src/main/java/MelonUtilities/command/commands/CommandMelonUtilities.java @@ -2,7 +2,7 @@ import MelonUtilities.command.commandlogic.CommandLogicMelonUtilities; import com.mojang.brigadier.CommandDispatcher; -import com.mojang.brigadier.builder.LiteralArgumentBuilder; +import com.mojang.brigadier.builder.ArgumentBuilderLiteral; import net.minecraft.core.net.command.CommandManager; import net.minecraft.core.net.command.CommandSource; import net.minecraft.server.entity.player.PlayerServer; @@ -10,8 +10,8 @@ public class CommandMelonUtilities implements CommandManager.CommandRegistry{ @Override public void register(CommandDispatcher dispatcher) { - dispatcher.register((LiteralArgumentBuilder.literal("melonutilities").requires(CommandSource::hasAdmin)) - .then(LiteralArgumentBuilder.literal("reload") + dispatcher.register((ArgumentBuilderLiteral.literal("melonutilities").requires(CommandSource::hasAdmin)) + .then(ArgumentBuilderLiteral.literal("reload") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); diff --git a/src/main/java/MelonUtilities/command/commands/CommandRole.java b/src/main/java/MelonUtilities/command/commands/CommandRole.java index a239b39..94c1ab4 100644 --- a/src/main/java/MelonUtilities/command/commands/CommandRole.java +++ b/src/main/java/MelonUtilities/command/commands/CommandRole.java @@ -6,12 +6,12 @@ import MelonUtilities.config.datatypes.data.Role; import MelonUtilities.utility.MUtil; import com.mojang.brigadier.CommandDispatcher; -import com.mojang.brigadier.arguments.BoolArgumentType; -import com.mojang.brigadier.arguments.IntegerArgumentType; -import com.mojang.brigadier.arguments.StringArgumentType; +import com.mojang.brigadier.arguments.ArgumentTypeBool; +import com.mojang.brigadier.arguments.ArgumentTypeInteger; +import com.mojang.brigadier.arguments.ArgumentTypeString; import com.mojang.brigadier.builder.ArgumentBuilder; -import com.mojang.brigadier.builder.LiteralArgumentBuilder; -import com.mojang.brigadier.builder.RequiredArgumentBuilder; +import com.mojang.brigadier.builder.ArgumentBuilderLiteral; +import com.mojang.brigadier.builder.ArgumentBuilderRequired; import net.minecraft.core.entity.player.Player; import net.minecraft.core.net.command.CommandManager; import net.minecraft.core.net.command.CommandSource; @@ -22,10 +22,10 @@ @SuppressWarnings("UnusedReturnValue") public class CommandRole implements CommandManager.CommandRegistry{ - public static ArgumentBuilder> roleCreate(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("create") - .then(RequiredArgumentBuilder.argument("roleID", StringArgumentType.string()) - .then(RequiredArgumentBuilder.argument("priorityvalue", IntegerArgumentType.integer(0, 99)) + public static ArgumentBuilder> roleCreate(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("create") + .then(ArgumentBuilderRequired.argument("roleID", ArgumentTypeString.string()) + .then(ArgumentBuilderRequired.argument("priorityvalue", ArgumentTypeInteger.integer(0, 99)) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -40,9 +40,9 @@ public static ArgumentBuilder> roleDelete(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("delete") - .then(RequiredArgumentBuilder.argument("role", ArgumentTypeRole.role()) + public static ArgumentBuilder> roleDelete(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("delete") + .then(ArgumentBuilderRequired.argument("role", ArgumentTypeRole.role()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -55,11 +55,11 @@ public static ArgumentBuilder> roleEdit(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("edit") - .then(RequiredArgumentBuilder.argument("role", ArgumentTypeRole.role()) - .then(LiteralArgumentBuilder.literal("priority") - .then(RequiredArgumentBuilder.argument("priorityvalue", IntegerArgumentType.integer(0, 99)) + public static ArgumentBuilder> roleEdit(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("edit") + .then(ArgumentBuilderRequired.argument("role", ArgumentTypeRole.role()) + .then(ArgumentBuilderLiteral.literal("priority") + .then(ArgumentBuilderRequired.argument("priorityvalue", ArgumentTypeInteger.integer(0, 99)) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -70,9 +70,9 @@ public static ArgumentBuilderliteral("display") - .then(LiteralArgumentBuilder.literal("name") - .then(RequiredArgumentBuilder.argument("displayname", StringArgumentType.greedyString()) + .then(ArgumentBuilderLiteral.literal("display") + .then(ArgumentBuilderLiteral.literal("name") + .then(ArgumentBuilderRequired.argument("displayname", ArgumentTypeString.greedyString()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -83,8 +83,8 @@ public static ArgumentBuilderliteral("color") - .then(RequiredArgumentBuilder.argument("color", ArgumentTypeColor.color()) + .then(ArgumentBuilderLiteral.literal("color") + .then(ArgumentBuilderRequired.argument("color", ArgumentTypeColor.color()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -94,7 +94,7 @@ public static ArgumentBuilderargument("hex", StringArgumentType.string()) + .then(ArgumentBuilderRequired.argument("hex", ArgumentTypeString.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -105,8 +105,8 @@ public static ArgumentBuilderliteral("underline") - .then(RequiredArgumentBuilder.argument("value", BoolArgumentType.bool()) + .then(ArgumentBuilderLiteral.literal("underline") + .then(ArgumentBuilderRequired.argument("value", ArgumentTypeBool.bool()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -117,8 +117,8 @@ public static ArgumentBuilderliteral("bold") - .then(RequiredArgumentBuilder.argument("value", BoolArgumentType.bool()) + .then(ArgumentBuilderLiteral.literal("bold") + .then(ArgumentBuilderRequired.argument("value", ArgumentTypeBool.bool()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -129,8 +129,8 @@ public static ArgumentBuilderliteral("italics") - .then(RequiredArgumentBuilder.argument("value", BoolArgumentType.bool()) + .then(ArgumentBuilderLiteral.literal("italics") + .then(ArgumentBuilderRequired.argument("value", ArgumentTypeBool.bool()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -141,9 +141,9 @@ public static ArgumentBuilderliteral("border") - .then(LiteralArgumentBuilder.literal("color") - .then(RequiredArgumentBuilder.argument("color", ArgumentTypeColor.color()) + .then(ArgumentBuilderLiteral.literal("border") + .then(ArgumentBuilderLiteral.literal("color") + .then(ArgumentBuilderRequired.argument("color", ArgumentTypeColor.color()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -153,7 +153,7 @@ public static ArgumentBuilderargument("hex", StringArgumentType.string()) + .then(ArgumentBuilderRequired.argument("hex", ArgumentTypeString.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -164,8 +164,8 @@ public static ArgumentBuilderliteral("style") - .then(LiteralArgumentBuilder.literal("none") + .then(ArgumentBuilderLiteral.literal("style") + .then(ArgumentBuilderLiteral.literal("none") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -174,7 +174,7 @@ public static ArgumentBuilderliteral("bracket") + .then(ArgumentBuilderLiteral.literal("bracket") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -183,7 +183,7 @@ public static ArgumentBuilderliteral("curly") + .then(ArgumentBuilderLiteral.literal("curly") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -192,7 +192,7 @@ public static ArgumentBuilderliteral("caret") + .then(ArgumentBuilderLiteral.literal("caret") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -201,9 +201,9 @@ public static ArgumentBuilderliteral("custom") - .then(LiteralArgumentBuilder.literal("prefix") - .then(RequiredArgumentBuilder.argument("customaffix", StringArgumentType.greedyString()) + .then(ArgumentBuilderLiteral.literal("custom") + .then(ArgumentBuilderLiteral.literal("prefix") + .then(ArgumentBuilderRequired.argument("customaffix", ArgumentTypeString.greedyString()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -214,8 +214,8 @@ public static ArgumentBuilderliteral("suffix") - .then(RequiredArgumentBuilder.argument("customaffix", StringArgumentType.greedyString()) + .then(ArgumentBuilderLiteral.literal("suffix") + .then(ArgumentBuilderRequired.argument("customaffix", ArgumentTypeString.greedyString()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -230,10 +230,10 @@ public static ArgumentBuilderliteral("username") - .then(LiteralArgumentBuilder.literal("border") - .then(LiteralArgumentBuilder.literal("color") - .then(RequiredArgumentBuilder.argument("color", ArgumentTypeColor.color()) + .then(ArgumentBuilderLiteral.literal("username") + .then(ArgumentBuilderLiteral.literal("border") + .then(ArgumentBuilderLiteral.literal("color") + .then(ArgumentBuilderRequired.argument("color", ArgumentTypeColor.color()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -243,7 +243,7 @@ public static ArgumentBuilderargument("hex", StringArgumentType.string()) + .then(ArgumentBuilderRequired.argument("hex", ArgumentTypeString.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -254,8 +254,8 @@ public static ArgumentBuilderliteral("style") - .then(LiteralArgumentBuilder.literal("none") + .then(ArgumentBuilderLiteral.literal("style") + .then(ArgumentBuilderLiteral.literal("none") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -264,7 +264,7 @@ public static ArgumentBuilderliteral("bracket") + .then(ArgumentBuilderLiteral.literal("bracket") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -273,7 +273,7 @@ public static ArgumentBuilderliteral("curly") + .then(ArgumentBuilderLiteral.literal("curly") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -282,7 +282,7 @@ public static ArgumentBuilderliteral("caret") + .then(ArgumentBuilderLiteral.literal("caret") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -291,9 +291,9 @@ public static ArgumentBuilderliteral("custom") - .then(LiteralArgumentBuilder.literal("prefix") - .then(RequiredArgumentBuilder.argument("customaffix", StringArgumentType.greedyString()) + .then(ArgumentBuilderLiteral.literal("custom") + .then(ArgumentBuilderLiteral.literal("prefix") + .then(ArgumentBuilderRequired.argument("customaffix", ArgumentTypeString.greedyString()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -304,8 +304,8 @@ public static ArgumentBuilderliteral("suffix") - .then(RequiredArgumentBuilder.argument("customaffix", StringArgumentType.greedyString()) + .then(ArgumentBuilderLiteral.literal("suffix") + .then(ArgumentBuilderRequired.argument("customaffix", ArgumentTypeString.greedyString()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -320,9 +320,9 @@ public static ArgumentBuilderliteral("text") - .then(LiteralArgumentBuilder.literal("color") - .then(RequiredArgumentBuilder.argument("color", ArgumentTypeColor.color()) + .then(ArgumentBuilderLiteral.literal("text") + .then(ArgumentBuilderLiteral.literal("color") + .then(ArgumentBuilderRequired.argument("color", ArgumentTypeColor.color()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -332,7 +332,7 @@ public static ArgumentBuilderargument("hex", StringArgumentType.string()) + .then(ArgumentBuilderRequired.argument("hex", ArgumentTypeString.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -343,8 +343,8 @@ public static ArgumentBuilderliteral("underline") - .then(RequiredArgumentBuilder.argument("value", BoolArgumentType.bool()) + .then(ArgumentBuilderLiteral.literal("underline") + .then(ArgumentBuilderRequired.argument("value", ArgumentTypeBool.bool()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -355,8 +355,8 @@ public static ArgumentBuilderliteral("bold") - .then(RequiredArgumentBuilder.argument("value", BoolArgumentType.bool()) + .then(ArgumentBuilderLiteral.literal("bold") + .then(ArgumentBuilderRequired.argument("value", ArgumentTypeBool.bool()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -367,8 +367,8 @@ public static ArgumentBuilderliteral("italics") - .then(RequiredArgumentBuilder.argument("value", BoolArgumentType.bool()) + .then(ArgumentBuilderLiteral.literal("italics") + .then(ArgumentBuilderRequired.argument("value", ArgumentTypeBool.bool()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -385,10 +385,10 @@ public static ArgumentBuilder> roleGrant(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("grant") - .then(RequiredArgumentBuilder.argument("role", ArgumentTypeRole.role()) - .then(RequiredArgumentBuilder.argument("target", ArgumentTypeEntity.player()) + public static ArgumentBuilder> roleGrant(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("grant") + .then(ArgumentBuilderRequired.argument("role", ArgumentTypeRole.role()) + .then(ArgumentBuilderRequired.argument("target", ArgumentTypeEntity.username()) .executes( context -> { @@ -404,10 +404,10 @@ public static ArgumentBuilder> roleRevoke(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("revoke") - .then(RequiredArgumentBuilder.argument("role", ArgumentTypeRole.role()) - .then(RequiredArgumentBuilder.argument("target", ArgumentTypeEntity.player()) + public static ArgumentBuilder> roleRevoke(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("revoke") + .then(ArgumentBuilderRequired.argument("role", ArgumentTypeRole.role()) + .then(ArgumentBuilderRequired.argument("target", ArgumentTypeEntity.username()) .executes(context -> { Role role = context.getArgument("role", Role.class); @@ -422,10 +422,10 @@ public static ArgumentBuilder> roleSet(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("set") - .then(LiteralArgumentBuilder.literal("defaultrole") - .then(RequiredArgumentBuilder.argument("role", ArgumentTypeRole.role()) + public static ArgumentBuilder> roleSet(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("set") + .then(ArgumentBuilderLiteral.literal("defaultrole") + .then(ArgumentBuilderRequired.argument("role", ArgumentTypeRole.role()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -434,7 +434,7 @@ public static ArgumentBuilderliteral("none") + .then(ArgumentBuilderLiteral.literal("none") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -443,8 +443,8 @@ public static ArgumentBuilderliteral("displaymode") - .then(LiteralArgumentBuilder.literal("single") + .then(ArgumentBuilderLiteral.literal("displaymode") + .then(ArgumentBuilderLiteral.literal("single") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -452,7 +452,7 @@ public static ArgumentBuilderliteral("multi") + .then(ArgumentBuilderLiteral.literal("multi") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -465,8 +465,8 @@ public static ArgumentBuilder> roleList(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("list") + public static ArgumentBuilder> roleList(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("list") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -477,8 +477,8 @@ public static ArgumentBuilder> roleReload(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("reload") + public static ArgumentBuilder> roleReload(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("reload") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -489,7 +489,7 @@ public static ArgumentBuilder> role(ArgumentBuilder> builder) { +/* public static ArgumentBuilder> role(ArgumentBuilder> builder) { builder.executes( context -> CommandLogicRole.role((PlayerServer) context.getSource().getSender()) ); @@ -498,7 +498,7 @@ public static ArgumentBuilder dispatcher) { - LiteralArgumentBuilder commandBuilder = LiteralArgumentBuilder.literal("role").requires(CommandSource::hasAdmin); + ArgumentBuilderLiteral commandBuilder = ArgumentBuilderLiteral.literal("role").requires(CommandSource::hasAdmin); //role(commandBuilder); roleCreate(commandBuilder); diff --git a/src/main/java/MelonUtilities/command/commands/CommandRollback.java b/src/main/java/MelonUtilities/command/commands/CommandRollback.java index 5009323..10fd9e3 100644 --- a/src/main/java/MelonUtilities/command/commands/CommandRollback.java +++ b/src/main/java/MelonUtilities/command/commands/CommandRollback.java @@ -3,8 +3,8 @@ import MelonUtilities.command.commandlogic.CommandLogicRollback; import com.mojang.brigadier.CommandDispatcher; import com.mojang.brigadier.builder.ArgumentBuilder; -import com.mojang.brigadier.builder.LiteralArgumentBuilder; -import com.mojang.brigadier.builder.RequiredArgumentBuilder; +import com.mojang.brigadier.builder.ArgumentBuilderLiteral; +import com.mojang.brigadier.builder.ArgumentBuilderRequired; import net.minecraft.core.net.command.CommandManager; import net.minecraft.core.net.command.CommandSource; import net.minecraft.core.net.command.arguments.ArgumentTypeChunkCoordinates; @@ -13,7 +13,7 @@ @SuppressWarnings("UnusedReturnValue") public class CommandRollback implements CommandManager.CommandRegistry{ - public static ArgumentBuilder> rollback(ArgumentBuilder> builder) { + public static ArgumentBuilder> rollback(ArgumentBuilder> builder) { builder.requires(CommandSource::hasAdmin).executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -23,10 +23,10 @@ public static ArgumentBuilder> rollbackArea(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("area") - .then(RequiredArgumentBuilder.argument("x1 z1", ArgumentTypeChunkCoordinates.chunkCoordinates()) - .then(RequiredArgumentBuilder.argument("x2 z2", ArgumentTypeChunkCoordinates.chunkCoordinates()) + public static ArgumentBuilder> rollbackArea(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("area") + .then(ArgumentBuilderRequired.argument("x1 z1", ArgumentTypeChunkCoordinates.chunkCoordinates()) + .then(ArgumentBuilderRequired.argument("x2 z2", ArgumentTypeChunkCoordinates.chunkCoordinates()) .executes(context -> { CommandSource source = context.getSource(); @@ -47,8 +47,8 @@ public static ArgumentBuilder> rollbackTakeSnapshot(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("takesnapshot") + public static ArgumentBuilder> rollbackTakeSnapshot(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("takesnapshot") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -59,8 +59,8 @@ public static ArgumentBuilder> rollbackTakeBackup(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("takebackup") + public static ArgumentBuilder> rollbackTakeBackup(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("takebackup") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -71,8 +71,8 @@ public static ArgumentBuilder> rollbackPruneSnapshots(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("prunesnapshots") + public static ArgumentBuilder> rollbackPruneSnapshots(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("prunesnapshots") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -83,8 +83,8 @@ public static ArgumentBuilder> rollbackPruneBackups(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("prunebackups") + public static ArgumentBuilder> rollbackPruneBackups(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("prunebackups") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -95,8 +95,8 @@ public static ArgumentBuilder> rollbackAutoSnapshots(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("autosnapshots") + public static ArgumentBuilder> rollbackAutoSnapshots(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("autosnapshots") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -107,8 +107,8 @@ public static ArgumentBuilder> rollbackAutoBackups(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("autobackups") + public static ArgumentBuilder> rollbackAutoBackups(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("autobackups") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -122,7 +122,7 @@ public static ArgumentBuilder dispatcher) { - LiteralArgumentBuilder builder = LiteralArgumentBuilder.literal("rollback"); + ArgumentBuilderLiteral builder = ArgumentBuilderLiteral.literal("rollback"); rollback(builder); rollbackArea(builder); diff --git a/src/main/java/MelonUtilities/command/commands/CommandTPA.java b/src/main/java/MelonUtilities/command/commands/CommandTPA.java index faed180..bb4d5ec 100644 --- a/src/main/java/MelonUtilities/command/commands/CommandTPA.java +++ b/src/main/java/MelonUtilities/command/commands/CommandTPA.java @@ -4,8 +4,8 @@ import MelonUtilities.command.commandlogic.CommandLogicTPA; import com.mojang.brigadier.CommandDispatcher; import com.mojang.brigadier.builder.ArgumentBuilder; -import com.mojang.brigadier.builder.LiteralArgumentBuilder; -import com.mojang.brigadier.builder.RequiredArgumentBuilder; +import com.mojang.brigadier.builder.ArgumentBuilderLiteral; +import com.mojang.brigadier.builder.ArgumentBuilderRequired; import net.minecraft.core.net.command.CommandManager; import net.minecraft.core.net.command.CommandSource; import net.minecraft.server.entity.player.PlayerServer; @@ -13,8 +13,8 @@ @SuppressWarnings("UnusedReturnValue") public class CommandTPA implements CommandManager.CommandRegistry{ - public static ArgumentBuilder> tpa(ArgumentBuilder> builder) { - builder.then(RequiredArgumentBuilder.argument("target", ArgumentTypeUsername.string()) + public static ArgumentBuilder> tpa(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderRequired.argument("target", ArgumentTypeUsername.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -28,7 +28,7 @@ public static ArgumentBuilder dispatcher) { - LiteralArgumentBuilder builder = LiteralArgumentBuilder.literal("tpa"); + ArgumentBuilderLiteral builder = ArgumentBuilderLiteral.literal("tpa"); tpa(builder); diff --git a/src/main/java/MelonUtilities/command/commands/CommandTPAHere.java b/src/main/java/MelonUtilities/command/commands/CommandTPAHere.java index f969ac7..0cc6a85 100644 --- a/src/main/java/MelonUtilities/command/commands/CommandTPAHere.java +++ b/src/main/java/MelonUtilities/command/commands/CommandTPAHere.java @@ -4,8 +4,8 @@ import MelonUtilities.command.commandlogic.CommandLogicTPAHere; import com.mojang.brigadier.CommandDispatcher; import com.mojang.brigadier.builder.ArgumentBuilder; -import com.mojang.brigadier.builder.LiteralArgumentBuilder; -import com.mojang.brigadier.builder.RequiredArgumentBuilder; +import com.mojang.brigadier.builder.ArgumentBuilderLiteral; +import com.mojang.brigadier.builder.ArgumentBuilderRequired; import net.minecraft.core.net.command.CommandManager; import net.minecraft.core.net.command.CommandSource; import net.minecraft.server.entity.player.PlayerServer; @@ -13,8 +13,8 @@ @SuppressWarnings("UnusedReturnValue") public class CommandTPAHere implements CommandManager.CommandRegistry{ - public static ArgumentBuilder> tpahere(ArgumentBuilder> builder) { - builder.then(RequiredArgumentBuilder.argument("target", ArgumentTypeUsername.string()) + public static ArgumentBuilder> tpahere(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderRequired.argument("target", ArgumentTypeUsername.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -28,7 +28,7 @@ public static ArgumentBuilder dispatcher) { - LiteralArgumentBuilder builder = LiteralArgumentBuilder.literal("tpahere"); + ArgumentBuilderLiteral builder = ArgumentBuilderLiteral.literal("tpahere"); tpahere(builder); diff --git a/src/main/java/MelonUtilities/command/commands/CommandTPAccept.java b/src/main/java/MelonUtilities/command/commands/CommandTPAccept.java index c00704d..97b727d 100644 --- a/src/main/java/MelonUtilities/command/commands/CommandTPAccept.java +++ b/src/main/java/MelonUtilities/command/commands/CommandTPAccept.java @@ -3,7 +3,7 @@ import MelonUtilities.command.commandlogic.CommandLogicTPA; import com.mojang.brigadier.CommandDispatcher; import com.mojang.brigadier.builder.ArgumentBuilder; -import com.mojang.brigadier.builder.LiteralArgumentBuilder; +import com.mojang.brigadier.builder.ArgumentBuilderLiteral; import net.minecraft.core.net.command.CommandManager; import net.minecraft.core.net.command.CommandSource; import net.minecraft.server.entity.player.PlayerServer; @@ -11,7 +11,7 @@ @SuppressWarnings("UnusedReturnValue") public class CommandTPAccept implements CommandManager.CommandRegistry{ - public static ArgumentBuilder> tpaccept(ArgumentBuilder> builder) { + public static ArgumentBuilder> tpaccept(ArgumentBuilder> builder) { builder.executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -23,7 +23,7 @@ public static ArgumentBuilder dispatcher) { - LiteralArgumentBuilder builder = LiteralArgumentBuilder.literal("tpaccept"); + ArgumentBuilderLiteral builder = ArgumentBuilderLiteral.literal("tpaccept"); tpaccept(builder); diff --git a/src/main/java/MelonUtilities/command/commands/CommandTPDeny.java b/src/main/java/MelonUtilities/command/commands/CommandTPDeny.java index 9a5237a..04810c3 100644 --- a/src/main/java/MelonUtilities/command/commands/CommandTPDeny.java +++ b/src/main/java/MelonUtilities/command/commands/CommandTPDeny.java @@ -3,7 +3,7 @@ import MelonUtilities.command.commandlogic.CommandLogicTPA; import com.mojang.brigadier.CommandDispatcher; import com.mojang.brigadier.builder.ArgumentBuilder; -import com.mojang.brigadier.builder.LiteralArgumentBuilder; +import com.mojang.brigadier.builder.ArgumentBuilderLiteral; import net.minecraft.core.net.command.CommandManager; import net.minecraft.core.net.command.CommandSource; import net.minecraft.server.entity.player.PlayerServer; @@ -11,7 +11,7 @@ @SuppressWarnings("UnusedReturnValue") public class CommandTPDeny implements CommandManager.CommandRegistry{ - public static ArgumentBuilder> tpdeny(ArgumentBuilder> builder) { + public static ArgumentBuilder> tpdeny(ArgumentBuilder> builder) { builder.executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -23,7 +23,7 @@ public static ArgumentBuilder dispatcher) { - LiteralArgumentBuilder builder = LiteralArgumentBuilder.literal("tpdeny"); + ArgumentBuilderLiteral builder = ArgumentBuilderLiteral.literal("tpdeny"); tpdeny(builder); diff --git a/src/main/java/MelonUtilities/command/commands/CommandWarp.java b/src/main/java/MelonUtilities/command/commands/CommandWarp.java index 41e87be..cd78e12 100644 --- a/src/main/java/MelonUtilities/command/commands/CommandWarp.java +++ b/src/main/java/MelonUtilities/command/commands/CommandWarp.java @@ -4,19 +4,19 @@ import MelonUtilities.command.commandlogic.CommandLogicWarp; import MelonUtilities.config.datatypes.data.Warp; import com.mojang.brigadier.CommandDispatcher; -import com.mojang.brigadier.arguments.StringArgumentType; +import com.mojang.brigadier.arguments.ArgumentTypeString; import com.mojang.brigadier.builder.ArgumentBuilder; -import com.mojang.brigadier.builder.LiteralArgumentBuilder; -import com.mojang.brigadier.builder.RequiredArgumentBuilder; +import com.mojang.brigadier.builder.ArgumentBuilderLiteral; +import com.mojang.brigadier.builder.ArgumentBuilderRequired; import net.minecraft.core.net.command.CommandManager; import net.minecraft.core.net.command.CommandSource; import net.minecraft.server.entity.player.PlayerServer; @SuppressWarnings("UnusedReturnValue") public class CommandWarp implements CommandManager.CommandRegistry{ - public static ArgumentBuilder> warpTP(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("tp") - .then(RequiredArgumentBuilder.argument("warp", ArgumentTypeWarp.warp()) + public static ArgumentBuilder> warpTP(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("tp") + .then(ArgumentBuilderRequired.argument("warp", ArgumentTypeWarp.warp()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -29,9 +29,9 @@ public static ArgumentBuilder> warpDelete(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("delete").requires(CommandSource::hasAdmin) - .then(RequiredArgumentBuilder.argument("warp", ArgumentTypeWarp.warp()) + public static ArgumentBuilder> warpDelete(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("delete").requires(CommandSource::hasAdmin) + .then(ArgumentBuilderRequired.argument("warp", ArgumentTypeWarp.warp()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -44,9 +44,9 @@ public static ArgumentBuilder> warpCreate(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("create").requires(CommandSource::hasAdmin) - .then(RequiredArgumentBuilder.argument("name", StringArgumentType.string()) + public static ArgumentBuilder> warpCreate(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("create").requires(CommandSource::hasAdmin) + .then(ArgumentBuilderRequired.argument("name", ArgumentTypeString.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -59,10 +59,10 @@ public static ArgumentBuilder> warpRename(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("rename").requires(CommandSource::hasAdmin) - .then(RequiredArgumentBuilder.argument("warp", ArgumentTypeWarp.warp()) - .then(RequiredArgumentBuilder.argument("name", StringArgumentType.string()) + public static ArgumentBuilder> warpRename(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("rename").requires(CommandSource::hasAdmin) + .then(ArgumentBuilderRequired.argument("warp", ArgumentTypeWarp.warp()) + .then(ArgumentBuilderRequired.argument("name", ArgumentTypeString.string()) .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -77,8 +77,8 @@ public static ArgumentBuilder> warpList(ArgumentBuilder> builder) { - builder.then(LiteralArgumentBuilder.literal("list") + public static ArgumentBuilder> warpList(ArgumentBuilder> builder) { + builder.then(ArgumentBuilderLiteral.literal("list") .executes(context -> { PlayerServer sender = (PlayerServer) context.getSource().getSender(); if(sender == null){return 0;} @@ -91,7 +91,7 @@ public static ArgumentBuilder dispatcher) { - LiteralArgumentBuilder builder = LiteralArgumentBuilder.literal("warp"); + ArgumentBuilderLiteral builder = ArgumentBuilderLiteral.literal("warp"); warpTP(builder); warpDelete(builder); diff --git a/src/main/java/MelonUtilities/config/datatypes/jsonadapters/ConfigJsonAdapter.java b/src/main/java/MelonUtilities/config/datatypes/jsonadapters/ConfigJsonAdapter.java index 0077344..049cbf2 100644 --- a/src/main/java/MelonUtilities/config/datatypes/jsonadapters/ConfigJsonAdapter.java +++ b/src/main/java/MelonUtilities/config/datatypes/jsonadapters/ConfigJsonAdapter.java @@ -28,7 +28,7 @@ public Config deserialize(JsonElement json, Type typeOfT, JsonDeserializationCon JsonObject rollbackConfig = obj.getAsJsonObject("Rollback Config"); JsonObject elevatorConfig = obj.getAsJsonObject("Elevator Config"); JsonObject sqlLogConfig = obj.getAsJsonObject("SQL Log Config"); - JsonObject warpConfig = obj.getAsJsonObject("Warp Config"); + JsonObject warpConfig = obj.getAsJsonObject("Warp Data"); JsonObject discordIntegrationConfig = obj.getAsJsonObject("Discord Integration Config"); diff --git a/src/main/java/MelonUtilities/mixins/PacketHandlerServerMixin.java b/src/main/java/MelonUtilities/mixins/PacketHandlerServerMixin.java index bb83ec0..423cf24 100644 --- a/src/main/java/MelonUtilities/mixins/PacketHandlerServerMixin.java +++ b/src/main/java/MelonUtilities/mixins/PacketHandlerServerMixin.java @@ -202,8 +202,31 @@ private void handleBlockDigInject(PacketPlayerAction packet, CallbackInfo ci){ && !Data.Users.getOrCreate(player.uuid).lockBypass){ ci.cancel(); sendPacket(new PacketBlockUpdate(packet.xPosition, packet.yPosition, packet.zPosition, world)); + return; } - if(packet.action == PacketPlayerAction.ACTION_DIG_CONTINUED && Data.Users.getOrCreate(player.uuid).lockOnBlockPunched && !lockable.getIsLocked()){ + + if (packet.action == PacketPlayerAction.ACTION_DIG_START + && Data.Users.getOrCreate(player.uuid).lockOnBlockPunched + && lockable.getIsLocked() + && !lockable.getLockOwner().equals(player.uuid)) + { + FeedbackHandlerServer.sendFeedback(FeedbackType.error, player, "Failed to Lock Container! (Not Owned By You)"); + ci.cancel(); + return; + } + + if (packet.action == PacketPlayerAction.ACTION_DIG_START + && Data.Users.getOrCreate(player.uuid).lockOnBlockPunched + && lockable.getIsLocked() + && lockable.getLockOwner().equals(player.uuid)) + { + FeedbackHandlerServer.sendFeedback(FeedbackType.error, player, "Failed to Lock Container! (Already Locked)"); + FeedbackHandlerServer.sendFeedback(FeedbackType.error, player, packet.toString()); + ci.cancel(); + return; + } + + if(packet.action == PacketPlayerAction.ACTION_DIG_START && Data.Users.getOrCreate(player.uuid).lockOnBlockPunched && !lockable.getIsLocked()){ if (container instanceof TileEntityChest) { Lockable iOtherContainer = (Lockable) MUtil.getOtherChest(world, (TileEntityChest) container); if (iOtherContainer != null) { @@ -213,6 +236,7 @@ private void handleBlockDigInject(PacketPlayerAction packet, CallbackInfo ci){ iOtherContainer.setLockOwner(player.uuid); FeedbackHandlerServer.sendFeedback(FeedbackType.success, player, "Locked Double Chest!"); ci.cancel(); + return; } else { FeedbackHandlerServer.sendFeedback(FeedbackType.success, player, "Locked Chest!"); } @@ -235,24 +259,7 @@ private void handleBlockDigInject(PacketPlayerAction packet, CallbackInfo ci){ lockable.setIsLocked(true); lockable.setLockOwner(player.uuid); ci.cancel(); - } - - else if (packet.action == PacketPlayerAction.ACTION_DIG_CONTINUED - && Data.Users.getOrCreate(player.uuid).lockOnBlockPunched - && lockable.getIsLocked() - && !lockable.getLockOwner().equals(player.uuid)) - { - FeedbackHandlerServer.sendFeedback(FeedbackType.error, player, "Failed to Lock Container! (Not Owned By You)"); - ci.cancel(); - } - - else if (packet.action == PacketPlayerAction.ACTION_DIG_CONTINUED - && Data.Users.getOrCreate(player.uuid).lockOnBlockPunched - && lockable.getIsLocked() - && lockable.getLockOwner().equals(player.uuid)) - { - FeedbackHandlerServer.sendFeedback(FeedbackType.error, player, "Failed to Lock Container! (Already Locked)"); - ci.cancel(); + return; } } }