Fix Glow ESP

Authored by WideCat

Author: Big-Iron-Cheems

src/main/java/meteordevelopment/meteorclient/mixin/MinecraftClientMixin.java

@@ -22,6 +22,7 @@
 import meteordevelopment.meteorclient.systems.modules.Modules;
 import meteordevelopment.meteorclient.systems.modules.player.FastUse;
 import meteordevelopment.meteorclient.systems.modules.player.Multitask;
+import meteordevelopment.meteorclient.systems.modules.render.ESP;
 import meteordevelopment.meteorclient.systems.modules.world.HighwayBuilder;
 import meteordevelopment.meteorclient.utils.Utils;
 import meteordevelopment.meteorclient.utils.misc.CPSUtils;
@@ -36,6 +37,7 @@
 import net.minecraft.client.option.GameOptions;
 import net.minecraft.client.util.Window;
 import net.minecraft.client.world.ClientWorld;
+import net.minecraft.entity.Entity;
 import net.minecraft.entity.player.PlayerEntity;
 import net.minecraft.item.ItemStack;
 import net.minecraft.util.hit.HitResult;
@@ -198,7 +200,7 @@ private void onRender(CallbackInfo info) {
         lastTime = time;
     }
 
-    // multitask
+    // Multitask
 
     @ModifyExpressionValue(method = "doItemUse", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/network/ClientPlayerInteractionManager;isBreakingBlock()Z"))
     private boolean doItemUseModifyIsBreakingBlock(boolean original) {
@@ -223,6 +225,17 @@ private void handleInputEventsInjectStopUsingItem(CallbackInfo info) {
         }
     }
 
+    // Glow esp
+
+    @ModifyReturnValue(method = "hasOutline", at = @At("RETURN"))
+    private boolean hasOutlineModifyIsOutline(boolean original, Entity entity) {
+        ESP esp = Modules.get().get(ESP.class);
+        if (esp == null) return original;
+        if (!esp.isGlow() || esp.shouldSkip(entity)) return original;
+
+        return esp.getColor(entity) != null || original;
+    }
+
     // Interface
 
     @Override

src/main/java/meteordevelopment/meteorclient/mixin/WorldRendererMixin.java

@@ -111,13 +111,6 @@ private void onRender(CallbackInfo ci) {
         PostProcessShaders.endRender();
     }
 
-    @ModifyExpressionValue(method = "renderEntities", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/MinecraftClient;hasOutline(Lnet/minecraft/entity/Entity;)Z"))
-    private boolean shouldMobGlow(boolean original, @Local Entity entity) {
-        if (!getESP().isGlow() || getESP().shouldSkip(entity)) return original;
-
-        return getESP().getColor(entity) != null || original;
-    }
-
     @WrapOperation(method = "renderEntities", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/render/OutlineVertexConsumerProvider;setColor(IIII)V"))
     private void setGlowColor(OutlineVertexConsumerProvider instance, int red, int green, int blue, int alpha, Operation<Void> original, @Local LocalRef<Entity> entity) {
         if (!getESP().isGlow() || getESP().shouldSkip(entity.get())) original.call(instance, red, green, blue, alpha);