Fix providing an item energy cap that can't actually even be interacted with for laser amplifiers and tractor beams (#8097)

Author: pupnewfster

src/main/java/mekanism/common/item/block/ItemBlockInductionCell.java

@@ -36,7 +36,7 @@ protected void addStats(@NotNull ItemStack stack, @NotNull Item.TooltipContext c
     }
 
     @Override
-    protected boolean exposesEnergyCap() {
+    protected boolean exposesEnergyCapOrTooltips() {
         return false;
     }
 }

src/main/java/mekanism/common/item/block/ItemBlockLaserAmplifier.java

@@ -27,4 +27,10 @@ public ItemBlockLaserAmplifier(BlockTileModel<TileEntityLaserAmplifier, BlockTyp
         //Don't allow charging laser amplifiers inside of energy storage devices
         return BasicEnergyContainer.manualOnly;
     }
+
+    @Override
+    protected boolean exposesEnergyCap() {
+        //Don't allow charging laser amplifiers inside of energy storage devices
+        return false;
+    }
 }

src/main/java/mekanism/common/item/block/ItemBlockTooltip.java

@@ -122,14 +122,14 @@ protected void addDetails(@NotNull ItemStack stack, @NotNull Item.TooltipContext
 
     protected void addTypeDetails(@NotNull ItemStack stack, @NotNull Item.TooltipContext context, @NotNull List<Component> tooltip, @NotNull TooltipFlag flag) {
         //Put this here so that energy cubes can skip rendering energy here
-        if (exposesEnergyCap()) {
+        if (exposesEnergyCapOrTooltips()) {
             StorageUtils.addStoredEnergy(stack, tooltip, false);
         }
     }
 
     @Override
     public boolean shouldCauseReequipAnimation(@NotNull ItemStack oldStack, @NotNull ItemStack newStack, boolean slotChanged) {
-        if (exposesEnergyCap()) {
+        if (exposesEnergyCapOrTooltips()) {
             //Ignore NBT for energized items causing re-equip animations
             //TODO: Only ignore the energy attachment?
             return slotChanged || oldStack.getItem() != newStack.getItem();
@@ -139,7 +139,7 @@ public boolean shouldCauseReequipAnimation(@NotNull ItemStack oldStack, @NotNull
 
     @Override
     public boolean shouldCauseBlockBreakReset(@NotNull ItemStack oldStack, @NotNull ItemStack newStack) {
-        if (exposesEnergyCap()) {
+        if (exposesEnergyCapOrTooltips()) {
             //Ignore NBT for energized items causing block break reset
             //TODO: Only ignore the energy attachment?
             return oldStack.getItem() != newStack.getItem();
@@ -152,6 +152,10 @@ public boolean shouldCauseBlockBreakReset(@NotNull ItemStack oldStack, @NotNull
     }
 
     protected boolean exposesEnergyCap() {
+        return exposesEnergyCapOrTooltips();
+    }
+
+    protected boolean exposesEnergyCapOrTooltips() {
         return Attribute.has(getBlock(), AttributeEnergy.class);
     }
 

src/main/java/mekanism/common/item/block/machine/ItemBlockLaserTractorBeam.java

@@ -19,4 +19,10 @@ public ItemBlockLaserTractorBeam(BlockTile<?, ?> block, Item.Properties properti
         //Don't allow charging laser tractor beams inside of energy storage devices
         return BasicEnergyContainer.manualOnly;
     }
+
+    @Override
+    protected boolean exposesEnergyCap() {
+        //Don't allow charging laser amplifiers inside of energy storage devices
+        return false;
+    }
 }