ItemBlock has no texture.

[Lambda]

Hello,

 

So I'm having an issue with my itemblock not registering a texture.

The console does not output any missing resourcelocation so I'm guessing its somewhere in my code, but I had no luck in finding it.

 

Here is my ItemUtil class:

package com.lambda.plentifulmisc.util;

/**
* Created by Blake on 11/22/2016.
*/

import com.lambda.plentifulmisc.PlentifulMisc;
import com.lambda.plentifulmisc.blocks.base.ItemBlockBase;
import net.minecraft.block.Block;
import net.minecraft.item.Item;
import net.minecraft.util.ResourceLocation;
import net.minecraftforge.fml.common.registry.GameRegistry;

import java.util.*;


public final class ItemUtil{

    public static Item getItemFromName(String name){
        ResourceLocation resLoc = new ResourceLocation(name);
        if(Item.REGISTRY.containsKey(resLoc)){
            return Item.REGISTRY.getObject(resLoc);
        }
        return null;
    }

    public static void registerBlock(Block block, ItemBlockBase itemBlock, String name, boolean addTab){
        block.setUnlocalizedName(ModUtil.MOD_ID+"."+name);

        block.setRegistryName(ModUtil.MOD_ID, name);
        GameRegistry.register(block);

        itemBlock.setRegistryName(block.getRegistryName());
        GameRegistry.register(itemBlock);

        block.setCreativeTab(addTab ? PlentifulMisc.instance.CREATIVE_TAB : null);

    }

    public static void registerItem(Item item, String name, boolean addTab){
        item.setUnlocalizedName(ModUtil.MOD_ID+"."+name);

        item.setRegistryName(ModUtil.MOD_ID, name);
        GameRegistry.register(item);

        item.setCreativeTab(addTab ? PlentifulMisc.instance.CREATIVE_TAB : null);

    }


}

 

Here is my BlockBase:

package com.lambda.plentifulmisc.blocks.base;

import com.lambda.plentifulmisc.PlentifulMisc;
import com.lambda.plentifulmisc.util.ItemUtil;
import net.minecraft.block.Block;
import net.minecraft.block.material.Material;
import net.minecraft.item.ItemStack;


/**
* Created by Blake on 11/22/2016.
*/
public class BlockBase extends Block{

    private final String name;

    public BlockBase(Material material, String name){
        super(material);
        this.name = name;

        this.register();
    }

    private void register(){
        ItemUtil.registerBlock(this, this.getItemBlock(), this.getBaseName(), this.shouldAddCreative());

        this.registerRendering();
    }

    protected String getBaseName(){
        return this.name;
    }

    protected ItemBlockBase getItemBlock(){
        return new ItemBlockBase(this);
    }

    public boolean shouldAddCreative(){
        return true;
    }

    protected void registerRendering(){
        PlentifulMisc.proxy.addRenderRegister(new ItemStack(this), this.getRegistryName(), "inventory");
    }
}

 

Here is my Block Class

 

package com.lambda.plentifulmisc.blocks;

import com.lambda.plentifulmisc.blocks.base.BlockBase;
import com.lambda.plentifulmisc.blocks.base.ItemBlockBase;
import net.minecraft.block.Block;
import net.minecraft.block.SoundType;
import net.minecraft.block.material.Material;
import net.minecraft.item.ItemStack;

/**
* Created by Blake on 11/22/2016.
*/
public class BlockCeleistrum extends BlockBase{

    BlockCeleistrum(String name) {
        super(Material.ROCK, name);
        this.setHarvestLevel("pickaxe", 0);
        this.setHardness(2.5F);
        this.setResistance(10.0F);
        this.setSoundType(SoundType.STONE);
    }

}

 

Lastly, my BlockInit:

package com.lambda.plentifulmisc.blocks;

import com.lambda.plentifulmisc.util.ModUtil;
import net.minecraft.block.Block;

/**
* Created by Blake on 11/22/2016.
*/
public class InitBlocks {

    public static Block blockCeleistrum;

    public static void init() {
        ModUtil.LOGGER.info("Initializing Blocks...");
        blockCeleistrum = new BlockCeleistrum("block_celestrium");
    }
}

 

Thanks!

[gurujive]

I have an open src on curse that is for 1.11 blocks. If you would like to look at it, It may help.

Just search for "portal block".

[Lambda]

Client Proxy:

package com.lambda.plentifulmisc.proxy;

import com.lambda.plentifulmisc.util.ModUtil;
import net.minecraft.client.renderer.block.model.ModelResourceLocation;
import net.minecraft.item.ItemStack;
import net.minecraft.util.ResourceLocation;
import net.minecraftforge.client.model.ModelLoader;
import net.minecraftforge.fml.common.event.FMLInitializationEvent;
import net.minecraftforge.fml.common.event.FMLPostInitializationEvent;
import net.minecraftforge.fml.common.event.FMLPreInitializationEvent;

