Fixed a bunch of issues where chunks would unintentionally be loaded by RS. Fixes part of #3468

CHANGELOG.md

@@ -10,6 +10,7 @@ to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 ### Fixed
 
 - Fixed not being able to search with JEI when the Grid is open.
+- Fixed a bunch of issues where chunks would unintentionally be loaded by RS.
 - Reduced block updates when a controller is turning on and off constantly.
 
 ## [v1.11.5] - 2023-02-12

src/main/java/com/refinedmods/refinedstorage/apiimpl/network/Network.java

@@ -101,8 +101,10 @@ public class Network implements INetwork, IRedstoneConfigurable {
         this.type = type;
         this.root = new RootNetworkNode(this, level, pos);
         this.nodeGraph.addListener(() -> {
+            if (!level.isLoaded(pos)) {
+                return;
+            }
             BlockEntity blockEntity = level.getBlockEntity(pos);
-
             if (blockEntity instanceof ControllerBlockEntity) {
                 ((ControllerBlockEntity) blockEntity).getDataManager().sendParameterToWatchers(ControllerBlockEntity.NODES);
             }

src/main/java/com/refinedmods/refinedstorage/apiimpl/network/NetworkNodeGraph.java

@@ -118,9 +118,11 @@ public class NetworkNodeGraph implements INetworkNodeGraph {
                 for (Direction checkSide : Direction.values()) {
                     if (checkSide != side) { // Avoid going backward
                         INetworkNode nodeOnSide = NetworkUtils.getNodeFromBlockEntity(blockEntity);
-
                         if (nodeOnSide == node) {
-                            operator.apply(level, pos.relative(checkSide), checkSide.getOpposite());
+                            BlockPos relativePos = pos.relative(checkSide);
+                            if (level.isLoaded(relativePos)) {
+                                operator.apply(level, relativePos, checkSide.getOpposite());
+                            }
                         }
                     }
                 }

src/main/java/com/refinedmods/refinedstorage/apiimpl/network/node/DestructorNetworkNode.java

@@ -100,6 +100,9 @@ public class DestructorNetworkNode extends NetworkNode implements IComparable, I
 
     private void pickupItems() {
         BlockPos front = pos.relative(getDirection());
+        if (!level.isLoaded(front)) {
+            return;
+        }
 
         List<ItemEntity> droppedItems = level.getEntitiesOfClass(ItemEntity.class, new AABB(front));
 
@@ -124,6 +127,9 @@ public class DestructorNetworkNode extends NetworkNode implements IComparable, I
 
     private void breakBlock() {
         BlockPos front = pos.relative(getDirection());
+        if (!level.isLoaded(front)) {
+            return;
+        }
         BlockState frontBlockState = level.getBlockState(front);
         Block frontBlock = frontBlockState.getBlock();
         ItemStack frontStack = frontBlock.getCloneItemStack(
@@ -174,6 +180,10 @@ public class DestructorNetworkNode extends NetworkNode implements IComparable, I
 
     private void breakFluid() {
         BlockPos front = pos.relative(getDirection());
+        if (!level.isLoaded(front)) {
+            return;
+        }
+
         BlockState frontBlockState = level.getBlockState(front);
         Block frontBlock = frontBlockState.getBlock();
 

src/main/java/com/refinedmods/refinedstorage/apiimpl/network/node/FluidInterfaceNetworkNode.java

@@ -45,7 +45,7 @@ public class FluidInterfaceNetworkNode extends NetworkNode {
         protected void onContentsChanged() {
             super.onContentsChanged();
 
-            if (!level.isClientSide) {
+            if (!level.isClientSide && level.isLoaded(pos)) {
                 ((FluidInterfaceBlockEntity) level.getBlockEntity(pos)).getDataManager().sendParameterToWatchers(FluidInterfaceBlockEntity.TANK_IN);
             }
 

src/main/java/com/refinedmods/refinedstorage/apiimpl/network/node/NetworkNode.java

@@ -237,12 +237,16 @@ public abstract class NetworkNode implements INetworkNode, INetworkNodeVisitor {
     @Override
     public void visit(Operator operator) {
         for (Direction facing : Direction.values()) {
-            INetworkNode oppositeNode = NetworkUtils.getNodeFromBlockEntity(level.getBlockEntity(pos.relative(facing)));
+            BlockPos facingPos = pos.relative(facing);
+            if (!level.isLoaded(facingPos)) {
+                return;
+            }
+            INetworkNode oppositeNode = NetworkUtils.getNodeFromBlockEntity(level.getBlockEntity(facingPos));
             if (oppositeNode == null) {
                 continue;
             }
             if (canConduct(facing) && oppositeNode.canReceive(facing.getOpposite())) {
-                operator.apply(level, pos.relative(facing), facing.getOpposite());
+                operator.apply(level, facingPos, facing.getOpposite());
             }
         }
     }

src/main/java/com/refinedmods/refinedstorage/apiimpl/network/node/NetworkTransmitterNetworkNode.java

@@ -116,11 +116,11 @@ public class NetworkTransmitterNetworkNode extends NetworkNode {
             if (!isSameDimension()) {
                 Level dimensionLevel = level.getServer().getLevel(receiverDimension);
 
-                if (dimensionLevel != null && dimensionLevel.getBlockEntity(receiver) instanceof NetworkReceiverBlockEntity) {
+                if (dimensionLevel != null && dimensionLevel.isLoaded(receiver) && dimensionLevel.getBlockEntity(receiver) instanceof NetworkReceiverBlockEntity) {
                     operator.apply(dimensionLevel, receiver, null);
                 }
             } else {
-                if (level.getBlockEntity(receiver) instanceof NetworkReceiverBlockEntity) {
+                if (level.isLoaded(receiver) && level.getBlockEntity(receiver) instanceof NetworkReceiverBlockEntity) {
                     operator.apply(level, receiver, null);
                 }
             }

src/main/java/com/refinedmods/refinedstorage/apiimpl/network/node/RootNetworkNode.java

@@ -107,7 +107,11 @@ public class RootNetworkNode implements INetworkNode, INetworkNodeVisitor {
     @Override
     public void visit(Operator operator) {
         for (Direction facing : Direction.values()) {
-            operator.apply(level, pos.relative(facing), facing.getOpposite());
+            BlockPos relativePos = pos.relative(facing);
+            if (!level.isLoaded(relativePos)) {
+                continue;
+            }
+            operator.apply(level, relativePos, facing.getOpposite());
         }
     }
 }

src/main/java/com/refinedmods/refinedstorage/blockentity/craftingmonitor/WirelessCraftingMonitor.java

@@ -4,15 +4,14 @@ import com.refinedmods.refinedstorage.RS;
 import com.refinedmods.refinedstorage.api.autocrafting.ICraftingManager;
 import com.refinedmods.refinedstorage.api.autocrafting.task.ICraftingTask;
 import com.refinedmods.refinedstorage.api.network.INetwork;
+import com.refinedmods.refinedstorage.blockentity.data.BlockEntitySynchronizationParameter;
 import com.refinedmods.refinedstorage.inventory.player.PlayerSlot;
 import com.refinedmods.refinedstorage.item.NetworkItem;
 import com.refinedmods.refinedstorage.item.WirelessCraftingMonitorItem;
 import com.refinedmods.refinedstorage.network.craftingmonitor.WirelessCraftingMonitorSettingsUpdateMessage;
-import com.refinedmods.refinedstorage.blockentity.data.BlockEntitySynchronizationParameter;
 import com.refinedmods.refinedstorage.util.NetworkUtils;
 import net.minecraft.core.BlockPos;
 import net.minecraft.network.chat.Component;
-
 import net.minecraft.resources.ResourceKey;
 import net.minecraft.server.MinecraftServer;
 import net.minecraft.server.level.ServerPlayer;
@@ -99,11 +98,14 @@ public class WirelessCraftingMonitor implements ICraftingMonitor {
 
     private INetwork getNetwork() {
         Level level = server.getLevel(nodeDimension);
-        if (level != null) {
+        if (level == null) {
-            return NetworkUtils.getNetworkFromNode(NetworkUtils.getNodeFromBlockEntity(level.getBlockEntity(nodePos)));
-        }
             return null;
         }
+        if (!level.isLoaded(nodePos)) {
+            return null;
+        }
+        return NetworkUtils.getNetworkFromNode(NetworkUtils.getNodeFromBlockEntity(level.getBlockEntity(nodePos)));
+    }
 
     public ItemStack getStack() {
         return stack;

src/main/java/com/refinedmods/refinedstorage/blockentity/grid/WirelessFluidGrid.java

@@ -22,7 +22,6 @@ import com.refinedmods.refinedstorage.util.StackUtils;
 import net.minecraft.core.BlockPos;
 import net.minecraft.nbt.CompoundTag;
 import net.minecraft.network.chat.Component;
-
 import net.minecraft.resources.ResourceKey;
 import net.minecraft.server.MinecraftServer;
 import net.minecraft.server.level.ServerPlayer;
@@ -93,11 +92,14 @@ public class WirelessFluidGrid implements INetworkAwareGrid {
     @Nullable
     public INetwork getNetwork() {
         Level level = server.getLevel(nodeDimension);
-        if (level != null) {
+        if (level == null) {
-            return NetworkUtils.getNetworkFromNode(NetworkUtils.getNodeFromBlockEntity(level.getBlockEntity(nodePos)));
-        }
             return null;
         }
+        if (!level.isLoaded(nodePos)) {
+            return null;
+        }
+        return NetworkUtils.getNetworkFromNode(NetworkUtils.getNodeFromBlockEntity(level.getBlockEntity(nodePos)));
+    }
 
     @Override
     public IStorageCacheListener createListener(ServerPlayer player) {

src/main/java/com/refinedmods/refinedstorage/blockentity/grid/WirelessGrid.java

@@ -95,11 +95,14 @@ public class WirelessGrid implements INetworkAwareGrid {
     @Nullable
     public INetwork getNetwork() {
         Level level = server.getLevel(nodeDimension);
-        if (level != null) {
+        if (level == null) {
-            return NetworkUtils.getNetworkFromNode(NetworkUtils.getNodeFromBlockEntity(level.getBlockEntity(nodePos)));
-        }
             return null;
         }
+        if (!level.isLoaded(nodePos)) {
+            return null;
+        }
+        return NetworkUtils.getNetworkFromNode(NetworkUtils.getNodeFromBlockEntity(level.getBlockEntity(nodePos)));
+    }
 
     @Override
     public IStorageCacheListener createListener(ServerPlayer player) {

src/main/java/com/refinedmods/refinedstorage/item/NetworkItem.java

@@ -104,7 +104,13 @@ public abstract class NetworkItem extends EnergyItem implements INetworkItemProv
             return;
         }
 
-        INetwork network = NetworkUtils.getNetworkFromNode(NetworkUtils.getNodeFromBlockEntity(nodeLevel.getBlockEntity(new BlockPos(getX(stack), getY(stack), getZ(stack)))));
+        BlockPos pos = new BlockPos(getX(stack), getY(stack), getZ(stack));
+        if (!nodeLevel.isLoaded(pos)) {
+            onError.accept(notFound);
+            return;
+        }
+
+        INetwork network = NetworkUtils.getNetworkFromNode(NetworkUtils.getNodeFromBlockEntity(nodeLevel.getBlockEntity(pos)));
         if (network == null) {
             onError.accept(notFound);
             return;