refactor: move ClientProxy to forge layer

refinedmods · Nov 3, 2023 · 82658d8 · 82658d8
1 parent e3c4298
commit 82658d8
Show file tree

Hide file tree

Showing 9 changed files with 6 additions and 15 deletions.
diff --git a/...m/refinedmods/refinedstorage2/platform/common/support/resource/ItemResourceRendering.java b/...m/refinedmods/refinedstorage2/platform/common/support/resource/ItemResourceRendering.java
@@ -3,9 +3,7 @@
 import com.refinedmods.refinedstorage2.platform.api.support.AmountFormatting;
 import com.refinedmods.refinedstorage2.platform.api.support.resource.ItemResource;
 import com.refinedmods.refinedstorage2.platform.api.support.resource.ResourceRendering;
-import com.refinedmods.refinedstorage2.platform.common.util.ClientProxy;
 
-import java.util.Collections;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -50,10 +48,10 @@ public Component getDisplayName(final ItemResource resource) {
     @Override
     public List<Component> getTooltip(final ItemResource resource) {
         final Minecraft minecraft = Minecraft.getInstance();
-        return ClientProxy.getPlayer().map(player -> getStack(resource).getTooltipLines(
-            player,
+        return getStack(resource).getTooltipLines(
+            minecraft.player,
             minecraft.options.advancedItemTooltips ? TooltipFlag.ADVANCED : TooltipFlag.NORMAL
-        )).orElse(Collections.emptyList());
+        );
     }
 
     @Override

diff --git a/...ge2/platform/common/util/ClientProxy.java → ...latform/forge/packet/s2c/ClientProxy.java b/...ge2/platform/common/util/ClientProxy.java → ...latform/forge/packet/s2c/ClientProxy.java
@@ -1,16 +1,16 @@
-package com.refinedmods.refinedstorage2.platform.common.util;
+package com.refinedmods.refinedstorage2.platform.forge.packet.s2c;
 
 import java.util.Optional;
 
 import net.minecraft.client.Minecraft;
 import net.minecraft.world.entity.player.Player;
 
-public final class ClientProxy {
+final class ClientProxy {
     private ClientProxy() {
     }
 
     // This method exists to avoid classloading errors on LocalPlayer.
-    public static Optional<Player> getPlayer() {
+    static Optional<Player> getPlayer() {
         return Optional.ofNullable(Minecraft.getInstance().player);
     }
 }
diff --git a/...com/refinedmods/refinedstorage2/platform/forge/packet/s2c/ControllerEnergyInfoPacket.java b/...com/refinedmods/refinedstorage2/platform/forge/packet/s2c/ControllerEnergyInfoPacket.java
@@ -1,7 +1,6 @@
 package com.refinedmods.refinedstorage2.platform.forge.packet.s2c;
 
 import com.refinedmods.refinedstorage2.platform.common.controller.ControllerContainerMenu;
-import com.refinedmods.refinedstorage2.platform.common.util.ClientProxy;
 
 import java.util.function.Supplier;
 

diff --git a/...main/java/com/refinedmods/refinedstorage2/platform/forge/packet/s2c/GridActivePacket.java b/...main/java/com/refinedmods/refinedstorage2/platform/forge/packet/s2c/GridActivePacket.java
@@ -1,7 +1,6 @@
 package com.refinedmods.refinedstorage2.platform.forge.packet.s2c;
 
 import com.refinedmods.refinedstorage2.api.grid.GridWatcher;
-import com.refinedmods.refinedstorage2.platform.common.util.ClientProxy;
 
 import java.util.function.Supplier;
 

diff --git a/.../main/java/com/refinedmods/refinedstorage2/platform/forge/packet/s2c/GridClearPacket.java b/.../main/java/com/refinedmods/refinedstorage2/platform/forge/packet/s2c/GridClearPacket.java
@@ -1,7 +1,6 @@
 package com.refinedmods.refinedstorage2.platform.forge.packet.s2c;
 
 import com.refinedmods.refinedstorage2.platform.common.grid.AbstractGridContainerMenu;
-import com.refinedmods.refinedstorage2.platform.common.util.ClientProxy;
 
 import java.util.function.Supplier;
 

diff --git a/...main/java/com/refinedmods/refinedstorage2/platform/forge/packet/s2c/GridUpdatePacket.java b/...main/java/com/refinedmods/refinedstorage2/platform/forge/packet/s2c/GridUpdatePacket.java
@@ -4,7 +4,6 @@
 import com.refinedmods.refinedstorage2.platform.api.PlatformApi;
 import com.refinedmods.refinedstorage2.platform.api.storage.channel.PlatformStorageChannelType;
 import com.refinedmods.refinedstorage2.platform.common.grid.AbstractGridContainerMenu;
-import com.refinedmods.refinedstorage2.platform.common.util.ClientProxy;
 import com.refinedmods.refinedstorage2.platform.common.util.PacketUtil;
 
 import java.util.function.Supplier;

diff --git a/...a/com/refinedmods/refinedstorage2/platform/forge/packet/s2c/ResourceSlotUpdatePacket.java b/...a/com/refinedmods/refinedstorage2/platform/forge/packet/s2c/ResourceSlotUpdatePacket.java
@@ -4,7 +4,6 @@
 import com.refinedmods.refinedstorage2.platform.api.storage.channel.PlatformStorageChannelType;
 import com.refinedmods.refinedstorage2.platform.api.support.resource.ResourceAmountTemplate;
 import com.refinedmods.refinedstorage2.platform.common.support.containermenu.AbstractResourceContainerMenu;
-import com.refinedmods.refinedstorage2.platform.common.util.ClientProxy;
 
 import java.util.function.Supplier;
 import javax.annotation.Nullable;

diff --git a/.../com/refinedmods/refinedstorage2/platform/forge/packet/s2c/StorageInfoResponsePacket.java b/.../com/refinedmods/refinedstorage2/platform/forge/packet/s2c/StorageInfoResponsePacket.java
@@ -2,7 +2,6 @@
 
 import com.refinedmods.refinedstorage2.platform.api.PlatformApi;
 import com.refinedmods.refinedstorage2.platform.common.storage.ClientStorageRepository;
-import com.refinedmods.refinedstorage2.platform.common.util.ClientProxy;
 
 import java.util.UUID;
 import java.util.function.Supplier;

diff --git a/...refinedmods/refinedstorage2/platform/forge/packet/s2c/WirelessTransmitterRangePacket.java b/...refinedmods/refinedstorage2/platform/forge/packet/s2c/WirelessTransmitterRangePacket.java
@@ -1,6 +1,5 @@
 package com.refinedmods.refinedstorage2.platform.forge.packet.s2c;
 
-import com.refinedmods.refinedstorage2.platform.common.util.ClientProxy;
 import com.refinedmods.refinedstorage2.platform.common.wirelesstransmitter.WirelessTransmitterContainerMenu;
 
 import java.util.function.Supplier;