From 8c98433da9cb8f0eef9a823352d07398f59c59af Mon Sep 17 00:00:00 2001 From: CosminPerRam Date: Thu, 29 Dec 2022 16:37:42 +0200 Subject: [PATCH] Fix online/max players being reversed on minecraft legacy 1.6 --- CHANGELOG.md | 3 ++- src/protocols/minecraft/protocol/legacy_v1_6.rs | 4 ++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 4970d55..e5df8a6 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -4,7 +4,8 @@ Who knows what the future holds... # 0.0.7 - ??/??/2022 ### Changes: [Minecraft](https://www.minecraft.com) bedrock edition support. -Also added a `query_legacy_specific` method to the mc game queries. +Fix Minecraft legacy v1.6 max/online players count being reversed. +Added `query_legacy_specific` method to the Minecraft protocol. ### Breaking: Removed `query_specific` from the mc protocol in favor of `query_java`, `query_legacy` and `query_legacy_specific`. diff --git a/src/protocols/minecraft/protocol/legacy_v1_6.rs b/src/protocols/minecraft/protocol/legacy_v1_6.rs index 9bd14a4..cd92380 100644 --- a/src/protocols/minecraft/protocol/legacy_v1_6.rs +++ b/src/protocols/minecraft/protocol/legacy_v1_6.rs @@ -55,9 +55,9 @@ impl LegacyV1_6 { .map_err(|_| GDError::PacketBad("Failed to parse to expected int.".to_string()))?; let version_name = split[1].to_string(); let description = split[2].to_string(); - let max_players = split[3].parse() + let online_players = split[3].parse() .map_err(|_| GDError::PacketBad("Failed to parse to expected int.".to_string()))?; - let online_players = split[4].parse() + let max_players = split[4].parse() .map_err(|_| GDError::PacketBad("Failed to parse to expected int.".to_string()))?; Ok(Response {