[SOLVED][1.8] Crash when placing underwater plant

[JimiIT92]

Hi guys, i'm trying to updating my underwater plants from 1.7.10 to 1.8. Unfortunately when i place it underwater the game crash and i don't know why

So here is the code for my underwater plant

package mw.blocks;

import java.util.List;
import java.util.Random;

import mw.core.utils.Utils;
import net.minecraft.block.Block;
import net.minecraft.block.BlockBush;
import net.minecraft.block.material.Material;
import net.minecraft.block.properties.IProperty;
import net.minecraft.block.properties.PropertyEnum;
import net.minecraft.block.state.BlockState;
import net.minecraft.block.state.IBlockState;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.init.Blocks;
import net.minecraft.init.Items;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.stats.StatList;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.AxisAlignedBB;
import net.minecraft.util.BlockPos;
import net.minecraft.util.EnumWorldBlockLayer;
import net.minecraft.util.IStringSerializable;
import net.minecraft.world.IBlockAccess;
import net.minecraft.world.World;
import net.minecraftforge.common.EnumPlantType;
import net.minecraftforge.common.IPlantable;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

public class BlockUnderwaterPlant extends Block implements IPlantable
{

public static final PropertyEnum VARIANT = PropertyEnum.create("variant", BlockUnderwaterPlant.EnumType.class);

    public BlockUnderwaterPlant()
    {
        super(Material.water);
        Utils.setPlantBlockInfo(this, "underwater_plant");
    }

    /**
     * is the block grass, dirt or farmland
     */
    protected boolean canPlaceBlockOn(Block ground)
    {
        return ground == Blocks.sand || ground == Blocks.gravel;
    }

    /**
     * Whether this Block can be replaced directly by other blocks (true for e.g. tall grass)
     */
    public boolean isReplaceable(World worldIn, BlockPos pos)
    {
        return true;
    }

    /**
     * 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 null;
    }

    public void harvestBlock(World worldIn, EntityPlayer player, BlockPos pos, IBlockState state, TileEntity te)
    {
    	super.harvestBlock(worldIn, player, pos, state, te);
    }
    /**
     * Get the damage value that this Block should drop
     */
    public int damageDropped(IBlockState state)
    {
        return ((BlockUnderwaterPlant.EnumType)state.getValue(VARIANT)).getMetadata();
    }

    /**
     * returns a list of blocks with the same ID, but different meta (eg: wood returns 4 blocks)
     */
    @SideOnly(Side.CLIENT)
    public void getSubBlocks(Item itemIn, CreativeTabs tab, List list)
    {
        BlockUnderwaterPlant.EnumType[] aenumtype = BlockUnderwaterPlant.EnumType.values();
        int i = aenumtype.length;

        for (int j = 0; j < i; ++j)
        {
            BlockUnderwaterPlant.EnumType enumtype = aenumtype[j];
            list.add(new ItemStack(itemIn, 1, enumtype.getMetadata()));
        }
    }

    /**
     * Convert the given metadata into a BlockState for this Block
     */
    public IBlockState getStateFromMeta(int meta)
    {
        return this.getDefaultState().withProperty(VARIANT, BlockUnderwaterPlant.EnumType.byMetadata(meta));
    }

    /**
     * Convert the BlockState into the correct metadata value
     */
    public int getMetaFromState(IBlockState state)
    {
        return ((BlockUnderwaterPlant.EnumType)state.getValue(VARIANT)).getMetadata();
    }

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

    public static enum EnumType implements IStringSerializable
    {
        KELP(0, "kelp"),
        RED_CORAL(1, "red_coral"),
    	BLUE_CORAL(2, "blue_coral");
        private static final BlockUnderwaterPlant.EnumType[] META_LOOKUP = new BlockUnderwaterPlant.EnumType[values().length];
        private final int meta;
        private final String name;
        private final String unlocalizedName;


        private EnumType(int meta, String name)
        {
            this(meta, name, name);
        }

        private EnumType(int meta, String name, String unlocalizedName)
        {
            this.meta = meta;
            this.name = name;
            this.unlocalizedName = unlocalizedName;
        }

        public int getMetadata()
        {
            return this.meta;
        }

        public String toString()
        {
            return this.name;
        }

        public static BlockUnderwaterPlant.EnumType byMetadata(int meta)
        {
            if (meta < 0 || meta >= META_LOOKUP.length)
            {
                meta = 0;
            }

            return META_LOOKUP[meta];
        }

        public String getName()
        {
            return this.name;
        }

        public String getUnlocalizedName()
        {
            return this.unlocalizedName;
        }

