Fix omnihopper not remembering upgrades & direction on placement

#1209
TeamPneumatic · Sep 10, 2023 · 5d1bac4 · 5d1bac4
1 parent f014ce6
commit 5d1bac4
Showing 1 changed file with 7 additions and 10 deletions.
diff --git a/src/main/java/me/desht/pneumaticcraft/common/block/entity/AbstractHopperBlockEntity.java b/src/main/java/me/desht/pneumaticcraft/common/block/entity/AbstractHopperBlockEntity.java
@@ -56,6 +56,7 @@ public abstract class AbstractHopperBlockEntity<T extends BlockEntity & IRedston
     AABB outputAABB;
     final List<Entity> cachedInputEntities = new ArrayList<>();
     final List<Entity> cachedOutputEntities = new ArrayList<>();
+    private boolean firstTick = true;
 
     AbstractHopperBlockEntity(BlockEntityType<?> type, BlockPos pos, BlockState state) {
         super(type, pos, state, 4);
@@ -65,16 +66,6 @@ public Direction getInputDirection() {
         return getBlockState().getValue(OmnidirectionalHopperBlock.INPUT_FACING);
     }
 
-    @Override
-    public void onLoad() {
-        super.onLoad();
-
-        if (!nonNullLevel().isClientSide) {
-            isCreative = getUpgrades(ModUpgrades.CREATIVE.get()) > 0;
-            setupInputOutputRegions();
-        }
-    }
-
     @Override
     public void onBlockRotated() {
         super.onBlockRotated();
@@ -87,6 +78,12 @@ public void onBlockRotated() {
     public void tickServer() {
         inputDir = getInputDirection();
 
+        if (firstTick) {
+            isCreative = getUpgrades(ModUpgrades.CREATIVE.get()) > 0;
+            setupInputOutputRegions();
+            firstTick = false;
+        }
+
         super.tickServer();
 
         if (getRedstoneController().shouldRun()) {