fix MC-158900 (thanks Microcontrollers #482)

Author: isXander

.bugs

@@ -111,6 +111,7 @@ patched 264979 server basic
 patched 264285 server basic
 patched 131562 server basic
 patched 133218 server basic
+patched 158900 server basic
 
 previous 2025 17w47a
 previous 53312 22w17a

src/main/java/dev/isxander/debugify/mixins/basic/mc158900/PlayerListMixin.java

@@ -0,0 +1,22 @@
+package dev.isxander.debugify.mixins.basic.mc158900;
+
+import com.llamalad7.mixinextras.injector.wrapoperation.Operation;
+import com.llamalad7.mixinextras.injector.wrapoperation.WrapOperation;
+import dev.isxander.debugify.fixes.BugFix;
+import dev.isxander.debugify.fixes.FixCategory;
+import net.minecraft.server.players.NameAndId;
+import net.minecraft.server.players.PlayerList;
+import net.minecraft.server.players.UserBanList;
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.injection.At;
+
+@BugFix(id = "MC-158900", category = FixCategory.BASIC, env = BugFix.Env.SERVER, description = "\"bad packet id 26\" disconnect after first-login after a temporary ban expires")
+@Mixin(PlayerList.class)
+public class PlayerListMixin {
+    @WrapOperation(method = "canPlayerLogin", at = @At(value = "INVOKE", target = "Lnet/minecraft/server/players/UserBanList;isBanned(Lnet/minecraft/server/players/NameAndId;)Z"))
+    private boolean removeExpiredBeforeCheck(UserBanList instance, NameAndId user, Operation<Boolean> original) {
+        instance.get(user); // get will remove expired bans
+        return original.call(instance, user);
+    }
+
+}

src/main/resources/debugify.mixins.json

@@ -28,6 +28,7 @@
     "basic.mc147659.CatSpawnerMixin",
     "basic.mc153010.FoxMixin",
     "basic.mc155509.PufferfishMixin",
+    "basic.mc158900.PlayerListMixin",
     "basic.mc159283.DensityFunctionsMixin",
     "basic.mc160095.CactusBlockMixin",
     "basic.mc168573.BlocksAttacksMixin",