fixup default logging base class

- removed translation of some packets (left over from original source)
- all events are handled the same way now and translation is only performed in the subclass
This commit is contained in:
Blue Kelp 2019-08-01 18:53:35 -07:00
parent e85e16ecfe
commit 909b81275b

View file

@ -30,7 +30,6 @@ import com.nukkitx.protocol.bedrock.handler.BedrockPacketHandler;
import com.nukkitx.protocol.bedrock.packet.*;
import org.geysermc.connector.GeyserConnector;
import org.geysermc.connector.network.session.GeyserSession;
import org.geysermc.connector.network.translators.Registry;
/**
* Bare bones implementation of BedrockPacketHandler suitable for extension.
@ -70,9 +69,7 @@ public class LoggingPacketHandler implements BedrockPacketHandler {
@Override
public boolean handle(AnimatePacket packet) {
connector.getLogger().debug("Handled packet: " + packet.getClass().getSimpleName());
Registry.BEDROCK.translate(packet.getClass(), packet, session);
return true;
return defaultHandler(packet);
}
@Override
@ -117,9 +114,7 @@ public class LoggingPacketHandler implements BedrockPacketHandler {
@Override
public boolean handle(CommandRequestPacket packet) {
connector.getLogger().debug("Handled packet: " + packet.getClass().getSimpleName());
Registry.BEDROCK.translate(packet.getClass(), packet, session);
return true;
return defaultHandler(packet);
}
@Override
@ -309,9 +304,7 @@ public class LoggingPacketHandler implements BedrockPacketHandler {
@Override
public boolean handle(TextPacket packet) {
connector.getLogger().debug("Handled packet: " + packet.getClass().getSimpleName());
Registry.BEDROCK.translate(packet.getClass(), packet, session);
return true;
return defaultHandler(packet);
}
@Override