Fix crash when placing some blocks in Prod due to AT level not being properly recognized

Author: pupnewfster

src/main/java/mekanism/common/attachments/containers/chemical/ComponentBackedChemicalTank.java

@@ -20,7 +20,7 @@
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
-@NothingNullByDefault//TODO - 1.20.5: BasicChemicalTank also implement IChemicalHandler. Do we want to be doing that here?
+@NothingNullByDefault
 public abstract class ComponentBackedChemicalTank<CHEMICAL extends Chemical<CHEMICAL>, STACK extends ChemicalStack<CHEMICAL>,
       ATTACHED extends IAttachedContainers<STACK, ATTACHED>> extends ComponentBackedContainer<STACK, ATTACHED> implements IChemicalTank<CHEMICAL, STACK> {
 
@@ -54,9 +54,13 @@ protected boolean isEmpty(STACK value) {
         return value.isEmpty();
     }
 
+    /**
+     * {@inheritDoc}
+     *
+     * @apiNote Try to minimize the number of calls to this method so that we don't have to look up the data component multiple times.
+     */
     @Override
     public STACK getStack() {
-        //TODO - 1.20.5: Similar to getBasicInventorySlot do we want to reduce calls to this? Probably (We mostly do so, but we probably want to add a note here)
         return getContents(getAttached());
     }
 

src/main/java/mekanism/common/attachments/containers/chemical/merged/MergedTankCreator.java

@@ -15,7 +15,7 @@
 import net.neoforged.neoforge.common.util.INBTSerializable;
 import org.jetbrains.annotations.Nullable;
 
-//TODO - 1.20.5: Re-evaluate/rethink this as using rawtypes to get around things like this is very cursed
+//TODO: Re-evaluate/rethink this as using rawtypes to get around things like this is very cursed
 @NothingNullByDefault
 @SuppressWarnings({"rawtypes", "unchecked"})
 public class MergedTankCreator implements IBasicContainerCreator {

src/main/java/mekanism/common/attachments/containers/energy/ComponentBackedEnergyContainer.java

@@ -49,9 +49,13 @@ protected boolean isEmpty(FloatingLong value) {
         return ContainerType.ENERGY;
     }
 
+    /**
+     * {@inheritDoc}
+     *
+     * @apiNote Try to minimize the number of calls to this method so that we don't have to look up the data component multiple times.
+     */
     @Override
     public FloatingLong getEnergy() {
-        //TODO - 1.20.5: Similar to getBasicInventorySlot do we want to reduce calls to this? Probably (We mostly do so, but we probably want to add a note here)
         return getContents(getAttached());
     }
 

src/main/java/mekanism/common/attachments/containers/fluid/ComponentBackedFluidTank.java

@@ -53,9 +53,13 @@ protected boolean isEmpty(FluidStack value) {
         return ContainerType.FLUID;
     }
 
+    /**
+     * {@inheritDoc}
+     *
+     * @apiNote Try to minimize the number of calls to this method so that we don't have to look up the data component multiple times.
+     */
     @Override
     public FluidStack getFluid() {
-        //TODO - 1.20.5: Similar to getBasicInventorySlot do we want to reduce calls to this? Probably (We mostly do so, but we probably want to add a note here)
         return getContents(getAttached());
     }
 

src/main/java/mekanism/common/attachments/containers/heat/ComponentBackedHeatCapacitor.java

@@ -55,8 +55,12 @@ protected HeatCapacitorData getContents(AttachedHeat attached) {
         return attached.get(containerIndex);
     }
 
+    /**
+     * {@inheritDoc}
+     *
+     * @apiNote Try to minimize the number of calls to this method so that we don't have to look up the data component multiple times.
+     */
     protected HeatCapacitorData getData() {
-        //TODO - 1.20.5: Similar to getBasicInventorySlot do we want to reduce calls to this? Probably (We mostly do so, but we probably want to add a note here)
         return getContents(getAttached());
     }
 

src/main/java/mekanism/common/attachments/containers/item/ComponentBackedInventorySlot.java

@@ -57,9 +57,13 @@ protected boolean isEmpty(ItemStack value) {
         return ContainerType.ITEM;
     }
 
+    /**
+     * {@inheritDoc}
+     *
+     * @apiNote Try to minimize the number of calls to this method so that we don't have to look up the data component multiple times.
+     */
     @Override
     public ItemStack getStack() {
-        //TODO - 1.20.5: Similar to getBasicInventorySlot do we want to reduce calls to this? Probably (We mostly do so, but we probably want to add a note here)
         return getContents(getAttached());
     }
 

src/main/resources/META-INF/accesstransformer.cfg

@@ -93,7 +93,7 @@ protected net.minecraft.world.item.ArmorItem ARMOR_MODIFIER_UUID_PER_TYPE
 
 public net.minecraft.world.level.Explosion addOrAppendStack(Ljava/util/List;Lnet/minecraft/world/item/ItemStack;Lnet/minecraft/core/BlockPos;)V
 public net.minecraft.world.level.biome.Biome getTemperature(Lnet/minecraft/core/BlockPos;)F
-protected net.minecraft.world.level.block.entity.BlockEntity$ComponentHelper
+public net.minecraft.world.level.block.entity.BlockEntity$ComponentHelper
 public net.minecraft.world.level.block.entity.BlockEntity$DataComponentInput
 public net.minecraft.world.level.block.state.BlockBehaviour$BlockStateBase destroySpeed
 public net.minecraft.world.level.block.state.BlockBehaviour$Properties lightEmission