From 424bb2017c7307133aea9232772b4b654c5faa0c Mon Sep 17 00:00:00 2001 From: GatoArtStudio Date: Fri, 26 Sep 2025 12:58:22 -0500 Subject: [PATCH 1/3] Refactored: Reduces the amount of duplicate code and increases readability --- .../server/command/ShowViewCommand.java | 46 +++++++++++-------- 1 file changed, 26 insertions(+), 20 deletions(-) diff --git a/src/main/java/art/gatoartstudio/moneyscreen/server/command/ShowViewCommand.java b/src/main/java/art/gatoartstudio/moneyscreen/server/command/ShowViewCommand.java index 3805fd5..c59ff2c 100644 --- a/src/main/java/art/gatoartstudio/moneyscreen/server/command/ShowViewCommand.java +++ b/src/main/java/art/gatoartstudio/moneyscreen/server/command/ShowViewCommand.java @@ -5,6 +5,7 @@ import com.mojang.brigadier.Command; import com.mojang.brigadier.CommandDispatcher; import com.mojang.brigadier.arguments.IntegerArgumentType; +import com.mojang.brigadier.context.CommandContext; import net.fabricmc.fabric.api.networking.v1.ServerPlayNetworking; import net.minecraft.server.command.CommandManager; import net.minecraft.server.command.ServerCommandSource; @@ -16,28 +17,33 @@ public static void register(CommandDispatcher dispatcher) { .requires(serverCommandSource -> serverCommandSource.hasPermissionLevel(2)) .then(CommandManager.argument("money", IntegerArgumentType.integer()) .then(CommandManager.argument("players", IntegerArgumentType.integer()) - .executes(context -> { - int money = IntegerArgumentType.getInteger(context, "money"); - int players = IntegerArgumentType.getInteger(context, "players"); - - // Here you would implement the logic to show the money screen to players - context.getSource().sendFeedback(() -> Text.literal("MoneyScreen command executed, datos recibidos: money=" + money + ", players=" + players), false); - - MoneyUpdatePayload moneyUpdatePayload = new MoneyUpdatePayload(money, players); - context.getSource().getServer().getPlayerManager().getPlayerList() - .forEach(player -> { - ServerPlayNetworking.send(player, moneyUpdatePayload); - Moneyscreen.LOGGER.info("Sent MoneyUpdatePayload to player {}: money={}, players={}", player.getName().getString(), money, players); - }); - - return Command.SINGLE_SUCCESS; - }) + .executes(ShowViewCommand::execute) ) + .executes(ShowViewCommand::executeHelp) ) - .executes(context -> { - context.getSource().sendFeedback(() -> Text.literal("Please user: /moneyscreen money players, example: /moneyscreen 120 520"), false); - return Command.SINGLE_SUCCESS; - }) + .executes(ShowViewCommand::executeHelp) ); } + + public static int execute(CommandContext context) { + int money = IntegerArgumentType.getInteger(context, "money"); + int players = IntegerArgumentType.getInteger(context, "players"); + + // Here you would implement the logic to show the money screen to players + context.getSource().sendFeedback(() -> Text.literal("MoneyScreen command executed, datos recibidos: money=" + money + ", players=" + players), false); + + MoneyUpdatePayload moneyUpdatePayload = new MoneyUpdatePayload(money, players); + context.getSource().getServer().getPlayerManager().getPlayerList() + .forEach(player -> { + ServerPlayNetworking.send(player, moneyUpdatePayload); + Moneyscreen.LOGGER.info("Sent MoneyUpdatePayload to player {}: money={}, players={}", player.getName().getString(), money, players); + }); + + return Command.SINGLE_SUCCESS; + } + + public static int executeHelp(CommandContext context) { + context.getSource().sendFeedback(() -> Text.literal("Please user: /moneyscreen money players, example: /moneyscreen 120 520"), false); + return Command.SINGLE_SUCCESS; + } } From 01b94b5610e50db94c0020658df0cbab4a75e189 Mon Sep 17 00:00:00 2001 From: GatoArtStudio Date: Sat, 27 Sep 2025 17:06:12 -0500 Subject: [PATCH 2/3] Modified: The Fabric loader version has been downloaded.The Fabric loader version has been downloaded. --- gradle.properties | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gradle.properties b/gradle.properties index ad51d7f..85f91c8 100644 --- a/gradle.properties +++ b/gradle.properties @@ -4,7 +4,7 @@ org.gradle.jvmargs=-Xmx1G # check these on https://modmuss50.me/fabric.html minecraft_version=1.21.1 yarn_mappings=1.21.1+build.3 -loader_version=0.17.2 +loader_version=0.16.14 # Mod Properties mod_version=1.1-SNAPSHOT maven_group=art.gatoartstudio From d67527a0159c2bf06e0364b8577f1422b4c18f93 Mon Sep 17 00:00:00 2001 From: GatoArtStudio Date: Sat, 27 Sep 2025 17:07:53 -0500 Subject: [PATCH 3/3] Bumped to 1.2-SNAPSHOT --- gradle.properties | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gradle.properties b/gradle.properties index 85f91c8..7396a0d 100644 --- a/gradle.properties +++ b/gradle.properties @@ -6,7 +6,7 @@ minecraft_version=1.21.1 yarn_mappings=1.21.1+build.3 loader_version=0.16.14 # Mod Properties -mod_version=1.1-SNAPSHOT +mod_version=1.2-SNAPSHOT maven_group=art.gatoartstudio archives_base_name=MoneyScreen # Dependencies