import java.util.*;



/**
* Created by Blake on 11/22/2016.
*/
public class ClientProxy implements IProxy
{
    private static final Map<ItemStack, ModelResourceLocation> MODEL_LOCATIONS_FOR_REGISTERING = new HashMap<ItemStack, ModelResourceLocation>();


    @Override
    public void preInit(FMLPreInitializationEvent event) {
        ModUtil.LOGGER.info("PreInitializing ClientProxy...");

        for(Map.Entry<ItemStack, ModelResourceLocation> entry : MODEL_LOCATIONS_FOR_REGISTERING.entrySet()){
            ModelLoader.setCustomModelResourceLocation(entry.getKey().getItem(), entry.getKey().getItemDamage(), entry.getValue());
        }
    }

    @Override
    public void init(FMLInitializationEvent event) {

    }

    @Override
    public void postInit(FMLPostInitializationEvent event) {

    }

    @Override
    public void addRenderRegister(ItemStack stack, ResourceLocation location, String variant) {
        MODEL_LOCATIONS_FOR_REGISTERING.put(stack, new ModelResourceLocation(location, variant));
    }
}

[TheGreyGhost]

Hi

 

  On 11/23/2016 at 1:36 AM, Lambda said:

So I'm having an issue with my itemblock not registering a texture.

The console does not output any missing resourcelocation so I'm guessing its somewhere in my code, but I had no luck in finding it.

when you say "not registering" I guess you mean that you get a purple-and-black chequered cube when you're holding the block, and when it's in the inventory?

Does your itemblock look correct when it's placed in the world?

 

My first comment is- I think your registration code is making your life difficult with the method calls and assembling the name strings from pieces - hard to spot things being missed or called out of order or names being wrong.

 

I'd suggest you put the registration code directly into preInit and Init and use literal strings for the names, etc, and once that is working, move it back out /change it back one piece at a time till you figure out the problem.

 

eg in your mod preInit:

  {
    // each instance of your block should have two names:
    // 1) a registry name that is used to uniquely identify this block.  Should be unique within your mod.  use lower case.
    // 2) an 'unlocalised name' that is used to retrieve the text name of your block in the player's language.  For example-
    //    the unlocalised name might be "water", which is printed on the user's screen as "Wasser" in German or
    //    "aqua" in Italian.
    //
    //    Multiple blocks can have the same unlocalised name - for example
    //  +----RegistryName----+---UnlocalisedName----+
    //  |  flowing_water     +       water          |
    //  |  stationary_water  +       water          |
    //  +--------------------+----------------------+
    //
    blockSimple = (BlockSimple)(new BlockSimple().setUnlocalizedName("mbe01_block_simple_unlocalised_name"));
    blockSimple.setRegistryName("mbe01_block_simple_registry_name");
    GameRegistry.register(blockSimple);

    // We also need to create and register an ItemBlock for this block otherwise it won't appear in the inventory
    itemBlockSimple = new ItemBlock(blockSimple);
    itemBlockSimple.setRegistryName(blockSimple.getRegistryName());
    GameRegistry.register(itemBlockSimple);
  }

and then in your client preinit, which must be called after common preinit,

    ModelResourceLocation itemModelResourceLocation = new ModelResourceLocation("minecraftbyexample:mbe01_block_simple", "inventory");
    final int DEFAULT_ITEM_SUBTYPE = 0;
    ModelLoader.setCustomModelResourceLocation(StartupCommon.itemBlockSimple, DEFAULT_ITEM_SUBTYPE, itemModelResourceLocation);

 

-TGG

[Lambda]

Okay, tried something a bit more basic, removed the hashmap( I dont really know why I did that in the first place  ), now:

- Block & Item(not ItemBlock) textures are registering

- ItemBlock still 'unregistered' (default texture)

 

Here is the updated code:

 

ItemUtil

package com.lambda.plentifulmisc.util;

/**
* Created by Blake on 11/22/2016.
*/

import com.lambda.plentifulmisc.PlentifulMisc;
import com.lambda.plentifulmisc.blocks.base.ItemBlockBase;
import net.minecraft.block.Block;
import net.minecraft.item.Item;
import net.minecraftforge.fml.common.registry.GameRegistry;


public final class ItemUtil{

    public static void registerBlock(Block block, ItemBlockBase itemBlock, String name, boolean addTab){
        block.setUnlocalizedName(ModUtil.MOD_ID+"."+name);

        block.setRegistryName(ModUtil.MOD_ID, name);
        GameRegistry.register(block);

        itemBlock.setRegistryName(block.getRegistryName());
        GameRegistry.register(itemBlock);

        block.setCreativeTab(addTab ? PlentifulMisc.instance.CREATIVE_TAB : null);

    }

