simplify to not have to deal with any stupid logic (i hate de morgan)

Author: MicrocontrollersDev

src/main/java/dev/isxander/debugify/mixins/basic/mc171420/MinecraftServerMixin.java

@@ -1,11 +1,13 @@
 package dev.isxander.debugify.mixins.basic.mc171420;
 
-import com.llamalad7.mixinextras.injector.ModifyExpressionValue;
+import com.llamalad7.mixinextras.injector.v2.WrapWithCondition;
 import com.llamalad7.mixinextras.sugar.Local;
 import dev.isxander.debugify.fixes.BugFix;
 import dev.isxander.debugify.fixes.FixCategory;
+import net.minecraft.network.chat.Component;
 import net.minecraft.server.MinecraftServer;
 import net.minecraft.server.level.ServerPlayer;
+import net.minecraft.server.network.ServerGamePacketListenerImpl;
 import net.minecraft.server.players.PlayerList;
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.Shadow;
@@ -17,12 +19,8 @@ public abstract class MinecraftServerMixin {
     @Shadow
     public abstract PlayerList getPlayerList();
 
-    /**
-     * The goal is to replace the !isWhiteListed with !isWhiteListed && !isOp
-     * The original isWhiteListed call is negated, so we need to negate our mixin logic here
-     */
-    @ModifyExpressionValue(method = "kickUnlistedPlayers", at = @At(value = "INVOKE", target = "Lnet/minecraft/server/players/UserWhiteList;isWhiteListed(Lcom/mojang/authlib/GameProfile;)Z"))
-    private boolean isOpCheck(boolean original, @Local ServerPlayer serverPlayer) {
-        return original || this.getPlayerList().isOp(serverPlayer.getGameProfile());
+    @WrapWithCondition(method = "kickUnlistedPlayers", at = @At(value = "INVOKE", target = "Lnet/minecraft/server/network/ServerGamePacketListenerImpl;disconnect(Lnet/minecraft/network/chat/Component;)V"))
+    private boolean isOpCheck(ServerGamePacketListenerImpl instance, Component component, @Local ServerPlayer serverPlayer) {
+        return !this.getPlayerList().isOp(serverPlayer.getGameProfile());
     }
 }