Jump to content
Search the Forums BEFORE making a new post.
Currently supported: All versions

[1.12.2]Harvester crashes world when setting a crops age to 0

tebreca

By tebreca
in Modder Support

 Share

Recommended Posts

tebreca Creeper Killer

tebreca

Posted
Posted

So i got this error 

net.minecraft.util.ReportedException: Ticking block entity
	at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:846) ~[MinecraftServer.class:?]
	at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:741) ~[MinecraftServer.class:?]
	at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:192) ~[IntegratedServer.class:?]
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:590) [MinecraftServer.class:?]
	at java.lang.Thread.run(Unknown Source) [?:1.8.0_131]
Caused by: java.lang.IllegalArgumentException: Cannot set property PropertyInteger{name=age, clazz=class java.lang.Integer, values=[0, 1, 2, 3, 4, 5, 6, 7]} as it does not exist in BlockStateContainer{block=minecraft:beetroots, properties=[age]}
	at net.minecraft.block.state.BlockStateContainer$StateImplementation.withProperty(BlockStateContainer.java:221) ~[BlockStateContainer$StateImplementation.class:?]
	at btf.objects.blocks.tiles.TileHarvesterTicker.update(TileHarvesterTicker.java:61) ~[TileHarvesterTicker.class:?]
	at net.minecraft.world.World.updateEntities(World.java:1988) ~[World.class:?]
	at net.minecraft.world.WorldServer.updateEntities(WorldServer.java:642) ~[WorldServer.class:?]
	at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:840) ~[MinecraftServer.class:?]
	... 4 more

 

this is the TE''s code

  

package btf.objects.blocks.tiles;

import btf.main.Main;
import btf.objects.blocks.Machine;
import jline.console.completer.ArgumentCompleter.WhitespaceArgumentDelimiter;
import net.minecraft.block.BlockChest;
import net.minecraft.block.BlockCrops;
import net.minecraft.block.state.BlockStateContainer;
import net.minecraft.block.state.IBlockState;
import net.minecraft.entity.item.EntityItem;
import net.minecraft.init.Blocks;
import net.minecraft.init.Items;
import net.minecraft.item.ItemStack;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.tileentity.TileEntityChest;
import net.minecraft.util.ITickable;
import net.minecraft.util.math.BlockPos;

public class TileHarvesterTicker extends TileEntity implements ITickable{
	BlockPos checkstart = null;
	IBlockState statebehind = null;
	BlockPos behind = null;
	boolean initialized = false;
	private int x = 0;
	private int y = 0;
	private int z = 0;
	@Override
	public void update() {
		if(!initialized) {
			init();
		}
		if (x == 3) {
			x=0;z++;
		}else if (z == 4) {
			z=0;x=0;
		}
			
		IBlockState state = world.getBlockState(new BlockPos(checkstart.getX() + x, checkstart.getY(), checkstart.getZ()+z));
		if (state.getBlock() == Blocks.WHEAT) {
			if (state == Blocks.WHEAT.getBlockState().getBaseState().withProperty(net.minecraft.block.BlockCrops.AGE, 7)) {
			world.setBlockState(new BlockPos(checkstart.getX() + x, checkstart.getY(), checkstart.getZ()+z), state.withProperty(BlockCrops.AGE, 0));
			EntityItem item = new EntityItem(world, behind.getX(), behind.getY(), behind.getZ(), new ItemStack(Items.WHEAT));
			if (!world.isRemote) {
			world.spawnEntity(item);
			}}
		} else if (state.getBlock() == Blocks.CARROTS) { 
			if (state == Blocks.CARROTS.getBlockState().getBaseState().withProperty(net.minecraft.block.BlockCrops.AGE, 7)) {
			world.setBlockState(new BlockPos(checkstart.getX() + x, checkstart.getY(), checkstart.getZ()+z), state.withProperty(BlockCrops.AGE, 0));
			EntityItem item = new EntityItem(world, behind.getX(), behind.getY(), behind.getZ(), new ItemStack(Items.CARROT, (int) (Math.random() * Math.random() * 2 +1)));
			if (!world.isRemote) {
			world.spawnEntity(item);
			}}
		}  else if (state.getBlock() == Blocks.POTATOES) {
			if (state == Blocks.POTATOES.getBlockState().getBaseState().withProperty(net.minecraft.block.BlockCrops.AGE, 7)) {
			world.setBlockState(new BlockPos(checkstart.getX() + x, checkstart.getY(), checkstart.getZ()+z), state.withProperty(BlockCrops.AGE, 0));
			EntityItem item = new EntityItem(world, behind.getX(), behind.getY(), behind.getZ(), new ItemStack(Items.CARROT, (int) (Math.random() * Math.random() * 2 +1)));
			if (!world.isRemote) {
			world.spawnEntity(item);
			}}
		}	 else if (state.getBlock() == Blocks.BEETROOTS) {
			if (state == Blocks.BEETROOTS.getBlockState().getBaseState().withProperty(net.minecraft.block.BlockCrops.AGE, 3)) {
			world.setBlockState(new BlockPos(checkstart.getX() + x, checkstart.getY(), checkstart.getZ()+z), state.withProperty(BlockCrops.AGE, 0));
			EntityItem item = new EntityItem(world, behind.getX(), behind.getY(), behind.getZ(), new ItemStack(Items.CARROT, (int) (Math.random() * Math.random() * 2 +1)));
			if (!world.isRemote) {
			world.spawnEntity(item);
			}}
		}
		x++;
}
	
