Remove unmapped piston warning

Remove some wont-fix/unneeded todos
This commit is contained in:
KennyTV 2020-06-19 17:47:43 +02:00
parent f83ac57d8f
commit bdc79c43eb
No known key found for this signature in database
GPG Key ID: 6BE3B555EBC5982B
6 changed files with 5 additions and 7 deletions

View File

@ -192,7 +192,7 @@ public class BlockItemPackets1_11 extends LegacyBlockItemRewriter<Protocol1_10To
handleChunk(chunk); handleChunk(chunk);
// only patch it for signs for now, TODO-> Find all the block entities old/new to replace ids and implement in ViaVersion // only patch it for signs for now
for (CompoundTag tag : chunk.getBlockEntities()) { for (CompoundTag tag : chunk.getBlockEntities()) {
Tag idTag = tag.get("id"); Tag idTag = tag.get("id");
if (!(idTag instanceof StringTag)) continue; if (!(idTag instanceof StringTag)) continue;

View File

@ -239,7 +239,7 @@ public class BlockItemPackets1_12 extends LegacyBlockItemRewriter<Protocol1_11_1
public void handle(PacketWrapper wrapper) throws Exception { public void handle(PacketWrapper wrapper) throws Exception {
// Open Inventory // Open Inventory
if (wrapper.get(Type.VAR_INT, 0) == 2) { if (wrapper.get(Type.VAR_INT, 0) == 2) {
wrapper.cancel(); // TODO is this replaced by something else? wrapper.cancel();
} }
} }
}); });

View File

@ -78,7 +78,7 @@ public class Protocol1_12_2To1_13 extends BackwardsProtocol<ClientboundPackets1_
cancelOutgoing(ClientboundPackets1_13.CRAFT_RECIPE_RESPONSE); cancelOutgoing(ClientboundPackets1_13.CRAFT_RECIPE_RESPONSE);
cancelOutgoing(ClientboundPackets1_13.UNLOCK_RECIPES); cancelOutgoing(ClientboundPackets1_13.UNLOCK_RECIPES);
cancelOutgoing(ClientboundPackets1_13.ADVANCEMENTS); cancelOutgoing(ClientboundPackets1_13.ADVANCEMENTS);
cancelOutgoing(ClientboundPackets1_13.DECLARE_RECIPES); //TODO ? cancelOutgoing(ClientboundPackets1_13.DECLARE_RECIPES);
cancelOutgoing(ClientboundPackets1_13.TAGS); cancelOutgoing(ClientboundPackets1_13.TAGS);
cancelIncoming(ServerboundPackets1_12_1.CRAFT_RECIPE_REQUEST); cancelIncoming(ServerboundPackets1_12_1.CRAFT_RECIPE_REQUEST);

View File

@ -1,6 +1,5 @@
package nl.matsv.viabackwards.protocol.protocol1_12_2to1_13.block_entity_handlers; package nl.matsv.viabackwards.protocol.protocol1_12_2to1_13.block_entity_handlers;
import nl.matsv.viabackwards.ViaBackwards;
import nl.matsv.viabackwards.protocol.protocol1_12_2to1_13.data.BackwardsMappings; import nl.matsv.viabackwards.protocol.protocol1_12_2to1_13.data.BackwardsMappings;
import nl.matsv.viabackwards.protocol.protocol1_12_2to1_13.providers.BackwardsBlockEntityProvider; import nl.matsv.viabackwards.protocol.protocol1_12_2to1_13.providers.BackwardsBlockEntityProvider;
import us.myles.ViaVersion.api.Via; import us.myles.ViaVersion.api.Via;
@ -74,7 +73,7 @@ public class PistonHandler implements BackwardsBlockEntityProvider.BackwardsBloc
Integer id = pistonIds.get(dataFromTag); Integer id = pistonIds.get(dataFromTag);
if (id == null) { if (id == null) {
ViaBackwards.getPlatform().getLogger().warning("Unmapped piston id: " + dataFromTag); //TODO see why this could be null and if this is bad
return tag; return tag;
} }

View File

@ -513,7 +513,7 @@ public class PlayerPacket1_13 extends Rewriter<Protocol1_12_2To1_13> {
int statisticId = wrapper.read(Type.VAR_INT); int statisticId = wrapper.read(Type.VAR_INT);
String name = ""; String name = "";
//TODO categories 0-7 (items, blocks, entities) - probably not feasible // categories 0-7 (items, blocks, entities) - probably not feasible
switch (categoryId) { switch (categoryId) {
case 0: case 0:
case 1: case 1:

View File

@ -381,7 +381,6 @@ public class EntityPackets1_14 extends LegacyEntityRewriter<Protocol1_13_2To1_14
if (type.isOrHasParent(Entity1_14Types.EntityType.ABSTRACT_ILLAGER_BASE) || type == Entity1_14Types.EntityType.RAVAGER || type == Entity1_14Types.EntityType.WITCH) { if (type.isOrHasParent(Entity1_14Types.EntityType.ABSTRACT_ILLAGER_BASE) || type == Entity1_14Types.EntityType.RAVAGER || type == Entity1_14Types.EntityType.WITCH) {
int index = e.getIndex(); int index = e.getIndex();
if (index == 14) { if (index == 14) {
//TODO handle
throw RemovedValueException.EX; throw RemovedValueException.EX;
} else if (index > 14) { } else if (index > 14) {
meta.setId(index - 1); meta.setId(index - 1);