diff --git a/src/main/java/gregtech/api/gui/modularui/GTUIInfos.java b/src/main/java/gregtech/api/gui/modularui/GTUIInfos.java index da20d5b17eb..c57fcd715f4 100644 --- a/src/main/java/gregtech/api/gui/modularui/GTUIInfos.java +++ b/src/main/java/gregtech/api/gui/modularui/GTUIInfos.java @@ -26,7 +26,6 @@ import gregtech.api.enums.GTValues; import gregtech.api.interfaces.tileentity.ICoverable; import gregtech.api.interfaces.tileentity.IHasWorldObjectAndCoords; -import gregtech.api.net.GTPacketSendCoverData; import gregtech.common.covers.CoverInfo; public class GTUIInfos { @@ -105,11 +104,8 @@ public static void openCoverUI(ICoverable tileEntity, EntityPlayer player, Forge if (tileEntity.isClientSide()) return; GTValues.NW.sendToPlayer( - new GTPacketSendCoverData( - side, - tileEntity.getCoverIDAtSide(side), - tileEntity.getComplexCoverDataAtSide(side), - tileEntity), + tileEntity.getCoverInfoAtSide(side) + .toPacket(), (EntityPlayerMP) player); coverUI.get(side) diff --git a/src/main/java/gregtech/api/interfaces/IRedstoneCircuitBlock.java b/src/main/java/gregtech/api/interfaces/IRedstoneCircuitBlock.java index 973a0fbcccf..865073e8ce0 100644 --- a/src/main/java/gregtech/api/interfaces/IRedstoneCircuitBlock.java +++ b/src/main/java/gregtech/api/interfaces/IRedstoneCircuitBlock.java @@ -5,6 +5,7 @@ import net.minecraftforge.common.util.ForgeDirection; import gregtech.api.interfaces.tileentity.ICoverable; +import gregtech.api.util.ISerializableObject; import gregtech.common.covers.CoverInfo; /** @@ -40,7 +41,7 @@ public interface IRedstoneCircuitBlock { int getCoverID(ForgeDirection side); - int getCoverVariable(ForgeDirection side); + ISerializableObject getCoverData(ForgeDirection side); /** * returns whatever Block-ID is adjacent to the Redstone Circuit Block diff --git a/src/main/java/gregtech/api/interfaces/tileentity/ICoverable.java b/src/main/java/gregtech/api/interfaces/tileentity/ICoverable.java index 7c9f5a84322..cb8d7f6545a 100644 --- a/src/main/java/gregtech/api/interfaces/tileentity/ICoverable.java +++ b/src/main/java/gregtech/api/interfaces/tileentity/ICoverable.java @@ -35,10 +35,6 @@ default CoverInfo getCoverInfoAtSide(ForgeDirection side) { return null; } - default ISerializableObject getComplexCoverDataAtSide(ForgeDirection side) { - return new ISerializableObject.LegacyCoverData(getCoverDataAtSide(side)); - } - int getCoverIDAtSide(ForgeDirection side); ItemStack getCoverItemAtSide(ForgeDirection side); diff --git a/src/main/java/gregtech/api/metatileentity/CoverableTileEntity.java b/src/main/java/gregtech/api/metatileentity/CoverableTileEntity.java index 1c074ade549..ebe40dd48c4 100644 --- a/src/main/java/gregtech/api/metatileentity/CoverableTileEntity.java +++ b/src/main/java/gregtech/api/metatileentity/CoverableTileEntity.java @@ -333,11 +333,6 @@ public int getCoverDataAtSide(ForgeDirection side) { return 0; } - @Override - public ISerializableObject getComplexCoverDataAtSide(ForgeDirection side) { - return getCoverInfoAtSide(side).getCoverData(); - } - public final void setCoverInfoAtSide(ForgeDirection side, CoverInfo coverInfo) { if (side != ForgeDirection.UNKNOWN) { coverInfos[side.ordinal()] = coverInfo; diff --git a/src/main/java/gregtech/api/util/CoverBehaviorBase.java b/src/main/java/gregtech/api/util/CoverBehaviorBase.java index 0d646354795..5bd87163676 100644 --- a/src/main/java/gregtech/api/util/CoverBehaviorBase.java +++ b/src/main/java/gregtech/api/util/CoverBehaviorBase.java @@ -484,7 +484,8 @@ public T getCoverData() { if (isCoverValid()) { return forceCast( getUIBuildContext().getTile() - .getComplexCoverDataAtSide(getUIBuildContext().getCoverSide())); + .getCoverInfoAtSide(getUIBuildContext().getCoverSide()) + .getCoverData()); } else { return null; } diff --git a/src/main/java/gregtech/common/covers/CoverInfo.java b/src/main/java/gregtech/common/covers/CoverInfo.java index a0c49b8034e..673af3110f1 100644 --- a/src/main/java/gregtech/common/covers/CoverInfo.java +++ b/src/main/java/gregtech/common/covers/CoverInfo.java @@ -21,6 +21,7 @@ import gregtech.api.gui.modularui.CoverUIBuildContext; import gregtech.api.interfaces.ITexture; import gregtech.api.interfaces.tileentity.ICoverable; +import gregtech.api.net.GTPacketSendCoverData; import gregtech.api.objects.GTCoverNone; import gregtech.api.util.CoverBehaviorBase; import gregtech.api.util.GTUtility; @@ -381,4 +382,8 @@ public ModularUIContainer createCoverContainer(EntityPlayer player) { if (window == null) return null; return new ModularUIContainer(new ModularUIContext(buildContext, tile::markDirty), window); } + + public GTPacketSendCoverData toPacket() { + return new GTPacketSendCoverData(this.coverSide, this.coverID, this.coverData, this.coveredTile.get()); + } } diff --git a/src/main/java/gregtech/common/items/behaviors/BehaviourCoverTool.java b/src/main/java/gregtech/common/items/behaviors/BehaviourCoverTool.java index a68a0f8aec2..88ae1696f8a 100644 --- a/src/main/java/gregtech/common/items/behaviors/BehaviourCoverTool.java +++ b/src/main/java/gregtech/common/items/behaviors/BehaviourCoverTool.java @@ -22,6 +22,7 @@ import gregtech.api.util.GTLanguageManager; import gregtech.api.util.GTUtility; import gregtech.api.util.ISerializableObject; +import gregtech.common.covers.CoverInfo; public class BehaviourCoverTool extends BehaviourNone { @@ -120,12 +121,11 @@ private void doCopy(TileEntity aTileEntity, World aWorld, int aX, int aY, int aZ ? GTUtility.determineWrenchingSide(side, hitX, hitY, hitZ) : ForgeDirection.UNKNOWN; if (tSide != ForgeDirection.UNKNOWN) { - if (tCoverable.getCoverInfoAtSide(tSide) - .allowsCopyPasteTool()) { - mStoredData = tCoverable.getComplexCoverDataAtSide(tSide); - mCoverType = tCoverable.getCoverIDAtSide(tSide); - mTickRateAddition = tCoverable.getCoverInfoAtSide(tSide) - .getTickRateAddition(); + CoverInfo coverInfo = tCoverable.getCoverInfoAtSide(tSide); + if (coverInfo.allowsCopyPasteTool()) { + mStoredData = coverInfo.getCoverData(); + mCoverType = coverInfo.getCoverID(); + mTickRateAddition = coverInfo.getTickRateAddition(); aList.add("Block Side: " + EnumChatFormatting.AQUA + tSide.name() + EnumChatFormatting.RESET); aList.add( diff --git a/src/main/java/gtPlusPlus/xmod/gregtech/common/tileentities/redstone/MTERedstoneCircuitBlock.java b/src/main/java/gtPlusPlus/xmod/gregtech/common/tileentities/redstone/MTERedstoneCircuitBlock.java index bfea88daa73..9e886520c3d 100644 --- a/src/main/java/gtPlusPlus/xmod/gregtech/common/tileentities/redstone/MTERedstoneCircuitBlock.java +++ b/src/main/java/gtPlusPlus/xmod/gregtech/common/tileentities/redstone/MTERedstoneCircuitBlock.java @@ -405,8 +405,9 @@ public int getCoverID(ForgeDirection side) { } @Override - public int getCoverVariable(ForgeDirection side) { - return ((ISerializableObject.LegacyCoverData) getBaseMetaTileEntity().getComplexCoverDataAtSide(side)).get(); + public ISerializableObject getCoverData(ForgeDirection side) { + return getBaseMetaTileEntity().getCoverInfoAtSide(side) + .getCoverData(); } @Override