	private void init() {
		initialized = true;
			switch (world.getBlockState(pos).getValue(Machine.FACING)) {
			case DOWN:
			{
				break;
			}
			case EAST:
			{
				checkstart = new BlockPos(pos.getX()+1, pos.getY(), pos.getZ()-1);
				break;
			}
			case NORTH:
			{
				checkstart = new BlockPos(pos.getX()-1, pos.getY(), pos.getZ()-3);
				break;
			}
			case SOUTH:
			{
				checkstart = new BlockPos(pos.getX()-1, pos.getY(), pos.getZ()+1);
				break;
			}
			case UP:
			{
				
			}
			case WEST:
			{
				checkstart = new BlockPos(pos.getX()-3, pos.getY(), pos.getZ()-1);
				break;
			}		
			}
			switch (world.getBlockState(pos).getValue(Machine.FACING)) {
			case DOWN:
			{
				break;
			}
			case EAST:
			{
				statebehind = world.getBlockState(new BlockPos(pos.getX()-1, pos.getY(), pos.getZ()));
				 behind = new BlockPos(pos.getX()-1, pos.getY(), pos.getZ());
				break;
			}
			case NORTH:
			{
				statebehind = world.getBlockState(new BlockPos(pos.getX(), pos.getY(), pos.getZ()+1));
				 behind = new BlockPos(pos.getX(), pos.getY(), pos.getZ()+1);
				break;
			}
			case SOUTH:
			{
				statebehind = world.getBlockState(new BlockPos(pos.getX(), pos.getY(), pos.getZ()-1));
				 behind = new BlockPos(pos.getX(), pos.getY(), pos.getZ()-1);
				break;
			}
			case UP:
			{
				
			}
			case WEST:
			{
				statebehind = world.getBlockState(new BlockPos(pos.getX()+1, pos.getY(), pos.getZ()));
				behind = new BlockPos(pos.getX()+1, pos.getY(), pos.getZ());
				break;
			}		
			}
	}

}

 

Draco18s Reality Controller

Draco18s

Posted
Posted

Beetroots don't use the same AGE property as other crops:

  

public static final PropertyInteger BEETROOT_AGE = PropertyInteger.create("age", 0, 3);


   

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.

Matryoshika Dragon Slayer

Matryoshika

Posted
Posted

You don't need to mess with individual if-statements.
All of these crops extend BlockCrops
 

Check if the block extends BlockCrops, then cast it. From there, call BlockCrops::isMaxAge(IBlockState), if so, call BlockCrops::withAge(0) to get a new IBlockState of the crop, with age of 0.

  • Like 1

Also previously known as eAndPi.

"Pi, is there a station coming up where we can board your train of thought?" -Kronnn

Published Mods: Underworld

Handy links: Vic_'s Forge events Own WIP Tutorials.

tebreca Creeper Killer

tebreca

Posted
  • Author
Posted

 

10 hours ago, Matryoshika said:

You don't need to mess with individual if-statements.
All of these crops extend BlockCrops
 

Check if the block extends BlockCrops, then cast it. From there, call BlockCrops::isMaxAge(IBlockState), if so, call BlockCrops::withAge(0) to get a new IBlockState of the crop, with age of 0.

 ok thx

12 hours ago, Draco18s said:

Beetroots don't use the same AGE property as other crops:

  


public static final PropertyInteger BEETROOT_AGE = PropertyInteger.create("age", 0, 3);


   

ok, im sorry for not looking up the files before posting here

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.

×
 Share
Go to topic listing

Announcements



×
×
  • Create New...

Important Information

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

  I accept