Jump to content

[1.12.2] Registering blocks with metadata


PyralIron

Recommended Posts

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):

2018-08-14_02_10_46.png.0f6fd23be1a8a8cd21f3698202fef26a.png2018-08-14_02_10_56.png.00af58a1bdcdaf55ed7592febf3e6eec.png

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 by PyralIron
Link to comment
Share on other sites

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 

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

Look at ItemMultiTexture and ItemCloth

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

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!

Link to comment
Share on other sites

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 by Pheenixm
  • Like 1
ExplosivesBanner-1.gif
Link to comment
Share on other sites

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.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.