[1.8] Change block model based on damage

JimiIT92 · September 4, 2015

Is there any way to see how a block is damaged? I mean, you hit the block for a period of time and then it breaks, is there a way to see

"if the block has been hitten for X time than use this model, if it has been hitten for Z time than use this other model"

I've tried using the damageValue but it always return the metadata (wich in my case is 0). So basically how can i check if a player is breaking a block and from how long he's doing it?

TheGreyGhost · September 4, 2015

Hi

You can hook into one of the block damaging events, and when you render your block (should be a TileEntity) check to see if the block being damaged is that one.

See here for background info

http://greyminecraftcoder.blogspot.ch/2015/01/mining-blocks-with-tools.html

-TGG

JimiIT92 · September 4, 2015

Thanks, i'll check it out I found another method but i don't know if is correct. Overriding this function

@SideOnly(Side.CLIENT)
    public boolean addHitEffects(World world, MovingObjectPosition target, net.minecraft.client.particle.EffectRenderer effectRenderer)
    {
return false;
}

lets me handle what happens when breaking the block. Is it correct doing stuff here?

JimiIT92 · September 5, 2015

Ok, so i've added the event handler, so i now have this

package blaze.core;

import net.minecraft.block.Block;
import net.minecraftforge.event.entity.player.PlayerEvent;
import net.minecraftforge.event.entity.player.PlayerInteractEvent;
import net.minecraftforge.event.world.BlockEvent;
import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;

public class Events
{
@SubscribeEvent
public void breakSpeed(PlayerEvent.BreakSpeed event)
{
	if (   (event.state == null || event.state.getBlock() != BLBlocks.coal_block)
			&& (event.entityPlayer == null || event.entityPlayer.getCurrentEquippedItem() == null || event.entityPlayer.getCurrentEquippedItem().getItem() != BLItems.topaz_hammer)) {
		return;
	}
	else
	{
		System.out.println("BREAKING");
		Block block = event.state.getBlock();

	}
}

}

So now how can i tell the game "change the block model while breaking"?

JimiIT92 · September 5, 2015

any suggest? I want to make something similar to how the discs works on pixelmon (so i place a block down, i keep hitting it and sometimes it change it's model), i figured out how to change its drop if i'm using a certain tool or a condition is true, but how can i also "make it visual", how can i change it's model the more the breaking progress goes up (so the more i breaking it the more the model will be smaller)

JimiIT92 · September 6, 2015

Ok, so i've done what you said but now i give this error


java.lang.IllegalArgumentException: Don't know how to convert null[damage=0] back into data...
at net.minecraft.block.Block.getMetaFromState(Block.java:272)
at net.minecraft.block.Block.setHarvestLevel(Block.java:2201)
at net.minecraft.block.Block.setHarvestLevel(Block.java:2183)
at blaze.blocks.BlockCoalBlock.<init>(BlockCoalBlock.java:40)
at blaze.core.BLBlocks.addBlocks(BLBlocks.java:108)
at blaze.core.BL.preInit(BL.java:33)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at net.minecraftforge.fml.common.FMLModContainer.handleModStateEvent(FMLModContainer.java:536)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at com.google.common.eventbus.EventSubscriber.handleEvent(EventSubscriber.java:74)
at com.google.common.eventbus.SynchronizedEventSubscriber.handleEvent(SynchronizedEventSubscriber.java:47)
at com.google.common.eventbus.EventBus.dispatch(EventBus.java:322)
at com.google.common.eventbus.EventBus.dispatchQueuedEvents(EventBus.java:304)
at com.google.common.eventbus.EventBus.post(EventBus.java:275)
at net.minecraftforge.fml.common.LoadController.sendEventToModContainer(LoadController.java:208)
at net.minecraftforge.fml.common.LoadController.propogateStateMessage(LoadController.java:187)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at com.google.common.eventbus.EventSubscriber.handleEvent(EventSubscriber.java:74)
at com.google.common.eventbus.SynchronizedEventSubscriber.handleEvent(SynchronizedEventSubscriber.java:47)
at com.google.common.eventbus.EventBus.dispatch(EventBus.java:322)
at com.google.common.eventbus.EventBus.dispatchQueuedEvents(EventBus.java:304)
at com.google.common.eventbus.EventBus.post(EventBus.java:275)
at net.minecraftforge.fml.common.LoadController.distributeStateMessage(LoadController.java:118)
at net.minecraftforge.fml.common.Loader.preinitializeMods(Loader.java:514)
at net.minecraftforge.fml.client.FMLClientHandler.beginMinecraftLoading(FMLClientHandler.java:243)
at net.minecraft.client.Minecraft.startGame(Minecraft.java:446)
at net.minecraft.client.Minecraft.run(Minecraft.java:356)
at net.minecraft.client.main.Main.main(Main.java:117)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at net.minecraft.launchwrapper.Launch.launch(Launch.java:135)
at net.minecraft.launchwrapper.Launch.main(Launch.java:28)
at net.minecraftforge.gradle.GradleStartCommon.launch(Unknown Source)
at GradleStart.main(Unknown Source)

I see is related to the getMetaFromState method (wich you told me to not add it), so how can i fix that?

This is the updated block class

package blaze.blocks;

import java.util.Random;

import blaze.core.BLItems;
import blaze.items.ItemHammer;
import blaze.tileentities.TileEntityCoalBlock;
import net.minecraft.block.Block;
import net.minecraft.block.material.Material;
import net.minecraft.block.properties.IProperty;
import net.minecraft.block.properties.PropertyInteger;
import net.minecraft.block.state.BlockState;
import net.minecraft.block.state.IBlockState;
import net.minecraft.client.Minecraft;
import net.minecraft.client.renderer.RenderGlobal;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.projectile.EntityArrow;
import net.minecraft.init.Blocks;
import net.minecraft.item.Item;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.BlockPos;
import net.minecraft.util.EnumFacing;
import net.minecraft.world.IBlockAccess;
import net.minecraft.world.World;
import net.minecraftforge.fml.relauncher.ReflectionHelper;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

public class BlockCoalBlock extends Block{

public static Item drop;

public static final PropertyInteger DAMAGE = PropertyInteger.create("damage", 0, 10);

public BlockCoalBlock() {
	super(Material.rock);
	this.setCreativeTab(CreativeTabs.tabBlock);
	this.setDefaultState(this.blockState.getBaseState().withProperty(DAMAGE, Integer.valueOf(0)));
	this.setHarvestLevel("pickaxe", 0);
	this.setHardness(3.0F);
	this.setResistance(15.0F);
}

/**
     * Get the actual Block state of this Block at the given position. This applies properties not visible in the
     * metadata, such as fence connections.
     */
@Override
@SideOnly(Side.CLIENT)
    public IBlockState getActualState(IBlockState state, IBlockAccess worldIn, BlockPos pos)
    {
    	IBlockState newState = state;
    	
    	RenderGlobal render = Minecraft.getMinecraft().renderGlobal;
    	
    	int damage = ReflectionHelper.getPrivateValue(RenderGlobal.class, render, 21);
    	
    	System.out.println(damage);
    	
        return newState;
    }

@SideOnly(Side.CLIENT)
    public Item getItem(World worldIn, BlockPos pos)
    {
        return Item.getItemFromBlock(this);
    }

/**
     * Get the Item that this Block should drop when harvested.
     *  
     * @param fortune the level of the Fortune enchantment on the player's tool
     */
    public Item getItemDropped(IBlockState state, Random rand, int fortune)
    {
        return drop == null ? Item.getItemFromBlock(this) : drop;
    }
    
    public static Item setItemToDrop(EntityPlayer player, World world, BlockPos pos)
    {
    	Block down = world.getBlockState(pos.down()).getBlock();
    	Item item = player.getHeldItem().getItem();
    	
    	if(item instanceof ItemHammer && down.equals(Blocks.anvil))
    	{
    		drop = BLItems.carbon;
    	}
    	else
    	{
    		drop = null;
    	}
    	return null;
    }

   
    protected BlockState createBlockState()
    {
        return new BlockState(this, new IProperty[] {DAMAGE});
    }
    
}

Choonster · September 6, 2015

You need to override

getMetaFromState

, just ignore the

DAMAGE

property in it.

getStateFromMeta

will always return the default state if you don't override it, which is what you want in this case.

JimiIT92 · September 6, 2015

Infact i see that when i place any block it calls taht method (and it always return null), so how to change this? How to use Field to get that value?

Failender · September 6, 2015

If ur question was how to get the value from a Field, u use Field#getValue and cast that

JimiIT92 · September 6, 2015

I don't know how to use field instead of reflection to access specific private values. I know that using this

Field[] fields = RenderGlobal.class.getFields();
System.out.println(fields.length);

it gives me the fields of a class but the system out return 0 (so is not returning anything at all). How can i access the damagedBlocks value using field?

Failender · September 6, 2015

RenderGlobal.class.getDeclaredField(fieldname)

JimiIT92 · September 6, 2015

Ok, so i've done this

try {
    		Field damage = RenderGlobal.class.getDeclaredField("damagedBlocks");
    		damage.setAccessible(true);
		System.out.println(damage.get(0));
	} catch (IllegalArgumentException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (IllegalAccessException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (NoSuchFieldException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (SecurityException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	}

But now every tick i get this

[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: java.lang.IllegalArgumentException: Can not set final java.util.Map field net.minecraft.client.renderer.RenderGlobal.damagedBlocks to java.lang.Integer
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at sun.reflect.UnsafeFieldAccessorImpl.throwSetIllegalArgumentException(Unknown Source)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at sun.reflect.UnsafeFieldAccessorImpl.throwSetIllegalArgumentException(Unknown Source)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at sun.reflect.UnsafeFieldAccessorImpl.ensureObj(Unknown Source)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at sun.reflect.UnsafeQualifiedObjectFieldAccessorImpl.get(Unknown Source)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at java.lang.reflect.Field.get(Unknown Source)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at blaze.blocks.BlockCoalBlock.getActualState(BlockCoalBlock.java:65)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at net.minecraft.block.Block.isSideSolid(Block.java:1295)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at net.minecraft.block.Block.canCreatureSpawn(Block.java:1585)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at net.minecraft.world.SpawnerAnimals.canCreatureTypeSpawnAtLocation(SpawnerAnimals.java:237)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at net.minecraft.world.SpawnerAnimals.findChunksForSpawning(SpawnerAnimals.java:152)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at net.minecraft.world.WorldServer.tick(WorldServer.java:206)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:763)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:669)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:171)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:540)
[15:25:20] [server thread/INFO] [sTDERR]: [java.lang.Throwable$WrappedPrintStream:println:-1]: 	at java.lang.Thread.run(Unknown Source)

this is called from the system out, where i do "damage.get(0)"

Failender · September 6, 2015

ur error is printed there. google it and u will find results.

why the hell are u calling get with a 0 as arg..

JimiIT92 · September 6, 2015

D'oh, i've changed the system out to this

RenderGlobal render = Minecraft.getMinecraft().renderGlobal;

    	try {
    		Field damage = RenderGlobal.class.getDeclaredField("damagedBlocks");
    		damage.setAccessible(true);
		System.out.println(damage.get(render).toString());
	} catch (IllegalArgumentException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (IllegalAccessException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (NoSuchFieldException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (SecurityException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	}

wich is now correct, but the system out return this

[15:41:17] [Client thread/INFO] [sTDOUT]: [blaze.blocks.BlockCoalBlock:getActualState:65]: {}

and when destroying it gives me this

[15:41:31] [Client thread/INFO] [sTDOUT]: [blaze.blocks.BlockCoalBlock:getActualState:65]: {157=net.minecraft.client.renderer.DestroyBlockProgress@58da1b27}

So how to get the exact value of the damaging progress?

JimiIT92 · September 6, 2015

Looping though the values give me this

157=net.minecraft.client.renderer.DestroyBlockProgress@58da1b27

from the damage.get(render)

How to turn this into an integer (from 1 to 10)? Or how from damage variable obtain the position?

JimiIT92 · September 6, 2015

Sorry, my bad! Ok, so i've now this

/**
     * Get the actual Block state of this Block at the given position. This applies properties not visible in the
     * metadata, such as fence connections.
     */
@Override
@SideOnly(Side.CLIENT)
    public IBlockState getActualState(IBlockState state, IBlockAccess worldIn, BlockPos pos)
    {
    	IBlockState newState = state;
    	
    	RenderGlobal render = Minecraft.getMinecraft().renderGlobal;

    	DestroyBlockProgress progress = null;
    	
    	try {
    		Field damage = RenderGlobal.class.getDeclaredField("damagedBlocks");
    		damage.setAccessible(true);
    		HashMap mp = (HashMap)damage.get(render);
    		Iterator it = mp.entrySet().iterator();
    	    while (it.hasNext()) {
    	        Map.Entry pair = (Map.Entry)it.next();
    	        System.out.println(pair.getKey() + " = " + pair.getValue());
    	        progress = (DestroyBlockProgress) pair.getValue();
    	        it.remove(); // avoids a ConcurrentModificationException
    	        System.out.println(progress.getPartialBlockDamage());
    	    }


	} catch (IllegalArgumentException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (IllegalAccessException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (NoSuchFieldException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (SecurityException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	}
    	
        return newState;
    }

and i can keep track of how the block is damaged. Now, i just have to set a new block state with the progress value right? For example this?

return this.getStateFromMeta(2);

JimiIT92 · September 6, 2015

Ok, so now it looks like this

/**
 * Get the actual Block state of this Block at the given position. This applies properties not visible in the
 * metadata, such as fence connections.
 */
@Override
@SideOnly(Side.CLIENT)
public IBlockState getActualState(IBlockState state, IBlockAccess worldIn, BlockPos pos)
{
	IBlockState newState = state;

	RenderGlobal render = Minecraft.getMinecraft().renderGlobal;

	DestroyBlockProgress progress = null;

	try {
		Field damage = RenderGlobal.class.getDeclaredField("damagedBlocks");
		damage.setAccessible(true);
		Map mp = (Map)damage.get(render);
		Iterator it = mp.entrySet().iterator();
		while (it.hasNext()) {
			Map.Entry pair = (Map.Entry)it.next();
			progress = (DestroyBlockProgress) pair.getValue();
			newState = this.blockState.getBaseState().withProperty(DAMAGE, Integer.valueOf(progress.getPartialBlockDamage()));
		}

	} catch (IllegalArgumentException e) {
		e.printStackTrace();
	} catch (IllegalAccessException e) {
		e.printStackTrace();
	} catch (NoSuchFieldException e) {
		e.printStackTrace();
	} catch (SecurityException e) {
		e.printStackTrace();
	}

	return newState;
}

And when placing the block down by default it gives me this

This is the blockstate file for the block

{
    "variants": {
        "damage=0": { "model": "blaze:coal_block" },
	"damage=1": { "model": "blaze:coal_block" },
	"damage=2": { "model": "blaze:coal_block" },
	"damage=3": { "model": "blaze:coal_block" },
	"damage=4": { "model": "blaze:coal_block" },
	"damage=5": { "model": "blaze:coal_block_5" },
	"damage=6": { "model": "blaze:coal_block_5" },
	"damage=7": { "model": "blaze:coal_block_5" },
	"damage=8": { "model": "blaze:coal_block_5" },
	"damage=9": { "model": "blaze:coal_block_5" },
	"damage=10": { "model": "blaze:coal_block_5" }
    }
}

Also how to prevent from looking it every time?

Failender · September 6, 2015

- You are still looking up the Field every time. You only need to do that once.

JimiIT92 · September 6, 2015

/**
 * Get the actual Block state of this Block at the given position. This applies properties not visible in the
 * metadata, such as fence connections.
 */
@Override
@SideOnly(Side.CLIENT)
public IBlockState getActualState(IBlockState state, IBlockAccess worldIn, BlockPos pos)
{
	IBlockState newState = state;

	newState = this.blockState.getBaseState().withProperty(DAMAGE, Integer.valueOf(this.damage));

	return newState;
}

static {
	RenderGlobal render = Minecraft.getMinecraft().renderGlobal;

	DestroyBlockProgress progress = null;
	Field damage;
	try {
		damage = RenderGlobal.class.getDeclaredField("damagedBlocks");
		damage.setAccessible(true);
		if(damage != null)
		{
			Map mp = (Map)damage.get(render);
			Iterator it = mp.entrySet().iterator();
			while (it.hasNext()) {
				Map.Entry pair = (Map.Entry)it.next();
				progress = (DestroyBlockProgress) pair.getValue();

				BlockCoalBlock.damage = progress.getPartialBlockDamage();

			}
		}

	} catch (NoSuchFieldException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (SecurityException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (IllegalArgumentException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (IllegalAccessException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	}

}

it gives me this

java.lang.ExceptionInInitializerError
at blaze.core.BLBlocks.addBlocks(BLBlocks.java:108) ~[bin/:?]
at blaze.core.BL.preInit(BL.java:33) ~[bin/:?]
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_51]
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_51]
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_51]
at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.8.0_51]
at net.minecraftforge.fml.common.FMLModContainer.handleModStateEvent(FMLModContainer.java:536) ~[forgeSrc-1.8-11.14.1.1375.jar:?]
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_51]
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_51]
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_51]
at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.8.0_51]
at com.google.common.eventbus.EventSubscriber.handleEvent(EventSubscriber.java:74) ~[guava-17.0.jar:?]
at com.google.common.eventbus.SynchronizedEventSubscriber.handleEvent(SynchronizedEventSubscriber.java:47) ~[guava-17.0.jar:?]
at com.google.common.eventbus.EventBus.dispatch(EventBus.java:322) ~[guava-17.0.jar:?]
at com.google.common.eventbus.EventBus.dispatchQueuedEvents(EventBus.java:304) ~[guava-17.0.jar:?]
at com.google.common.eventbus.EventBus.post(EventBus.java:275) ~[guava-17.0.jar:?]
at net.minecraftforge.fml.common.LoadController.sendEventToModContainer(LoadController.java:208) ~[forgeSrc-1.8-11.14.1.1375.jar:?]
at net.minecraftforge.fml.common.LoadController.propogateStateMessage(LoadController.java:187) ~[forgeSrc-1.8-11.14.1.1375.jar:?]
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_51]
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_51]
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_51]
at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.8.0_51]
at com.google.common.eventbus.EventSubscriber.handleEvent(EventSubscriber.java:74) ~[guava-17.0.jar:?]
at com.google.common.eventbus.SynchronizedEventSubscriber.handleEvent(SynchronizedEventSubscriber.java:47) ~[guava-17.0.jar:?]
at com.google.common.eventbus.EventBus.dispatch(EventBus.java:322) ~[guava-17.0.jar:?]
at com.google.common.eventbus.EventBus.dispatchQueuedEvents(EventBus.java:304) ~[guava-17.0.jar:?]
at com.google.common.eventbus.EventBus.post(EventBus.java:275) ~[guava-17.0.jar:?]
at net.minecraftforge.fml.common.LoadController.distributeStateMessage(LoadController.java:118) [LoadController.class:?]
at net.minecraftforge.fml.common.Loader.preinitializeMods(Loader.java:514) [Loader.class:?]
at net.minecraftforge.fml.client.FMLClientHandler.beginMinecraftLoading(FMLClientHandler.java:243) [FMLClientHandler.class:?]
at net.minecraft.client.Minecraft.startGame(Minecraft.java:446) [Minecraft.class:?]
at net.minecraft.client.Minecraft.run(Minecraft.java:356) [Minecraft.class:?]
at net.minecraft.client.main.Main.main(Main.java:117) [Main.class:?]
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_51]
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_51]
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_51]
at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.8.0_51]
at net.minecraft.launchwrapper.Launch.launch(Launch.java:135) [launchwrapper-1.11.jar:?]
at net.minecraft.launchwrapper.Launch.main(Launch.java:28) [launchwrapper-1.11.jar:?]
at net.minecraftforge.gradle.GradleStartCommon.launch(Unknown Source) [start/:?]
at GradleStart.main(Unknown Source) [start/:?]
Caused by: java.lang.NullPointerException
at sun.reflect.UnsafeFieldAccessorImpl.ensureObj(Unknown Source) ~[?:1.8.0_51]
at sun.reflect.UnsafeQualifiedObjectFieldAccessorImpl.get(Unknown Source) ~[?:1.8.0_51]
at java.lang.reflect.Field.get(Unknown Source) ~[?:1.8.0_51]
at blaze.blocks.BlockCoalBlock.<clinit>(BlockCoalBlock.java:86) ~[blockCoalBlock.class:?]
... 41 more

Also how to check the position? Since it's a static block i can't access the getPosition method

Failender · September 6, 2015

only lookup the field in the static block. everything else stays where it is

JimiIT92 · September 6, 2015

Alright, code changed to this

public static Field damage;
public static int d_progress;

...

/**
 * Get the actual Block state of this Block at the given position. This applies properties not visible in the
 * metadata, such as fence connections.
 */
@Override
@SideOnly(Side.CLIENT)
public IBlockState getActualState(IBlockState state, IBlockAccess worldIn, BlockPos pos)
{
	IBlockState newState = state;

	RenderGlobal render = Minecraft.getMinecraft().renderGlobal;

	DestroyBlockProgress progress = null;

	if(damage != null)
	{
		Map mp;
		try {
			mp = (Map)damage.get(render);
			Iterator it = mp.entrySet().iterator();
			while (it.hasNext()) {
				Map.Entry pair = (Map.Entry)it.next();
				progress = (DestroyBlockProgress) pair.getValue();

				if(progress.getPosition().equals(pos))
				{
					d_progress = progress.getPartialBlockDamage();
					newState = this.blockState.getBaseState().withProperty(DAMAGE, Integer.valueOf(this.d_progress));
				}
			}
		} catch (IllegalArgumentException e) {
			e.printStackTrace();
		} catch (IllegalAccessException e) {
			e.printStackTrace();
		}
	}
	return newState;
}

static {

	try {
		damage = RenderGlobal.class.getDeclaredField("damagedBlocks");
		damage.setAccessible(true);
	} catch (NoSuchFieldException e) {
		e.printStackTrace();
	} catch (SecurityException e) {
		e.printStackTrace();
	} catch (IllegalArgumentException e) {
		e.printStackTrace();
	}
}

But still get the incorrect blockstate when placing it down

JimiIT92 · September 6, 2015

forgot to change that, it was static beacause before was used in the static block.

So i need to return this right?

state.withProperty(DAMAGE, Integer.valueOf(this.d_progress));

EDIT: changed to this

@Override
@SideOnly(Side.CLIENT)
public IBlockState getActualState(IBlockState state, IBlockAccess worldIn, BlockPos pos)
{
	IBlockState newState = state;

	RenderGlobal render = Minecraft.getMinecraft().renderGlobal;

	DestroyBlockProgress progress = null;

	if(damage != null)
	{
		Map mp;
		try {
			mp = (Map)damage.get(render);
			Iterator it = mp.entrySet().iterator();
			while (it.hasNext()) {
				Map.Entry pair = (Map.Entry)it.next();
				progress = (DestroyBlockProgress) pair.getValue();

				if(progress.getPosition().equals(pos))
				{
					newState = state.withProperty(DAMAGE, Integer.valueOf(progress.getPartialBlockDamage()));
				}
			}
		} catch (IllegalArgumentException e) {
			e.printStackTrace();
		} catch (IllegalAccessException e) {
			e.printStackTrace();
		}
	}
	return newState;
}

still returning the incorrect block model

JimiIT92 · September 6, 2015

Removed, updated previous code

JimiIT92 · September 6, 2015

It goes from 0 to 8 while breaking the block, reaching the 8 when is nearly broken. I've printed this

progress.getPartialBlockDamage()

JimiIT92 · September 6, 2015

If i return this

newState = state.withProperty(DAMAGE, Integer.valueOf(0));

it's still returning me the model that should be corresponding the the damage 5 or higher, so i think could be this the problem.

This is how i register the block

block.setUnlocalizedName(name);
	GameRegistry.registerBlock(block, ItemCoalBlock.class, name);
	ModelBakery.addVariantName(Item.getItemFromBlock(coal_block), 
			BL.MODID + ":" + "coal_block", 
			BL.MODID + ":" + "coal_block",
			BL.MODID + ":" + "coal_block",
			BL.MODID + ":" + "coal_block",
			BL.MODID + ":" + "coal_block",
			BL.MODID + ":" + "coal_block",
			BL.MODID + ":" + "coal_block",
			BL.MODID + ":" + "coal_block",
			BL.MODID + ":" + "coal_block",
			BL.MODID + ":" + "coal_block",
			BL.MODID + ":" + "coal_block");
	for(int i = 0; i < 11; i++)
		renderItem.getItemModelMesher().register(Item.getItemFromBlock(block), i, new ModelResourceLocation(BL.MODID + ":" + name, "inventory"));

