[1.10.2] [SOLVED] Can't set blockstate after block is placed

[SuperHB]

When I place down a block and right click it, I want to change the blockstate. But when I right click the block, it changes blockstates for a second before switching back.

 

Here is the code:

 

Block

public class BlockSentry extends Block implements ISpecialBlock, ITileEntityProvider {
public static final PropertyEnum LEVEL = PropertyEnum.create("level", BlockSentry.EnumLevel.class);
public static final PropertyEnum TEAM = PropertyEnum.create("team", EnumTeam.class);

public BlockSentry (Material material) {
	super(material);

	setDefaultState(blockState.getBaseState().withProperty(LEVEL, EnumLevel.ONE).withProperty(TEAM, EnumTeam.BLU));
}

@Override
protected BlockStateContainer createBlockState () {
	return new BlockStateContainer(this, new IProperty[] { LEVEL, TEAM });
}

@Override
public IBlockState getStateFromMeta (int meta) {
	switch (meta) {
		case 0:
			return getDefaultState().withProperty(LEVEL, EnumLevel.MINI).withProperty(TEAM, EnumTeam.BLU);
		case 1:
			return getDefaultState().withProperty(LEVEL, EnumLevel.ONE).withProperty(TEAM, EnumTeam.BLU);
		case 2:
			return getDefaultState().withProperty(LEVEL, EnumLevel.TWO).withProperty(TEAM, EnumTeam.BLU);
		case 3:
			return getDefaultState().withProperty(LEVEL, EnumLevel.THREE).withProperty(TEAM, EnumTeam.BLU);
		case 4:
			return getDefaultState().withProperty(LEVEL, EnumLevel.MINI).withProperty(TEAM, EnumTeam.RED);
		case 5:
			return getDefaultState().withProperty(LEVEL, EnumLevel.ONE).withProperty(TEAM, EnumTeam.RED);
		case 6:
			return getDefaultState().withProperty(LEVEL, EnumLevel.TWO).withProperty(TEAM, EnumTeam.RED);
		case 7:
			return getDefaultState().withProperty(LEVEL, EnumLevel.THREE).withProperty(TEAM, EnumTeam.RED);
	}
	return getDefaultState().withProperty(LEVEL, EnumLevel.MINI).withProperty(TEAM, EnumTeam.BLU);
}

@Override
public boolean onBlockActivated (World worldIn, BlockPos pos, IBlockState state, EntityPlayer playerIn, EnumHand hand, ItemStack heldItem, EnumFacing side, float hitX, float hitY, float hitZ) {
	if (worldIn.isRemote) {
		TileEntity tile = worldIn.getTileEntity(pos);

		if (tile instanceof TileEntitySentry) {
			TileEntitySentry sentry = (TileEntitySentry)tile;

			if (heldItem != null) {
				if (heldItem.getItem() == TFCItems.wrench) {
					worldIn.setBlockState(pos, getStateFromMeta(6));
				}
			}
		}
	}
	return true;
}

@Override
public TileEntity createNewTileEntity (World worldIn, int meta) {
	return new TileEntitySentry(meta);
}
}

 

TileEntity

public class TileEntitySentry extends TileEntity {
public static int level;
private static int meta;

public TileEntitySentry (int meta) {
	level = getLevelFromMeta(meta);
	this.meta = meta;
}

public void setLevel (int level) {
	this.level = level;
}

public int getLevel () {
	return level;
}

public int getMeta () {
	return meta;
}

@Override
public NBTTagCompound writeToNBT (NBTTagCompound compound) {
	super.writeToNBT(compound);

	compound.setTag("sentry", new NBTTagList());
	NBTTagCompound sentry = compound.getCompoundTag("sentry");
	sentry.setInteger("level", level);

	return compound;
}

@Override
public void readFromNBT (NBTTagCompound compound) {
	super.readFromNBT(compound);

	level = compound.getInteger("level");
}

private int getLevelFromMeta (int meta) {
	switch (meta) {
		case 0: return 0;
		case 1: return 1;
		case 2: return 2;
		case 3: return 3;
		case 4: return 0;
		case 5: return 1;
		case 6: return 2;
		case 7: return 3;
	}
	return 0;
}

@Override
public boolean shouldRefresh (World world, BlockPos pos, IBlockState old, IBlockState newState) {
	return false;
}
}

[Draco18s]

if (worldIn.isRemote)

 

Lets only do this on the client because the server has control over everything.

Why?

Raisins.

[Choonster]

World#isRemote

is

true

on the client and

false

on the server. The server is in control of the game state, but you're making your changes on the client so they get overwritten the next time it syncs.

 

Side note: Consider using bitwise operations when storing multiple values in metadata instead of using harcoded metadata values for each combination.

[SuperHB]

if (worldIn.isRemote)

 

Lets only do this on the client because the server has control over everything.

Why?

Raisins.

 

I'm stupid... thank you