Skip to content

Fix upgrade_tier not persisting or applying to GT energy output voltage#283

Merged
igentuman merged 1 commit into
igentuman:1.20from
SkeletonAdventure:fix/fusion-energy-tier
Jun 6, 2026
Merged

Fix upgrade_tier not persisting or applying to GT energy output voltage#283
igentuman merged 1 commit into
igentuman:1.20from
SkeletonAdventure:fix/fusion-energy-tier

Conversation

@SkeletonAdventure

Copy link
Copy Markdown

In short, the EU upgrade tiers did not do anything in game for the multiblocks or fusion reactor.

Three related bugs prevented the fusion (and other multiblock) energy upgrade tier from taking effect:

  • MultiblockControllerBE.load() never read upgrade_tier from the Info NBT tag, so chunk reload always reset the energy storage to the config-default tier.
  • MultiblockControllerBE.saveAdditional() never wrote upgrade_tier to the Info tag, so the value was silently lost on unload.
  • FusionCoreBlock.setPlacedBy() called tileEntity.load() (which now reads the tag correctly) but didn't set tileEntity.upgrade_tier first, so the stored value remained 0. Same gap existed in MultiblockControllerBlock.setPlacedBy().

All three sites now read/write upgrade_tier and call updateEnergyTier() consistently.

Three related bugs prevented the fusion (and other multiblock) energy upgrade tier from taking effect:

- MultiblockControllerBE.load() never read upgrade_tier from the Info NBT tag, so chunk reload always reset the energy storage to the config-default tier.
- MultiblockControllerBE.saveAdditional() never wrote upgrade_tier to the Info tag, so the value was silently lost on unload.
- FusionCoreBlock.setPlacedBy() called tileEntity.load() (which now reads the tag correctly) but didn't set tileEntity.upgrade_tier first, so the stored value remained 0. Same gap existed in MultiblockControllerBlock.setPlacedBy().

All three sites now read/write upgrade_tier and call updateEnergyTier() consistently.
@igentuman igentuman merged commit 6948d09 into igentuman:1.20 Jun 6, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants