IItemColor Not working correctly

[Lambda]

Hello,

 

So I have some dusts, all of vanilla ores, and I'm using IItemColor to choose colors over the 1 Icon, here is the code:

[The Colors are not displaying.]

 

ItemDust:

package com.lambda.plentifulmisc.items;

/**
* Created by Blake on 11/26/2016.
*/
import com.lambda.plentifulmisc.PlentifulMisc;
import com.lambda.plentifulmisc.items.base.ItemBase;
import com.lambda.plentifulmisc.items.globalization.Dusts;
import com.lambda.plentifulmisc.util.StringUtil;
import net.minecraft.client.renderer.color.IItemColor;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.item.EnumRarity;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.util.NonNullList;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

public class ItemDust extends ItemBase implements IColorItem{

    public static final Dusts[] ALL_DUSTS = Dusts.values();

    public ItemDust(String name){
        super(name);
        this.setHasSubtypes(true);
    }

    @Override
    public int getMetadata(int damage){
        return damage;
    }


    @Override
    public String getUnlocalizedName(ItemStack stack){
        return stack.getItemDamage() >= ALL_DUSTS.length ? StringUtil.BUGGED_ITEM_NAME : this.getUnlocalizedName()+"_"+ALL_DUSTS[stack.getItemDamage()].name;
    }


    @Override
    public EnumRarity getRarity(ItemStack stack){
        return stack.getItemDamage() >= ALL_DUSTS.length ? EnumRarity.COMMON : ALL_DUSTS[stack.getItemDamage()].rarity;
    }

    @Override
    @SideOnly(Side.CLIENT)
    public void getSubItems(Item item, CreativeTabs tab, NonNullList list){
        for(int j = 0; j < ALL_DUSTS.length; j++){
            list.add(new ItemStack(this, 1, j));
        }
    }

    @Override
    protected void registerRendering(){
        for(int i = 0; i < ALL_DUSTS.length; i++){
            PlentifulMisc.proxy.addRenderRegister(new ItemStack(this, 1, i), this.getRegistryName(), "inventory");
        }
    }

    @SideOnly(Side.CLIENT)
    @Override
    public IItemColor getColor(){
        return new IItemColor(){
            @Override
            public int getColorFromItemstack(ItemStack stack, int pass){
                return stack.getItemDamage() >= ALL_DUSTS.length ? 0xFFFFFF : ALL_DUSTS[stack.getItemDamage()].color;
            }
        };
    }
}

 

Dusts:

package com.lambda.plentifulmisc.items.globalization;

import com.lambda.plentifulmisc.util.StringUtil;
import net.minecraft.item.EnumRarity;

/**
* Created by Blake on 11/26/2016.
*/
public enum Dusts{

    IRON("iron", 7826534, EnumRarity.COMMON),
    GOLD("gold", 14335744, EnumRarity.UNCOMMON),
    DIAMOND("diamond", 292003, EnumRarity.RARE),
    EMERALD("emerald", 4319527, EnumRarity.EPIC),
    LAPIS("lapis", 1849791, EnumRarity.UNCOMMON),
    QUARTZ("quartz", StringUtil.DECIMAL_COLOR_WHITE, EnumRarity.UNCOMMON),
    COAL("coal", 0, EnumRarity.UNCOMMON),
    CELESTRIUM("celestrium", 1251327, EnumRarity.RARE);

    public final String name;
    public final int color;
    public final EnumRarity rarity;

    Dusts(String name, int color, EnumRarity rarity){
        this.name = name;
        this.color = color;
        this.rarity = rarity;
    }


}

 

IColorItem:

package com.lambda.plentifulmisc.items;

import net.minecraft.client.renderer.color.IItemColor;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

/**
* Created by Blake on 11/26/2016.
*/
public interface IColorItem{

    @SideOnly(Side.CLIENT)
    IItemColor getColor();

}

 

Note: The icon is WHITE.

 

Lastly if you would like to help more: How could I search for existing 'mod ores' and create dusts for them?

 

Thanks for your time

[Draco18s]

Are you registering your item as a colored item?

 

Just because you declared and implemented that interface doesn't mean diddly: nothing calls

getColor()

.

[Lambda]

Are you registering your item as a colored item?

 

Just because you declared and implemented that interface doesn't mean diddly: nothing calls

getColor()

.

Okay that makes sense.. How would I register it as a colored item, however

[Draco18s]

Look for existing usages in vanilla.

[Lambda]

Looking into vanilla classes, I believe I'm already doing so.

[Choonster]

Post the code where you register your

IItemColor

.

 

Is your

IItemColor#getColorFromItemstack

method ever called?

[TheGreyGhost]

Hi

 

You might find this working example useful

 

https://github.com/TheGreyGhost/MinecraftByExample/tree/master/src/main/java/minecraftbyexample/mbe11_item_variants

 

-TGG