PyralIron Posted August 14, 2018 Share Posted August 14, 2018 (edited) I made a copy of Minecraft's BlockDirt class and modified it to go with my packages (i plan to change that later), but the sub-blocks of coarse dirt and podzol seem to not be getting unique ids and names. I'm not sure whether there's a problem in giving the different blockstates (i think that's what they are called) a registry name or something else. As you can see there are 3 different itemstacks in the creative inventory, but they all have the same id (ignore the temp texture by the way): Dirt block class: package com.pyraliron.pyralfishmod.block; import java.util.Random; import com.pyraliron.pyralfishmod.init.ModBlocks; import net.minecraft.block.Block; import net.minecraft.block.BlockDirt; import net.minecraft.block.material.MapColor; import net.minecraft.block.material.Material; import net.minecraft.block.properties.IProperty; import net.minecraft.block.properties.PropertyBool; import net.minecraft.block.properties.PropertyEnum; import net.minecraft.block.state.BlockStateContainer; import net.minecraft.block.state.IBlockState; import net.minecraft.creativetab.CreativeTabs; import net.minecraft.init.Blocks; import net.minecraft.item.Item; 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.IBlockAccess; import net.minecraft.world.World; import net.minecraftforge.fml.relauncher.Side; import net.minecraftforge.fml.relauncher.SideOnly; public class BlockDirtPyral extends BlockBase { public static final PropertyEnum<BlockDirtPyral.DirtType> VARIANT = PropertyEnum.<BlockDirtPyral.DirtType>create("variant", BlockDirtPyral.DirtType.class); public static final PropertyBool SNOWY = PropertyBool.create("snowy"); public BlockDirtPyral() { super(Material.GROUND, "dirt_block_pyral", CreativeTabs.BUILDING_BLOCKS); this.setDefaultState(this.blockState.getBaseState().withProperty(VARIANT, BlockDirtPyral.DirtType.DIRT).withProperty(SNOWY, Boolean.valueOf(false))); } public MapColor getMapColor(IBlockState state, IBlockAccess worldIn, BlockPos pos) { return ((BlockDirtPyral.DirtType)state.getValue(VARIANT)).getColor(); } /** * Get the actual Block state of this Block at the given position. This applies properties not visible in the * metadata, such as fence connections. */ public IBlockState getActualState(IBlockState state, IBlockAccess worldIn, BlockPos pos) { if (state.getValue(VARIANT) == BlockDirtPyral.DirtType.PODZOL) { Block block = worldIn.getBlockState(pos.up()).getBlock(); state = state.withProperty(SNOWY, Boolean.valueOf(block == Blocks.SNOW || block == Blocks.SNOW_LAYER)); } return state; } /** * returns a list of blocks with the same ID, but different meta (eg: wood returns 4 blocks) */ @SideOnly(Side.CLIENT) public void getSubBlocks(CreativeTabs itemIn, NonNullList<ItemStack> items) { items.add(new ItemStack(this, 1, BlockDirtPyral.DirtType.DIRT.getMetadata())); items.add(new ItemStack(this, 1, BlockDirtPyral.DirtType.COARSE_DIRT.getMetadata())); items.add(new ItemStack(this, 1, BlockDirtPyral.DirtType.PODZOL.getMetadata())); } public ItemStack getItem(World worldIn, BlockPos pos, IBlockState state) { return new ItemStack(this, 1, ((BlockDirtPyral.DirtType)state.getValue(VARIANT)).getMetadata()); } /** * Convert the given metadata into a BlockState for this Block */ public IBlockState getStateFromMeta(int meta) { return this.getDefaultState().withProperty(VARIANT, BlockDirtPyral.DirtType.byMetadata(meta)); } /** * Convert the BlockState into the correct metadata value */ public int getMetaFromState(IBlockState state) { return ((BlockDirtPyral.DirtType)state.getValue(VARIANT)).getMetadata(); } protected BlockStateContainer createBlockState() { return new BlockStateContainer(this, new IProperty[] {VARIANT, SNOWY}); } /** * Gets the metadata of the item this Block can drop. This method is called when the block gets destroyed. It * returns the metadata of the dropped item based on the old metadata of the block. */ @Override public int damageDropped(IBlockState state) { //metadata test BlockDirtPyral.DirtType blockdirt$dirttype = (BlockDirtPyral.DirtType)state.getValue(VARIANT); System.out.println(blockdirt$dirttype.getMetadata()); if (blockdirt$dirttype == BlockDirtPyral.DirtType.PODZOL) { System.out.println("Podzol"); blockdirt$dirttype = BlockDirtPyral.DirtType.DIRT; } else if (blockdirt$dirttype == BlockDirtPyral.DirtType.DIRT) { System.out.println("Dirt"); blockdirt$dirttype = BlockDirtPyral.DirtType.COARSE_DIRT; } else { System.out.println("Coarse Dirt"); blockdirt$dirttype = BlockDirtPyral.DirtType.PODZOL; } System.out.println(blockdirt$dirttype.getMetadata()); return blockdirt$dirttype.getMetadata(); } public static enum DirtType implements IStringSerializable { DIRT(0, "dirt", "default", MapColor.DIRT), COARSE_DIRT(1, "coarse_dirt", "coarse", MapColor.DIRT), PODZOL(2, "podzol", MapColor.OBSIDIAN); private static final BlockDirtPyral.DirtType[] METADATA_LOOKUP = new BlockDirtPyral.DirtType[values().length]; private final int metadata; private final String name; private final String unlocalizedName; private final MapColor color; private DirtType(int metadataIn, String nameIn, MapColor color) { this(metadataIn, nameIn, nameIn, color); } private DirtType(int metadataIn, String nameIn, String unlocalizedNameIn, MapColor color) { this.metadata = metadataIn; this.name = nameIn; this.unlocalizedName = unlocalizedNameIn; this.color = color; } public int getMetadata() { return this.metadata; } public String getUnlocalizedName() { return this.unlocalizedName; } public MapColor getColor() { return this.color; } public String toString() { return this.name; } public static BlockDirtPyral.DirtType byMetadata(int metadata) { if (metadata < 0 || metadata >= METADATA_LOOKUP.length) { metadata = 0; } return METADATA_LOOKUP[metadata]; } public String getName() { return this.name; } static { for (BlockDirtPyral.DirtType blockdirt$dirttype : values()) { METADATA_LOOKUP[blockdirt$dirttype.getMetadata()] = blockdirt$dirttype; } } } } BlockBase class: package com.pyraliron.pyralfishmod.block; import com.pyraliron.pyralfishmod.Main; import com.pyraliron.pyralfishmod.init.ModBlocks; import com.pyraliron.pyralfishmod.init.ModItems; import com.pyraliron.pyralfishmod.util.IHasModel; import net.minecraft.block.Block; import net.minecraft.block.material.Material; import net.minecraft.creativetab.CreativeTabs; import net.minecraft.item.Item; import net.minecraft.item.ItemBlock; public class BlockBase extends Block implements IHasModel { public BlockBase(Material material, String name, CreativeTabs tab) { super(material); setUnlocalizedName(name); setRegistryName(name); setCreativeTab(tab); ModBlocks.BLOCKS.add(this); ModItems.ITEMS.add(new ItemBlock(this).setRegistryName(this.getRegistryName())); } @Override public void registerModel() { Main.proxy.registerItemRenderer(Item.getItemFromBlock(this), 0, "inventory"); } } ModBlocks class: package com.pyraliron.pyralfishmod.init; import java.util.ArrayList; import java.util.List; import com.pyraliron.pyralfishmod.block.BlockBase; import com.pyraliron.pyralfishmod.block.BlockDirtPyral; import com.pyraliron.pyralfishmod.block.BlockGrassPyral; import net.minecraft.block.Block; import net.minecraft.block.material.Material; import net.minecraft.creativetab.CreativeTabs; import net.minecraft.item.Item; public class ModBlocks { public static final List<Block> BLOCKS = new ArrayList<Block>(); public static final Block GRASS_BLOCK_PYRAL = new BlockGrassPyral(); public static final Block DIRT_BLOCK_PYRAL = new BlockDirtPyral(); } RegistryHandler class: package com.pyraliron.pyralfishmod.util.handlers; import com.pyraliron.pyralfishmod.init.ModBlocks; import com.pyraliron.pyralfishmod.init.ModItems; import com.pyraliron.pyralfishmod.util.IHasModel; import net.minecraft.block.Block; import net.minecraft.item.Item; import net.minecraftforge.client.event.ModelRegistryEvent; import net.minecraftforge.event.RegistryEvent; import net.minecraftforge.fml.common.Mod.EventBusSubscriber; import net.minecraftforge.fml.common.eventhandler.SubscribeEvent; @EventBusSubscriber public class RegistryHandler { @SubscribeEvent public static void onItemRegister(RegistryEvent.Register<Item> event) { event.getRegistry().registerAll(ModItems.ITEMS.toArray(new Item[0])); } @SubscribeEvent public static void onBlockRegister(RegistryEvent.Register<Block> event) { event.getRegistry().registerAll(ModBlocks.BLOCKS.toArray(new Block[0])); } @SubscribeEvent public static void onModelRegister(ModelRegistryEvent event) { for (Item item : ModItems.ITEMS) { if (item instanceof IHasModel) { ((IHasModel)item).registerModel(); } } for (Block block : ModBlocks.BLOCKS) { if (block instanceof IHasModel) { ((IHasModel)block).registerModel(); } } } } ClientProxy: package com.pyraliron.pyralfishmod.proxy; import net.minecraft.client.renderer.block.model.ModelResourceLocation; import net.minecraft.item.Item; import net.minecraftforge.client.model.ModelLoader; public class ClientProxy extends CommonProxy { @Override public void registerItemRenderer(Item item, int meta, String id) { ModelLoader.setCustomModelResourceLocation(item, meta, new ModelResourceLocation(item.getRegistryName(), id)); } } IHasModel: package com.pyraliron.pyralfishmod.util; public interface IHasModel { public void registerModel(); } Edited August 14, 2018 by PyralIron Quote Link to comment Share on other sites More sharing options...
V0idWa1k3r Posted August 14, 2018 Share Posted August 14, 2018 That is how metadata works. You have 1 block ID that can have metadata to differentiate between different variants. You can't have unique registry names for metadata variants, that's not how this works. Code-style issues #1 and #3 Quote Link to comment Share on other sites More sharing options...
PyralIron Posted August 14, 2018 Author Share Posted August 14, 2018 But what am I doing wrong? I only have one block ID currently, and everything else came from the original BlockDirt class. Quote Link to comment Share on other sites More sharing options...
V0idWa1k3r Posted August 14, 2018 Share Posted August 14, 2018 What behaviour do you expect? If you want different block names you need to override Item#getUnlocalizedName(ItemStack) in your ItemBlock. Vanilla does this using ItemMultiTexture as the ItemBlock for the dirt block. Quote Link to comment Share on other sites More sharing options...
PyralIron Posted August 14, 2018 Author Share Posted August 14, 2018 Yes I want the different variants to have their own names. So how do I get the unlocalized name from the itemstack? Quote Link to comment Share on other sites More sharing options...
Animefan8888 Posted August 14, 2018 Share Posted August 14, 2018 14 minutes ago, PyralIron said: Yes I want the different variants to have their own names. So how do I get the unlocalized name from the itemstack? You need to determine the unlocalized name based on the metadata of the ItemStack Quote VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect. Forge and vanilla BlockState generator. Link to comment Share on other sites More sharing options...
Draco18s Posted August 14, 2018 Share Posted August 14, 2018 Look at ItemMultiTexture and ItemCloth Quote Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable. If you think this is the case, JUST REPORT ME. Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice. Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked. DO NOT PM ME WITH PROBLEMS. No help will be given. Link to comment Share on other sites More sharing options...
Pheenixm Posted August 14, 2018 Share Posted August 14, 2018 You really shouldn't be using metadata for creating subblocks, that entire practice is being obviated in 1.3. Just register each block individually, with its own registry name. You'll be happy when 1.3 shows up and breaks everything. Quote Link to comment Share on other sites More sharing options...
PyralIron Posted August 14, 2018 Author Share Posted August 14, 2018 3 hours ago, Draco18s said: Look at ItemMultiTexture and ItemCloth Oh that helps 24 minutes ago, Pheenixm said: You really shouldn't be using metadata for creating subblocks, that entire practice is being obviated in 1.3. Just register each block individually, with its own registry name. You'll be happy when 1.3 shows up and breaks everything. You mean 1.13 right? Okay I was wondering whether to do that anyway but I thought I should try to follow the system already there. Thanks guys! Quote Link to comment Share on other sites More sharing options...
Pheenixm Posted August 14, 2018 Share Posted August 14, 2018 (edited) 2 hours ago, PyralIron said: You mean 1.13 right? Okay I was wondering whether to do that anyway but I thought I should try to follow the system already there. Thanks guys! Yeah, 1.13. If you want variations based on something like color, take a look at the way that BlockShulkerBox handles its constructor. You can use that sort of thing to do registry like so new BlockVariant(EnumColor.color).setRegistryName(registryName) new BlockVariant(EnumColor.color2).setRegistryName(registryName2) And thereby reuse the same class file while creating different blocks from it, with different blockState.json files for each one Edited August 14, 2018 by Pheenixm 1 Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.