block is the coal_block instance

name is the unlocalized name

renderItem is the RenderItem instance

and this is the blockstate file

{
    "variants": {
        "damage=0": { "model": "blaze:coal_block" },
	"damage=1": { "model": "blaze:coal_block" },
	"damage=2": { "model": "blaze:coal_block" },
	"damage=3": { "model": "blaze:coal_block" },
	"damage=4": { "model": "blaze:coal_block" },
	"damage=5": { "model": "blaze:coal_block_5" },
	"damage=6": { "model": "blaze:coal_block_5" },
	"damage=7": { "model": "blaze:coal_block_5" },
	"damage=8": { "model": "blaze:coal_block_5" },
	"damage=9": { "model": "blaze:coal_block_5" },
	"damage=10": { "model": "blaze:coal_block_5" }
    }
}

And this is the block model file for damage 0-4

{
    "parent": "block/cube_all",
    "textures": {
        "all": "blocks/coal_block"
    }
}

and for damage 5-10

{
    "textures": {
        "all": "blocks/coal_block",
	"particle": "blocks/coal_block"
    },
"elements": [
        {   "from": [ 0, 0, 0 ],
            "to": [ 16, 5, 16 ],
            "faces": {
                "down":  { "texture": "#all", "cullface": "down" },
                "up":    { "texture": "#all", "cullface": "up" },
                "north": { "texture": "#all", "cullface": "north" },
                "south": { "texture": "#all", "cullface": "south" },
                "west":  { "texture": "#all", "cullface": "west" },
                "east":  { "texture": "#all", "cullface": "east" }
            }
        }
    ]
}

Sign In

[1.8] Change block model based on damage

Recommended Posts

JimiIT92

TheGreyGhost

JimiIT92

JimiIT92

JimiIT92

JimiIT92

Choonster

JimiIT92

Failender

JimiIT92

Failender

JimiIT92

Failender

JimiIT92

JimiIT92

JimiIT92

JimiIT92

Failender

JimiIT92

Failender

JimiIT92

JimiIT92

JimiIT92

JimiIT92

JimiIT92

Join the conversation

Announcements

Recently Browsing

Posts

Topics

Who's Online (See full list)

Browse

Activity

Important Information