        static
        {
            BlockUnderwaterPlant.EnumType[] var0 = values();
            int var1 = var0.length;

            for (int var2 = 0; var2 < var1; ++var2)
            {
                BlockUnderwaterPlant.EnumType var3 = var0[var2];
                META_LOOKUP[var3.getMetadata()] = var3;
            }
        }
    }

@Override
public EnumPlantType getPlantType(IBlockAccess world, BlockPos pos) {
	return EnumPlantType.Desert;
}

@Override
public IBlockState getPlant(IBlockAccess world, BlockPos pos) {
	return this.getDefaultState();
}

/**
 * Is this block (a) opaque and (B) a full 1m cube? This determines whether or not to render the shared face of two
 * adjacent blocks and also whether the player can attach torches, redstone wire, etc to this block.
 */
public boolean isOpaqueCube()
{
	return false;
}
public boolean isFullCube()
    {
        return false;
    }

    @SideOnly(Side.CLIENT)
    public EnumWorldBlockLayer getBlockLayer()
    {
        return EnumWorldBlockLayer.CUTOUT;
    }
    
    public AxisAlignedBB getCollisionBoundingBox(World worldIn, BlockPos pos, IBlockState state)
    {
        return null;
    }


/**
 * Checks to see if its valid to put this block at the specified coordinates. Args: world, x, y, z
 */
@Override
public boolean canPlaceBlockAt(World worldIn, BlockPos pos)
{
	int par2 = pos.getX();
	int par3 = pos.getY();
	int par4 = pos.getZ();

	if((worldIn.getBlockState(pos).getBlock() == Blocks.water 
			&& worldIn.getBlockState(pos.down()).getBlock() == Blocks.sand
			&& worldIn.getBlockState(pos.up()).getBlock() == Blocks.water))
		return true;
	else if(worldIn.getBlockState(pos.down()).getBlock() == Blocks.sand
			&& worldIn.getBlockState(pos).getBlock() != Blocks.water)
		return true;
	else if((worldIn.getBlockState(pos).getBlock() == Blocks.water 
			&& worldIn.getBlockState(pos.down()).getBlock() == Blocks.gravel
			&& worldIn.getBlockState(pos.up()).getBlock() == Blocks.water))
		return true;
	else if(worldIn.getBlockState(pos.down()).getBlock() == Blocks.gravel
			&& worldIn.getBlockState(pos).getBlock() != Blocks.water)
		return true;
	else
		return false;
}

@Override
public void onNeighborBlockChange(World par1World, BlockPos pos, IBlockState state, Block par5)
{
	super.onNeighborBlockChange(par1World, pos, state, par5);
	this.checkFlowerChange(par1World, pos, state);
}

/**
 * Ticks the block if it's been scheduled
 */
@Override
public void updateTick(World par1World, BlockPos pos, IBlockState state, Random par5Random)
{
	this.checkFlowerChange(par1World, pos, state);
}

protected final void checkFlowerChange(World worldIn, BlockPos pos, IBlockState state)
{

	if (!this.canBlockStay(worldIn, pos, state))
	{
		this.dropBlockAsItem(worldIn, pos, state, 0);
		if(worldIn.getBlockState(pos.down()).getBlock().equals(Blocks.air))
			worldIn.setBlockToAir(pos);
		if(worldIn.getBlockState(pos.up()).getBlock().equals(Blocks.water))
			worldIn.setBlockToAir(pos);
		else
			worldIn.setBlockToAir(pos);
	}
}

/**
 * Can this block stay at this position.  Similar to canPlaceBlockAt except gets checked often with plants.
 */
public boolean canBlockStay(World worldIn, BlockPos pos, IBlockState state)
{
	BlockPos down = pos.down();
        Block soil = worldIn.getBlockState(down).getBlock();
        if (state.getBlock() != this) return this.canPlaceBlockOn(soil); //Forge: This function is called during world gen and placement, before this block is set, so if we are not 'here' then assume it's the pre-check.
        return soil.canSustainPlant(worldIn, down, net.minecraft.util.EnumFacing.UP, this);
}
}

 

This is the crash report, appearently the crash happen when i walk into the plant

 

