[1.8] Change block model based on damage

Posted September 4, 201510 yr

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?

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 4, 201510 yr

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

Quote

September 4, 201510 yr

Author

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?

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 5, 201510 yr

Author

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"?

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 5, 201510 yr

Author

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)

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

Author

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});
    }
    
}

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

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.

Please don't PM me to ask for help. Asking your question in a public thread preserves it for people who are having the same problem in the future.

Quote

September 6, 201510 yr

Author

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?

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

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

Quote

September 6, 201510 yr

Author

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?

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

RenderGlobal.class.getDeclaredField(fieldname)

Quote

September 6, 201510 yr

Author

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

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

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

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

Quote

September 6, 201510 yr

Author

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?

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

Author

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?

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

Author

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

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

Author

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?

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

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

Quote

September 6, 201510 yr

Author

/**
 * 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

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

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

Quote

September 6, 201510 yr

Author

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

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

Author

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

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

Author

Removed, updated previous code

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

Author

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

progress.getPartialBlockDamage()

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

September 6, 201510 yr

Author

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" }
            }
        }
    ]
}

Don't blame me if i always ask for your help. I just want to learn to be better

Quote

Sign In

[1.8] Change block model based on damage

Featured Replies

Join the conversation

Posts

Forge 59.0 Minecraft 1.21.9

Forge 58.1 Minecraft 1.21.8

Forge 55.1 Minecraft 1.21.5

Bad pathfinding for big mobs (1.21.1)

(((SOLVED))) Loading into certain worlds stuck on "Reading world data" then stops responding.

Who's Online (See full list)

Important Information

Account

Navigation

Search

Configure browser push notifications

Chrome (Android)

Chrome (Desktop)

Safari (iOS 16.4+)

Safari (macOS)

Edge (Android)

Edge (Desktop)

Firefox (Android)

Firefox (Desktop)