feat: assign proper soundtype for metablocks

Author: MCTian-mi

src/main/java/gregtech/api/block/IStateSoundType.java

@@ -0,0 +1,12 @@
+package gregtech.api.block;
+
+import net.minecraft.block.SoundType;
+import net.minecraft.block.state.IBlockState;
+
+import org.jetbrains.annotations.NotNull;
+
+public interface IStateSoundType {
+
+    @NotNull
+    SoundType getSoundType(@NotNull IBlockState state);
+}

src/main/java/gregtech/api/block/VariantBlock.java

@@ -4,16 +4,19 @@
 import gregtech.common.creativetab.GTCreativeTabs;
 
 import net.minecraft.block.Block;
+import net.minecraft.block.SoundType;
 import net.minecraft.block.material.Material;
 import net.minecraft.block.properties.PropertyEnum;
 import net.minecraft.block.state.BlockStateContainer;
 import net.minecraft.block.state.IBlockState;
 import net.minecraft.client.resources.I18n;
 import net.minecraft.client.util.ITooltipFlag;
 import net.minecraft.creativetab.CreativeTabs;
+import net.minecraft.entity.Entity;
 import net.minecraft.item.ItemStack;
 import net.minecraft.util.IStringSerializable;
 import net.minecraft.util.NonNullList;
+import net.minecraft.util.math.BlockPos;
 import net.minecraft.world.World;
 import net.minecraftforge.fml.relauncher.Side;
 import net.minecraftforge.fml.relauncher.SideOnly;
@@ -112,6 +115,16 @@ public int getMetaFromState(IBlockState state) {
         return state.getValue(VARIANT).ordinal();
     }
 
+    @NotNull
+    @Override
+    public SoundType getSoundType(@NotNull IBlockState state, @NotNull World world, @NotNull BlockPos pos,
+                                  @Nullable Entity entity) {
+        if (getState(state) instanceof IStateSoundType stateSoundType) {
+            return stateSoundType.getSoundType(state);
+        }
+        return super.getSoundType(state, world, pos, entity);
+    }
+
     // magic is here
     @SuppressWarnings("unchecked")
     protected static <T, R> Class<T> getActualTypeParameter(Class<? extends R> thisClass, Class<R> declaringClass) {

src/main/java/gregtech/common/blocks/BlockMetalCasing.java

@@ -1,6 +1,7 @@
 package gregtech.common.blocks;
 
 import gregtech.api.block.IStateHarvestLevel;
+import gregtech.api.block.IStateSoundType;
 import gregtech.api.block.VariantBlock;
 import gregtech.api.items.toolitem.ToolClasses;
 
@@ -31,27 +32,33 @@ public boolean canCreatureSpawn(@NotNull IBlockState state, @NotNull IBlockAcces
         return false;
     }
 
-    public enum MetalCasingType implements IStringSerializable, IStateHarvestLevel {
+    public enum MetalCasingType implements IStringSerializable, IStateHarvestLevel, IStateSoundType {
 
         BRONZE_BRICKS("bronze_bricks", 1),
-        PRIMITIVE_BRICKS("primitive_bricks", 1),
+        PRIMITIVE_BRICKS("primitive_bricks", 1, SoundType.STONE),
         INVAR_HEATPROOF("invar_heatproof", 1),
         ALUMINIUM_FROSTPROOF("aluminium_frostproof", 1),
         STEEL_SOLID("steel_solid", 2),
         STAINLESS_CLEAN("stainless_clean", 2),
         TITANIUM_STABLE("titanium_stable", 2),
         TUNGSTENSTEEL_ROBUST("tungstensteel_robust", 3),
-        COKE_BRICKS("coke_bricks", 1),
+        COKE_BRICKS("coke_bricks", 1, SoundType.STONE),
         PTFE_INERT_CASING("ptfe_inert", 0),
         HSSE_STURDY("hsse_sturdy", 3),
         PALLADIUM_SUBSTATION("palladium_substation", 3);
 
         private final String name;
         private final int harvestLevel;
+        private final SoundType soundType;
 
-        MetalCasingType(String name, int harvestLevel) {
+        MetalCasingType(String name, int harvestLevel, SoundType soundType) {
             this.name = name;
             this.harvestLevel = harvestLevel;
+            this.soundType = soundType;
+        }
+
+        MetalCasingType(String name, int harvestLevel) {
+            this(name, harvestLevel, SoundType.METAL);
         }
 
         @NotNull
@@ -69,5 +76,11 @@ public int getHarvestLevel(IBlockState state) {
         public String getHarvestTool(IBlockState state) {
             return ToolClasses.WRENCH;
         }
+
+        @NotNull
+        @Override
+        public SoundType getSoundType(@NotNull IBlockState state) {
+            return soundType;
+        }
     }
 }

src/main/java/gregtech/common/blocks/BlockSteamCasing.java

@@ -1,6 +1,7 @@
 package gregtech.common.blocks;
 
 import gregtech.api.block.IStateHarvestLevel;
+import gregtech.api.block.IStateSoundType;
 import gregtech.api.block.VariantBlock;
 import gregtech.api.items.toolitem.ToolClasses;
 
@@ -51,21 +52,27 @@ public void addInformation(@NotNull ItemStack stack, @Nullable World player, @No
         }
     }
 
-    public enum SteamCasingType implements IStringSerializable, IStateHarvestLevel {
+    public enum SteamCasingType implements IStringSerializable, IStateHarvestLevel, IStateSoundType {
 
         BRONZE_HULL("bronze_hull", 1),
         BRONZE_BRICKS_HULL("bronze_bricks_hull", 1),
         STEEL_HULL("steel_hull", 2),
         STEEL_BRICKS_HULL("steel_bricks_hull", 2),
-        PUMP_DECK("pump_deck", 1),
-        WOOD_WALL("wood_wall", 0);
+        PUMP_DECK("pump_deck", 1, SoundType.WOOD),
+        WOOD_WALL("wood_wall", 0, SoundType.WOOD);
 
         private final String name;
         private final int harvestLevel;
+        private final SoundType soundType;
 
-        SteamCasingType(String name, int harvestLevel) {
+        SteamCasingType(String name, int harvestLevel, SoundType soundType) {
             this.name = name;
             this.harvestLevel = harvestLevel;
+            this.soundType = soundType;
+        }
+
+        SteamCasingType(String name, int harvestLevel) {
+            this(name, harvestLevel, SoundType.METAL);
         }
 
         @Override
@@ -83,5 +90,11 @@ public int getHarvestLevel(IBlockState state) {
         public String getHarvestTool(IBlockState state) {
             return ToolClasses.WRENCH;
         }
+
+        @NotNull
+        @Override
+        public SoundType getSoundType(@NotNull IBlockState state) {
+            return soundType;
+        }
     }
 }