Fix bad injection type in MixinNettyPacketEncoder

Author: ZeroMemes

src/main/java/clientapi/event/defaults/game/network/PacketEvent.java

@@ -87,9 +87,9 @@ public static final class Encode extends PacketEvent {
 
         private final EnumConnectionState state;
 
-        public Encode(Packet<?> packet, EnumConnectionState state) {
+        public Encode(Packet<?> packet) {
             super(packet);
-            this.state = state;
+            this.state = EnumConnectionState.getFromPacket(packet);
         }
 
         public final EnumConnectionState getDirection() {

src/main/java/clientapi/load/mixin/MixinNettyPacketEncoder.java

@@ -26,7 +26,7 @@
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.injection.At;
 import org.spongepowered.asm.mixin.injection.Inject;
-import org.spongepowered.asm.mixin.injection.ModifyArg;
+import org.spongepowered.asm.mixin.injection.ModifyVariable;
 import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
 
 /**
@@ -38,9 +38,9 @@ public class MixinNettyPacketEncoder {
 
     private PacketEvent event;
 
-    @ModifyArg(method = "encode", at = @At(value = "INVOKE", target = "Lio/netty/util/Attribute;get()Ljava/lang/Object;", remap = false))
-    private Packet<?> mutatePacket(ChannelHandlerContext ctx, Packet<?> msg, ByteBuf out, CallbackInfo ci, EnumConnectionState state) {
-        event = new PacketEvent.Encode(msg, state);
+    @ModifyVariable(method = "encode", at = @At("HEAD"), index = 1, remap = false)
+    private Packet<?> mutatePacket(Packet<?> msg) {
+        event = new PacketEvent.Encode(msg);
         ClientAPI.EVENT_BUS.post(event);
         return event.getPacket();
     }