[00:43:05] [server thread/ERROR]: Encountered an unexpected exception
net.minecraft.util.ReportedException: Exception while ticking a block
at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:769) ~[MinecraftServer.class:?]
at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:669) ~[MinecraftServer.class:?]
at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:171) ~[integratedServer.class:?]
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:540) [MinecraftServer.class:?]
at java.lang.Thread.run(Unknown Source) [?:1.8.0_45]
Caused by: java.lang.IllegalArgumentException: Cannot get property PropertyInteger{name=level, clazz=class java.lang.Integer, values=[0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15]} as it does not exist in BlockState{block=mw:underwater_plant, properties=[variant]}
at net.minecraft.block.state.BlockState$StateImplementation.getValue(BlockState.java:167) ~[blockState$StateImplementation.class:?]
at net.minecraft.block.BlockLiquid.getLevel(BlockLiquid.java:64) ~[blockLiquid.class:?]
at net.minecraft.block.BlockDynamicLiquid.checkAdjacentBlock(BlockDynamicLiquid.java:273) ~[blockDynamicLiquid.class:?]
at net.minecraft.block.BlockDynamicLiquid.updateTick(BlockDynamicLiquid.java:48) ~[blockDynamicLiquid.class:?]
at net.minecraft.world.WorldServer.tickUpdates(WorldServer.java:660) ~[WorldServer.class:?]
at net.minecraft.world.WorldServer.tick(WorldServer.java:226) ~[WorldServer.class:?]
at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:763) ~[MinecraftServer.class:?]
... 4 more
[00:43:05] [server thread/ERROR]: This crash report has been saved to: D:\Documenti\Mineworld\MineWorld 1.8\eclipse\.\crash-reports\crash-2015-06-23_00.43.05-server.txt
[00:43:05] [server thread/INFO]: Stopping server
[00:43:05] [server thread/INFO]: Saving players
[00:43:05] [Client thread/INFO] [sTDOUT]: [net.minecraft.init.Bootstrap:printToSYSOUT:660]: ---- Minecraft Crash Report ----
// Surprise! Haha. Well, this is awkward.

Time: 23/06/15 0.43
Description: Exception while ticking a block

java.lang.IllegalArgumentException: Cannot get property PropertyInteger{name=level, clazz=class java.lang.Integer, values=[0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15]} as it does not exist in BlockState{block=mw:underwater_plant, properties=[variant]}
at net.minecraft.block.state.BlockState$StateImplementation.getValue(BlockState.java:167)
at net.minecraft.block.BlockLiquid.getLevel(BlockLiquid.java:64)
at net.minecraft.block.BlockDynamicLiquid.checkAdjacentBlock(BlockDynamicLiquid.java:273)
at net.minecraft.block.BlockDynamicLiquid.updateTick(BlockDynamicLiquid.java:48)
at net.minecraft.world.WorldServer.tickUpdates(WorldServer.java:660)
at net.minecraft.world.WorldServer.tick(WorldServer.java:226)
at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:763)
at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:669)
at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:171)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:540)
at java.lang.Thread.run(Unknown Source)


A detailed walkthrough of the error, its code path and all known details is as follows:
---------------------------------------------------------------------------------------

-- Head --
Stacktrace:
at net.minecraft.block.state.BlockState$StateImplementation.getValue(BlockState.java:167)
at net.minecraft.block.BlockLiquid.getLevel(BlockLiquid.java:64)
at net.minecraft.block.BlockDynamicLiquid.checkAdjacentBlock(BlockDynamicLiquid.java:273)
at net.minecraft.block.BlockDynamicLiquid.updateTick(BlockDynamicLiquid.java:48)

-- Block being ticked --
Details:
Block: minecraft:flowing_water[level=8]
Block location: World: (169,61,251), Chunk: (at 9,3,11 in 10,15; contains blocks 160,0,240 to 175,255,255), Region: (0,0; contains chunks 0,0 to 31,31, blocks 0,0,0 to 511,255,511)
Stacktrace:
at net.minecraft.world.WorldServer.tickUpdates(WorldServer.java:660)
at net.minecraft.world.WorldServer.tick(WorldServer.java:226)

-- Affected level --
Details:
Level name: New World
All players: 1 total; [EntityPlayerMP['Player652'/450, l='New World', x=168,85, y=62,00, z=251,09]]
Chunk stats: ServerChunkCache: 727 Drop: 0
Level seed: -6033536155721142239
Level generator: ID 00 - default, ver 1. Features enabled: true
Level generator options: 
Level spawn location: 208,00,64,00,240,00 - World: (208,64,240), Chunk: (at 0,4,0 in 13,15; contains blocks 208,0,240 to 223,255,255), Region: (0,0; contains chunks 0,0 to 31,31, blocks 0,0,0 to 511,255,511)
Level time: 605 game time, 605 day time
Level dimension: 0
Level storage version: 0x04ABD - Anvil
Level weather: Rain time: 92048 (now: false), thunder time: 101547 (now: false)
Level game mode: Game mode: creative (ID 1). Hardcore: false. Cheats: true
Stacktrace:
at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:763)
at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:669)
at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:171)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:540)
at java.lang.Thread.run(Unknown Source)

