Fix server sync issues

Cancel the PlayerInteractItemC2SPacket instead of the PlayerActionC2SPacket so the trident charge state is properly synced to the server.

gradle.properties

@@ -8,7 +8,7 @@ org.gradle.jvmargs=-Xmx1G
 	loader_version=0.16.0
 
 # Mod Properties
-	mod_version = 1.2.1
+	mod_version = 1.2.5
 	maven_group = com.kasetoatz
 	archives_base_name = TridentHacks
 

src/main/java/com/kasetoatz/tridenthacks/TridentHacks.java

@@ -7,13 +7,10 @@ import net.fabricmc.fabric.api.client.keybinding.v1.KeyBindingHelper;
 import net.minecraft.client.MinecraftClient;
 import net.minecraft.client.option.KeyBinding;
 import net.minecraft.client.util.InputUtil;
-import net.minecraft.entity.player.PlayerInventory;
 import net.minecraft.item.Items;
 import net.minecraft.text.Text;
 import org.lwjgl.glfw.GLFW;
 
-import java.util.HashMap;
-
 public class TridentHacks implements ClientModInitializer {
     public static MinecraftClient client;
     public static boolean onGround = true;

src/main/java/com/kasetoatz/tridenthacks/config/Config.java

@@ -68,7 +68,7 @@ public class Config {
         json.addProperty("returnToSameSlot", returnToSameSlot);
         json.addProperty("riptideOnMessage", riptideOnMessage);
         json.addProperty("riptideOffMessage", riptideOffMessage);
-        try (FileWriter writer = new FileWriter(new File(TridentHacks.client.runDirectory, "config/tridenthacks.json")))
+        try (FileWriter writer = new FileWriter(file))
         {
             gson.toJson(json, writer);
         }

src/main/java/com/kasetoatz/tridenthacks/mixin/ClientConnectionMixin.java

@@ -3,9 +3,10 @@ package com.kasetoatz.tridenthacks.mixin;
 import com.kasetoatz.tridenthacks.config.Config;
 import com.kasetoatz.tridenthacks.TridentHacks;
 import net.minecraft.client.network.ClientPlayerEntity;
+import net.minecraft.item.Items;
 import net.minecraft.network.ClientConnection;
 import net.minecraft.network.packet.Packet;
-import net.minecraft.network.packet.c2s.play.PlayerActionC2SPacket;
+import net.minecraft.network.packet.c2s.play.PlayerInteractItemC2SPacket;
 import net.minecraft.network.packet.c2s.play.PlayerMoveC2SPacket;
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.injection.At;
@@ -22,7 +23,7 @@ public abstract class ClientConnectionMixin {
             ClientPlayerEntity player = TridentHacks.client.player;
             if (player != null)
             {
-                if (packet instanceof PlayerActionC2SPacket && TridentHacks.noNormalRiptide() && ((PlayerActionC2SPacket)packet).getAction() == PlayerActionC2SPacket.Action.RELEASE_USE_ITEM)
+                if (packet instanceof PlayerInteractItemC2SPacket && player.getStackInHand(((PlayerInteractItemC2SPacket)packet).getHand()).getItem() == Items.TRIDENT && TridentHacks.noNormalRiptide())
                 {
                     ci.cancel();
                 }

src/main/java/com/kasetoatz/tridenthacks/mixin/TridentItemMixin.java

@@ -25,13 +25,16 @@ public class TridentItemMixin {
         if (Config.toggleRiptide)
         {
             PlayerEntity player = TridentHacks.client.player;
-            if (TridentHacks.noNormalRiptide() && user == player && user.getItemUseTime() > 10)
+            if (user == player && user.getItemUseTime() > 10)
+            {
+                if (TridentHacks.noNormalRiptide())
                 {
                     world.playSoundFromEntity(player, SoundEvents.ITEM_TRIDENT_RIPTIDE_3.value(), SoundCategory.PLAYERS, 1.F, 1.F);
-            }
                     TridentHacks.lastTridentUse = System.currentTimeMillis();
+                }
                 TridentHacks.onGround = false;
             }
+        }
         else if (Config.returnToSameSlot && !TridentHacks.hasRiptide)
         {
             PlayerEntity player = TridentHacks.client.player;

src/main/resources/fabric.mod.json

@@ -2,7 +2,7 @@
   "schemaVersion": 1,
   "id": "tridenthacks",
   "version": "${version}",
-  "name": "RiptideHacks",
+  "name": "TridentHacks",
   "description": "",
   "authors": [],
   "contact": {