    public static void registerItem(Item item, String name, boolean addTab){
        item.setUnlocalizedName(ModUtil.MOD_ID+"."+name);

        item.setRegistryName(ModUtil.MOD_ID, name);
        GameRegistry.register(item);

        item.setCreativeTab(addTab ? PlentifulMisc.instance.CREATIVE_TAB : null);

    }


}

 

blockBase

package com.lambda.plentifulmisc.blocks.base;

import com.lambda.plentifulmisc.PlentifulMisc;
import com.lambda.plentifulmisc.util.ItemUtil;
import net.minecraft.block.Block;
import net.minecraft.block.material.Material;
import net.minecraft.client.renderer.block.model.ModelResourceLocation;
import net.minecraft.item.ItemBlock;
import net.minecraft.item.ItemStack;
import net.minecraftforge.client.model.ModelLoader;


/**
* Created by Blake on 11/22/2016.
*/
public class BlockBase extends Block{

    private final String name;

    public BlockBase(Material material, String name){
        super(material);
        this.name = name;

        this.register();
    }

    private void register(){
        ItemUtil.registerBlock(this, this.getItemBlock(), this.getBaseName(), this.shouldAddCreative());

        this.registerRendering(this.getItemBlock(), 0);
    }

    protected String getBaseName(){
        return this.name;
    }

    protected ItemBlockBase getItemBlock(){
        return new ItemBlockBase(this);
    }

    public boolean shouldAddCreative(){
        return true;
    }

    protected void registerRendering(ItemBlock itemBlock, int metadata){
        PlentifulMisc.proxy.registerRendering(itemBlock, metadata, new ModelResourceLocation(this.getRegistryName(), "inventory"));
    }
}

 

Client Proxy:

 

package com.lambda.plentifulmisc.proxy;

import com.lambda.plentifulmisc.blocks.base.BlockBase;
import com.lambda.plentifulmisc.util.ModUtil;
import net.minecraft.client.renderer.block.model.ModelResourceLocation;
import net.minecraft.item.Item;
import net.minecraft.item.ItemBlock;
import net.minecraft.item.ItemStack;
import net.minecraft.util.ResourceLocation;
import net.minecraftforge.client.model.ModelLoader;
import net.minecraftforge.fml.common.event.FMLInitializationEvent;
import net.minecraftforge.fml.common.event.FMLPostInitializationEvent;
import net.minecraftforge.fml.common.event.FMLPreInitializationEvent;

import java.util.*;



/**
* Created by Blake on 11/22/2016.
*/
public class ClientProxy implements IProxy
{

    @Override
    public void preInit(FMLPreInitializationEvent event) {
        ModUtil.LOGGER.info("PreInitializing ClientProxy...");

    }

    @Override
    public void init(FMLInitializationEvent event) {

    }

    @Override
    public void postInit(FMLPostInitializationEvent event) {

    }

    @Override
    public void registerRendering(ItemBlock itemBlock, int metadata, ModelResourceLocation location){
        ModelLoader.setCustomModelResourceLocation(itemBlock, metadata, location);
    }

    @Override
    public void registerRenderingItem(Item item, int metadata, ModelResourceLocation location) {
        ModelLoader.setCustomModelResourceLocation(item, metadata, location);
    }
}

 

Block itself

package com.lambda.plentifulmisc.blocks;

import com.lambda.plentifulmisc.blocks.base.BlockBase;
import com.lambda.plentifulmisc.util.ItemUtil;
import net.minecraft.block.SoundType;
import net.minecraft.block.material.Material;

/**
* Created by Blake on 11/22/2016.
*/
public class BlockCeleistrum extends BlockBase{

    BlockCeleistrum(String name) {
        super(Material.ROCK, name);
        this.setHarvestLevel("pickaxe", 0);
        this.setHardness(2.5F);
        this.setResistance(10.0F);
        this.setSoundType(SoundType.STONE);
    }


}

[TheGreyGhost]

Hi

 

I suggest you go simpler still, take the registering out of the Block constructor, and just register it line by line in the Mod class methods, as per previous post

 

In particular this makes me nervous- it might be fine, but looking at it I'm not so sure:

 

public BlockBase(Material material, String name){
        super(material);
        this.name = name;

        this.register();
    }

    private void register(){
        ItemUtil.registerBlock(this, this.getItemBlock(), this.getBaseName(), this.shouldAddCreative());  // this isn't constructed yet

        this.registerRendering(this.getItemBlock(), 0);
    }

Calling methods and passing this, before the class is fully constructed, is always risky.

And it also appears you might be trying to retrieve an ItemBlock before you have registered the block itself, which also could be bad news.

 

I didn't understand what you mean here

"- Block & Item(not ItemBlock) textures are registering

- ItemBlock still 'unregistered' (default texture)

"

 

The Item is the ItemBlock, or not?  Normally you have a Block, an ItemBlock, and that's it.

 

-TGG

 

[Lambda]

Sorry miss-worded that, I was trying to say my items were registering now, as previously they were not