Auto update machine view every few ticks

This is configurable on the server.

Author: davenonymous

src/main/java/org/dave/compactmachines3/gui/GuiHandler.java

@@ -1,6 +1,7 @@
 package org.dave.compactmachines3.gui;
 
 import net.minecraft.entity.player.EntityPlayer;
+import net.minecraft.util.math.BlockPos;
 import net.minecraft.world.World;
 import net.minecraftforge.fml.common.network.IGuiHandler;
 import net.minecraftforge.fml.common.network.NetworkRegistry;
@@ -20,7 +21,7 @@ public static void init() {
     @Override
     public Object getServerGuiElement(int ID, EntityPlayer player, World world, int x, int y, int z) {
         if(ID == GuiIds.MACHINE_VIEW.ordinal() || ID == GuiIds.MACHINE_ADMIN.ordinal()) {
-            return new GuiMachineContainer();
+            return new GuiMachineContainer(world, new BlockPos(x, y, z));
         }
 
         return null;
@@ -33,9 +34,9 @@ public Object getClientGuiElement(int ID, EntityPlayer player, World world, int
             Pages.activePageOnClient = "welcome";
             return new GuiPSDScreen();
         } else if(ID == GuiIds.MACHINE_VIEW.ordinal()) {
-            return new GuiMachine();
+            return new GuiMachine(world, new BlockPos(x, y, z));
         } else if(ID == GuiIds.MACHINE_ADMIN.ordinal()) {
-            return new GuiMachineAdmin();
+            return new GuiMachineAdmin(world, new BlockPos(x, y, z));
         }
 
         return null;

src/main/java/org/dave/compactmachines3/gui/machine/GuiMachine.java

@@ -1,5 +1,6 @@
 package org.dave.compactmachines3.gui.machine;
 
+import net.minecraft.block.Block;
 import net.minecraft.block.state.IBlockState;
 import net.minecraft.client.Minecraft;
 import net.minecraft.client.gui.GuiButton;
@@ -19,6 +20,7 @@
 import net.minecraft.util.ITickable;
 import net.minecraft.util.ResourceLocation;
 import net.minecraft.util.math.BlockPos;
+import net.minecraft.world.World;
 import net.minecraftforge.client.ForgeHooksClient;
 import net.minecraftforge.fml.client.config.GuiCheckBox;
 import org.dave.compactmachines3.CompactMachines3;
@@ -60,8 +62,8 @@ public class GuiMachine extends GuiContainer {
     int glListId = -1;
     int activeTab = 0;  // TODO: This should not be an integer, but rather a GuiTab object or something like that
 
-    public GuiMachine() {
-        super(new GuiMachineContainer());
+    public GuiMachine(World world, BlockPos pos) {
+        super(new GuiMachineContainer(world, pos));
     }
 
     private boolean shouldShowTabs() {

src/main/java/org/dave/compactmachines3/gui/machine/GuiMachineAdmin.java

@@ -2,7 +2,9 @@
 
 import net.minecraft.client.gui.GuiButton;
 import net.minecraft.client.resources.I18n;
+import net.minecraft.util.math.BlockPos;
 import net.minecraft.util.text.TextFormatting;
+import net.minecraft.world.World;
 import org.dave.compactmachines3.network.MessageRequestMachineAction;
 import org.dave.compactmachines3.network.PackageHandler;
 
@@ -20,6 +22,10 @@ public class GuiMachineAdmin extends GuiMachine {
     private List<GuiButton> adminButtons;
     private int buttonIdOffset = 3;
 
+    public GuiMachineAdmin(World world, BlockPos pos) {
+        super(world, pos);
+    }
+
     @Override
     public void initGui() {
         super.initGui();

src/main/java/org/dave/compactmachines3/gui/machine/GuiMachineContainer.java

@@ -1,14 +1,61 @@
 package org.dave.compactmachines3.gui.machine;
 
 import net.minecraft.entity.player.EntityPlayer;
+import net.minecraft.entity.player.EntityPlayerMP;
 import net.minecraft.inventory.Container;
+import net.minecraft.inventory.IContainerListener;
+import net.minecraft.tileentity.TileEntity;
+import net.minecraft.util.math.BlockPos;
+import net.minecraft.world.World;
+import org.dave.compactmachines3.misc.ConfigurationHandler;
+import org.dave.compactmachines3.network.MessageMachineChunk;
+import org.dave.compactmachines3.network.MessageMachineContent;
+import org.dave.compactmachines3.network.PackageHandler;
+import org.dave.compactmachines3.tile.TileEntityMachine;
+import org.dave.compactmachines3.utility.Logz;
 
 public class GuiMachineContainer extends Container {
-    public GuiMachineContainer() {
+    World world;
+    BlockPos pos;
+
+    public GuiMachineContainer(World world, BlockPos pos) {
+        this.world = world;
+        this.pos = pos;
     }
 
     @Override
     public boolean canInteractWith(EntityPlayer playerIn) {
         return true;
     }
+
+    @Override
+    public void detectAndSendChanges() {
+        super.detectAndSendChanges();
+
+        if(ConfigurationHandler.MachineSettings.autoUpdateRate <= 0) {
+            return;
+        }
+
+        if (this.world.isRemote || this.world.getTotalWorldTime() % ConfigurationHandler.MachineSettings.autoUpdateRate != 0) {
+            return;
+        }
+
+        TileEntity te = this.world.getTileEntity(this.pos);
+
+        if(te == null || !(te instanceof TileEntityMachine)) {
+            return;
+        }
+
+        TileEntityMachine machine = (TileEntityMachine) te;
+        int coords = machine.coords;
+
+        for(IContainerListener listener : this.listeners) {
+            if (!(listener instanceof EntityPlayerMP)) {
+                continue;
+            }
+
+            PackageHandler.instance.sendTo(new MessageMachineContent(coords), (EntityPlayerMP) listener);
+            PackageHandler.instance.sendTo(new MessageMachineChunk(coords), (EntityPlayerMP) listener);
+        }
+    }
 }

src/main/java/org/dave/compactmachines3/misc/ConfigurationHandler.java

@@ -164,6 +164,14 @@ private static void loadConfiguration() {
                 "Render living entities in GUI"
         );
 
+        MachineSettings.autoUpdateRate = configuration.getInt(
+                "autoUpdateRate",
+                CATEGORY_MACHINES,
+                20,
+                0, Integer.MAX_VALUE,
+                "Update the machine view GUI every n ticks. Set to 0 to disable. Lower values decrease performance significantly!"
+        );
+
         CompatSettings.doesWaterVaporize = configuration.getBoolean(
                 "doesWaterVaporize",
                 CATEGORY_COMPAT,
@@ -204,6 +212,7 @@ public static class MachineSettings {
         public static boolean allowEnteringWithoutPSD;
         public static boolean renderTileEntitiesInGUI;
         public static boolean renderLivingEntitiesInGUI;
+        public static int autoUpdateRate;
     }
 
     public static class Settings {