diff --git a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_10to1_9_3/packets/InventoryPackets.java b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_10to1_9_3/packets/InventoryPackets.java index dcd7d310e..bb2269eba 100644 --- a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_10to1_9_3/packets/InventoryPackets.java +++ b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_10to1_9_3/packets/InventoryPackets.java @@ -27,12 +27,12 @@ import com.viaversion.viaversion.rewriter.ItemRewriter; public class InventoryPackets extends ItemRewriter { public InventoryPackets(Protocol1_10To1_9_3_4 protocol) { - super(protocol, null, null); + super(protocol, Type.ITEM1_8, null); } @Override public void registerPackets() { - registerCreativeInvAction(ServerboundPackets1_9_3.CREATIVE_INVENTORY_ACTION, Type.ITEM1_8); + registerCreativeInvAction(ServerboundPackets1_9_3.CREATIVE_INVENTORY_ACTION); } @Override diff --git a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_11_1to1_11/packets/InventoryPackets.java b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_11_1to1_11/packets/InventoryPackets.java index 026ca21d7..22633016b 100644 --- a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_11_1to1_11/packets/InventoryPackets.java +++ b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_11_1to1_11/packets/InventoryPackets.java @@ -27,12 +27,12 @@ import com.viaversion.viaversion.rewriter.ItemRewriter; public class InventoryPackets extends ItemRewriter { public InventoryPackets(Protocol1_11_1To1_11 protocol) { - super(protocol, null, null); + super(protocol, Type.ITEM1_8, null); } @Override public void registerPackets() { - registerCreativeInvAction(ServerboundPackets1_9_3.CREATIVE_INVENTORY_ACTION, Type.ITEM1_8); + registerCreativeInvAction(ServerboundPackets1_9_3.CREATIVE_INVENTORY_ACTION); } @Override diff --git a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_11to1_10/packets/InventoryPackets.java b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_11to1_10/packets/InventoryPackets.java index 94451b75f..0cd567dbd 100644 --- a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_11to1_10/packets/InventoryPackets.java +++ b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_11to1_10/packets/InventoryPackets.java @@ -29,14 +29,14 @@ import com.viaversion.viaversion.rewriter.ItemRewriter; public class InventoryPackets extends ItemRewriter { public InventoryPackets(Protocol1_11To1_10 protocol) { - super(protocol, null, null); + super(protocol, Type.ITEM1_8, Type.ITEM1_8_SHORT_ARRAY); } @Override public void registerPackets() { - registerSetSlot(ClientboundPackets1_9_3.SET_SLOT, Type.ITEM1_8); - registerWindowItems(ClientboundPackets1_9_3.WINDOW_ITEMS, Type.ITEM1_8_SHORT_ARRAY); - registerEntityEquipment(ClientboundPackets1_9_3.ENTITY_EQUIPMENT, Type.ITEM1_8); + registerSetSlot(ClientboundPackets1_9_3.SET_SLOT); + registerWindowItems(ClientboundPackets1_9_3.WINDOW_ITEMS); + registerEntityEquipment(ClientboundPackets1_9_3.ENTITY_EQUIPMENT); // Plugin message Packet -> Trading protocol.registerClientbound(ClientboundPackets1_9_3.PLUGIN_MESSAGE, new PacketHandlers() { @@ -66,8 +66,8 @@ public class InventoryPackets extends ItemRewriter { public InventoryPackets(Protocol1_12To1_11_1 protocol) { - super(protocol, null, null); + super(protocol, Type.ITEM1_8, Type.ITEM1_8_SHORT_ARRAY); } @Override public void registerPackets() { - registerSetSlot(ClientboundPackets1_9_3.SET_SLOT, Type.ITEM1_8); - registerWindowItems(ClientboundPackets1_9_3.WINDOW_ITEMS, Type.ITEM1_8_SHORT_ARRAY); - registerEntityEquipment(ClientboundPackets1_9_3.ENTITY_EQUIPMENT, Type.ITEM1_8); + registerSetSlot(ClientboundPackets1_9_3.SET_SLOT); + registerWindowItems(ClientboundPackets1_9_3.WINDOW_ITEMS); + registerEntityEquipment(ClientboundPackets1_9_3.ENTITY_EQUIPMENT); - // Plugin message Packet -> Trading + // Plugin message -> Trading protocol.registerClientbound(ClientboundPackets1_9_3.PLUGIN_MESSAGE, new PacketHandlers() { @Override public void register() { @@ -107,8 +107,7 @@ public class InventoryPackets extends ItemRewriter { public InventoryPackets(Protocol1_13_1To1_13 protocol) { - super(protocol, Type.ITEM1_13, Type.ITEM1_13_ARRAY); + super(protocol, Type.ITEM1_13, Type.ITEM1_13_SHORT_ARRAY); } @Override public void registerPackets() { - registerSetSlot(ClientboundPackets1_13.SET_SLOT, Type.ITEM1_13); - registerWindowItems(ClientboundPackets1_13.WINDOW_ITEMS, Type.ITEM1_13_SHORT_ARRAY); - registerAdvancements(ClientboundPackets1_13.ADVANCEMENTS, Type.ITEM1_13); + registerSetSlot(ClientboundPackets1_13.SET_SLOT); + registerWindowItems(ClientboundPackets1_13.WINDOW_ITEMS); + registerAdvancements(ClientboundPackets1_13.ADVANCEMENTS); registerSetCooldown(ClientboundPackets1_13.COOLDOWN); protocol.registerClientbound(ClientboundPackets1_13.PLUGIN_MESSAGE, new PacketHandlers() { @@ -71,7 +71,7 @@ public class InventoryPackets extends ItemRewriter recipeRewriter = new RecipeRewriter(protocol) { @Override @@ -94,9 +94,9 @@ public class InventoryPackets extends ItemRewriter { Short windowId = wrapper.read(Type.UNSIGNED_BYTE); @@ -136,8 +136,8 @@ public class InventoryPackets extends ItemRewriter recipeRewriter = new RecipeRewriter<>(protocol); protocol.registerClientbound(ClientboundPackets1_13.DECLARE_RECIPES, wrapper -> { @@ -212,7 +212,7 @@ public class InventoryPackets extends ItemRewriter { // Selecting trade now moves the items, we need to resync the inventory @@ -229,9 +229,9 @@ public class InventoryPackets extends ItemRewriter { public InventoryPackets(Protocol1_15To1_14_4 protocol) { - super(protocol, Type.ITEM1_13_2, Type.ITEM1_13_2_ARRAY); + super(protocol, Type.ITEM1_13_2, Type.ITEM1_13_2_SHORT_ARRAY); } @Override public void registerPackets() { registerSetCooldown(ClientboundPackets1_14_4.COOLDOWN); - registerWindowItems(ClientboundPackets1_14_4.WINDOW_ITEMS, Type.ITEM1_13_2_SHORT_ARRAY); + registerWindowItems(ClientboundPackets1_14_4.WINDOW_ITEMS); registerTradeList(ClientboundPackets1_14_4.TRADE_LIST); - registerSetSlot(ClientboundPackets1_14_4.SET_SLOT, Type.ITEM1_13_2); - registerEntityEquipment(ClientboundPackets1_14_4.ENTITY_EQUIPMENT, Type.ITEM1_13_2); - registerAdvancements(ClientboundPackets1_14_4.ADVANCEMENTS, Type.ITEM1_13_2); + registerSetSlot(ClientboundPackets1_14_4.SET_SLOT); + registerEntityEquipment(ClientboundPackets1_14_4.ENTITY_EQUIPMENT); + registerAdvancements(ClientboundPackets1_14_4.ADVANCEMENTS); new RecipeRewriter<>(protocol).register(ClientboundPackets1_14_4.DECLARE_RECIPES); - registerClickWindow(ServerboundPackets1_14.CLICK_WINDOW, Type.ITEM1_13_2); - registerCreativeInvAction(ServerboundPackets1_14.CREATIVE_INVENTORY_ACTION, Type.ITEM1_13_2); + registerClickWindow(ServerboundPackets1_14.CLICK_WINDOW); + registerCreativeInvAction(ServerboundPackets1_14.CREATIVE_INVENTORY_ACTION); } } diff --git a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_16_2to1_16_1/packets/InventoryPackets.java b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_16_2to1_16_1/packets/InventoryPackets.java index 559071e6d..a95c45374 100644 --- a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_16_2to1_16_1/packets/InventoryPackets.java +++ b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_16_2to1_16_1/packets/InventoryPackets.java @@ -27,17 +27,17 @@ import com.viaversion.viaversion.rewriter.RecipeRewriter; public class InventoryPackets extends ItemRewriter { public InventoryPackets(Protocol1_16_2To1_16_1 protocol) { - super(protocol, Type.ITEM1_13_2, Type.ITEM1_13_2_ARRAY); + super(protocol, Type.ITEM1_13_2, Type.ITEM1_13_2_SHORT_ARRAY); } @Override public void registerPackets() { registerSetCooldown(ClientboundPackets1_16.COOLDOWN); - registerWindowItems(ClientboundPackets1_16.WINDOW_ITEMS, Type.ITEM1_13_2_SHORT_ARRAY); + registerWindowItems(ClientboundPackets1_16.WINDOW_ITEMS); registerTradeList(ClientboundPackets1_16.TRADE_LIST); - registerSetSlot(ClientboundPackets1_16.SET_SLOT, Type.ITEM1_13_2); + registerSetSlot(ClientboundPackets1_16.SET_SLOT); registerEntityEquipmentArray(ClientboundPackets1_16.ENTITY_EQUIPMENT); - registerAdvancements(ClientboundPackets1_16.ADVANCEMENTS, Type.ITEM1_13_2); + registerAdvancements(ClientboundPackets1_16.ADVANCEMENTS); protocol.registerClientbound(ClientboundPackets1_16.UNLOCK_RECIPES, wrapper -> { wrapper.passthrough(Type.VAR_INT); @@ -54,10 +54,10 @@ public class InventoryPackets extends ItemRewriter(protocol).register(ClientboundPackets1_16.DECLARE_RECIPES); - registerClickWindow(ServerboundPackets1_16_2.CLICK_WINDOW, Type.ITEM1_13_2); - registerCreativeInvAction(ServerboundPackets1_16_2.CREATIVE_INVENTORY_ACTION, Type.ITEM1_13_2); + registerClickWindow(ServerboundPackets1_16_2.CLICK_WINDOW); + registerCreativeInvAction(ServerboundPackets1_16_2.CREATIVE_INVENTORY_ACTION); protocol.registerServerbound(ServerboundPackets1_16_2.EDIT_BOOK, wrapper -> handleItemToServer(wrapper.passthrough(Type.ITEM1_13_2))); - registerSpawnParticle(ClientboundPackets1_16.SPAWN_PARTICLE, Type.ITEM1_13_2, Type.DOUBLE); + registerSpawnParticle(ClientboundPackets1_16.SPAWN_PARTICLE, Type.DOUBLE); } } diff --git a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_16to1_15_2/packets/InventoryPackets.java b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_16to1_15_2/packets/InventoryPackets.java index d25b2a7d5..ff604e3db 100644 --- a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_16to1_15_2/packets/InventoryPackets.java +++ b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_16to1_15_2/packets/InventoryPackets.java @@ -43,7 +43,7 @@ import java.util.UUID; public class InventoryPackets extends ItemRewriter { public InventoryPackets(Protocol1_16To1_15_2 protocol) { - super(protocol, Type.ITEM1_13_2, Type.ITEM1_13_2_ARRAY); + super(protocol, Type.ITEM1_13_2, Type.ITEM1_13_2_SHORT_ARRAY); } @Override @@ -107,10 +107,10 @@ public class InventoryPackets extends ItemRewriter(protocol).register(ClientboundPackets1_15.DECLARE_RECIPES); - registerClickWindow(ServerboundPackets1_16.CLICK_WINDOW, Type.ITEM1_13_2); - registerCreativeInvAction(ServerboundPackets1_16.CREATIVE_INVENTORY_ACTION, Type.ITEM1_13_2); + registerClickWindow(ServerboundPackets1_16.CLICK_WINDOW); + registerCreativeInvAction(ServerboundPackets1_16.CREATIVE_INVENTORY_ACTION); protocol.registerServerbound(ServerboundPackets1_16.CLOSE_WINDOW, wrapper -> { InventoryTracker1_16 inventoryTracker = wrapper.user().get(InventoryTracker1_16.class); @@ -137,7 +137,7 @@ public class InventoryPackets extends ItemRewriter handleItemToServer(wrapper.passthrough(Type.ITEM1_13_2))); - registerSpawnParticle(ClientboundPackets1_15.SPAWN_PARTICLE, Type.ITEM1_13_2, Type.DOUBLE); + registerSpawnParticle(ClientboundPackets1_15.SPAWN_PARTICLE, Type.DOUBLE); } @Override diff --git a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_17to1_16_4/packets/InventoryPackets.java b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_17to1_16_4/packets/InventoryPackets.java index dcc5f824b..ed8e42fd7 100644 --- a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_17to1_16_4/packets/InventoryPackets.java +++ b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_17to1_16_4/packets/InventoryPackets.java @@ -36,22 +36,22 @@ import com.viaversion.viaversion.rewriter.RecipeRewriter; public final class InventoryPackets extends ItemRewriter { public InventoryPackets(Protocol1_17To1_16_4 protocol) { - super(protocol, Type.ITEM1_13_2, Type.ITEM1_13_2_ARRAY); + super(protocol, Type.ITEM1_13_2, Type.ITEM1_13_2_SHORT_ARRAY); } @Override public void registerPackets() { registerSetCooldown(ClientboundPackets1_16_2.COOLDOWN); - registerWindowItems(ClientboundPackets1_16_2.WINDOW_ITEMS, Type.ITEM1_13_2_SHORT_ARRAY); + registerWindowItems(ClientboundPackets1_16_2.WINDOW_ITEMS); registerTradeList(ClientboundPackets1_16_2.TRADE_LIST); - registerSetSlot(ClientboundPackets1_16_2.SET_SLOT, Type.ITEM1_13_2); - registerAdvancements(ClientboundPackets1_16_2.ADVANCEMENTS, Type.ITEM1_13_2); + registerSetSlot(ClientboundPackets1_16_2.SET_SLOT); + registerAdvancements(ClientboundPackets1_16_2.ADVANCEMENTS); registerEntityEquipmentArray(ClientboundPackets1_16_2.ENTITY_EQUIPMENT); - registerSpawnParticle(ClientboundPackets1_16_2.SPAWN_PARTICLE, Type.ITEM1_13_2, Type.DOUBLE); + registerSpawnParticle(ClientboundPackets1_16_2.SPAWN_PARTICLE, Type.DOUBLE); new RecipeRewriter<>(protocol).register(ClientboundPackets1_16_2.DECLARE_RECIPES); - registerCreativeInvAction(ServerboundPackets1_17.CREATIVE_INVENTORY_ACTION, Type.ITEM1_13_2); + registerCreativeInvAction(ServerboundPackets1_17.CREATIVE_INVENTORY_ACTION); protocol.registerServerbound(ServerboundPackets1_17.EDIT_BOOK, wrapper -> handleItemToServer(wrapper.passthrough(Type.ITEM1_13_2))); diff --git a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_18to1_17_1/packets/InventoryPackets.java b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_18to1_17_1/packets/InventoryPackets.java index e732e2011..af7b7f335 100644 --- a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_18to1_17_1/packets/InventoryPackets.java +++ b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_18to1_17_1/packets/InventoryPackets.java @@ -38,7 +38,7 @@ public final class InventoryPackets extends ItemRewriter(protocol).register(ClientboundPackets1_17_1.DECLARE_RECIPES); registerClickWindow1_17_1(ServerboundPackets1_17.CLICK_WINDOW); - registerCreativeInvAction(ServerboundPackets1_17.CREATIVE_INVENTORY_ACTION, Type.ITEM1_13_2); + registerCreativeInvAction(ServerboundPackets1_17.CREATIVE_INVENTORY_ACTION); } } diff --git a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_19_3to1_19_1/packets/InventoryPackets.java b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_19_3to1_19_1/packets/InventoryPackets.java index 025ca3bff..707418279 100644 --- a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_19_3to1_19_1/packets/InventoryPackets.java +++ b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_19_3to1_19_1/packets/InventoryPackets.java @@ -50,11 +50,11 @@ public final class InventoryPackets extends ItemRewriter(protocol) { diff --git a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_19to1_18_2/packets/InventoryPackets.java b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_19to1_18_2/packets/InventoryPackets.java index 0c506c250..8015ce262 100644 --- a/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_19to1_18_2/packets/InventoryPackets.java +++ b/common/src/main/java/com/viaversion/viaversion/protocols/protocol1_19to1_18_2/packets/InventoryPackets.java @@ -41,7 +41,7 @@ public final class InventoryPackets extends ItemRewriter itemType; private final Type itemArrayType; - @Deprecated/*(forRemoval = true)*/ - protected ItemRewriter(T protocol) { - this(protocol, Type.ITEM1_13_2, Type.ITEM1_13_2_ARRAY); - } - public ItemRewriter(T protocol, Type itemType, Type itemArrayType) { super(protocol); this.itemType = itemType; @@ -65,13 +60,13 @@ public class ItemRewriter type) { + public void registerWindowItems(C packetType) { protocol.registerClientbound(packetType, new PacketHandlers() { @Override public void register() { map(Type.UNSIGNED_BYTE); // Window id - map(type); // Items - handler(itemArrayToClientHandler(type)); + map(itemArrayType); // Items + handler(itemArrayToClientHandler(itemArrayType)); } }); } @@ -113,14 +108,14 @@ public class ItemRewriter type) { + public void registerSetSlot(C packetType) { protocol.registerClientbound(packetType, new PacketHandlers() { @Override public void register() { map(Type.UNSIGNED_BYTE); // Window id map(Type.SHORT); // Slot id - map(type); // Item - handler(itemToClientHandler(type)); + map(itemType); // Item + handler(itemToClientHandler(itemType)); } }); } @@ -139,15 +134,15 @@ public class ItemRewriter type) { + public void registerEntityEquipment(C packetType) { protocol.registerClientbound(packetType, new PacketHandlers() { @Override public void register() { map(Type.VAR_INT); // 0 - Entity ID map(Type.VAR_INT); // 1 - Slot ID - map(type); // 2 - Item + map(itemType); // 2 - Item - handler(itemToClientHandler(type)); + handler(itemToClientHandler(itemType)); } }); } @@ -172,21 +167,17 @@ public class ItemRewriter type) { protocol.registerServerbound(packetType, new PacketHandlers() { @Override public void register() { map(Type.SHORT); // 0 - Slot - map(type); // 1 - Clicked Item - handler(itemToServerHandler(type)); + map(itemType); // 1 - Clicked Item + handler(itemToServerHandler(itemType)); } }); } - public void registerClickWindow(S packetType, Type type) { + public void registerClickWindow(S packetType) { protocol.registerServerbound(packetType, new PacketHandlers() { @Override public void register() { @@ -195,9 +186,9 @@ public class ItemRewriter type) { + public void registerAdvancements(C packetType) { protocol.registerClientbound(packetType, wrapper -> { wrapper.passthrough(Type.BOOLEAN); // Reset/clear int size = wrapper.passthrough(Type.VAR_INT); // Mapping size @@ -296,7 +287,7 @@ public class ItemRewriter itemType, Type coordType) { + public void registerSpawnParticle(C packetType, Type coordType) { protocol.registerClientbound(packetType, new PacketHandlers() { @Override public void register() { @@ -462,4 +453,12 @@ public class ItemRewriter type) { return wrapper -> handleItemToServer(wrapper.get(type, 0)); } + + public Type getItemType() { + return itemType; + } + + public Type getItemArrayType() { + return itemArrayType; + } }