-- System Details --
Details:
Minecraft Version: 1.8
Operating System: Windows 8.1 (amd64) version 6.3
Java Version: 1.8.0_45, Oracle Corporation
Java VM Version: Java HotSpot(TM) 64-Bit Server VM (mixed mode), Oracle Corporation
Memory: 788284744 bytes (751 MB) / 1038876672 bytes (990 MB) up to 1038876672 bytes (990 MB)
JVM Flags: 3 total; -Xincgc -Xmx1024M -Xms1024M
IntCache: cache: 0, tcache: 0, allocated: 13, tallocated: 95
FML: MCP v9.10 FML v8.99.8.1412 Minecraft Forge 11.14.1.1412 4 mods loaded, 4 mods active
mcp{9.05} [Minecraft Coder Pack] (minecraft.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
FML{8.99.8.1412} [Forge Mod Loader] (forgeSrc-1.8-11.14.1.1412.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
Forge{11.14.1.1412} [Minecraft Forge] (forgeSrc-1.8-11.14.1.1412.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
mw{0.5} [MineWorld] (bin) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
Loaded coremods (and transformers): 
GL info: ~~ERROR~~ RuntimeException: No OpenGL context found in the current thread.
Profiler Position: N/A (disabled)
Player Count: 1 / 8; [EntityPlayerMP['Player652'/450, l='New World', x=168,85, y=62,00, z=251,09]]
Type: Integrated Server (map_client.txt)
Is Modded: Definitely; Client brand changed to 'fml,forge'
[00:43:05] [Client thread/INFO] [sTDOUT]: [net.minecraft.init.Bootstrap:printToSYSOUT:660]: #@!@# Game crashed! Crash report saved to: #@!@# .\crash-reports\crash-2015-06-23_00.43.05-server.txt
[00:43:05] [Client thread/INFO] [FML]: Waiting for the server to terminate/save.
[00:43:05] [server thread/INFO]: Saving worlds
[00:43:05] [server thread/INFO]: Saving chunks for level 'New World'/Overworld
[00:43:05] [server thread/INFO]: Saving chunks for level 'New World'/Nether
[00:43:05] [server thread/INFO]: Saving chunks for level 'New World'/The End
[00:43:06] [server thread/INFO] [FML]: Unloading dimension 0
[00:43:06] [server thread/INFO] [FML]: Unloading dimension -1
[00:43:06] [server thread/INFO] [FML]: Unloading dimension 1
[00:43:06] [server thread/INFO] [FML]: Applying holder lookups
[00:43:06] [server thread/INFO] [FML]: Holder lookups applied
[00:43:06] [server thread/INFO] [FML]: The state engine was in incorrect state SERVER_STOPPING and forced into state SERVER_STOPPED. Errors may have been discarded.
[00:43:06] [Client thread/INFO] [FML]: Server terminated.

 

Thanks in advance for all who will help me

[SCAREX]

Can you give us the crash report ?

[JimiIT92]

Crash report added to the main post

[larsgerrits]

Caused by: java.lang.IllegalArgumentException: Cannot get property PropertyInteger{name=level, clazz=class java.lang.Integer, values=[0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15]} as it does not exist in BlockState{block=mw:underwater_plant, properties=[variant]}
at net.minecraft.block.state.BlockState$StateImplementation.getValue(BlockState.java:167) ~[blockState$StateImplementation.class:?]
at net.minecraft.block.BlockLiquid.getLevel(BlockLiquid.java:64) ~[blockLiquid.class:?]

It looks like it has something to with a method in the

BlockLiquid

class. Look at that method on that line and look what is causing the crash.

[Jedispencer21]

Caused by: java.lang.IllegalArgumentException: Cannot get property
PropertyInteger{name=level, clazz=class java.lang.Integer, values=[0, 1, 2, 3, 4, 5, 6,
7, 8, 9, 10, 11, 12, 13, 14, 15]} as it does not exist in BlockState{block=mw:underwater_plant, properties=[variant]}

 

If you read that is says that the PropertyInterger doesn't exist in the block mw:underwater_plant

[SCAREX]

You are using a liquid material :

super(Material.water);

 

try with a correct Material.

[JimiIT92]

Thanks a lot! Changing it to

Material.ground

has solved the problem I don't know why, becaus in 1.7.x i used the water material and all works fine

Thanks a lot for helped me