Hello, I am working on a mod and i want to make a machine that if you click on it with a specific item takes the item and "says" something in chat and I've tried using the onBlockActivated but it doesn't work, here's my code:

public class ComplimentsMachine extends Block {
public Random random = new Random();
public EntityPlayer player;
public int compliment = random.nextInt(2);
boolean c0, c1;

public ComplimentsMachine(Material materialIn) {
	super(materialIn);
}

public boolean onBlockActivated(World worldIn, BlockPos pos, IBlockState state, EntityPlayer playerIn, EnumHand hand, @Nullable ItemStack heldItem, EnumFacing side, float hitX, float hitY, float hitZ) {
	Item equipped = playerIn.getHeldItemMainhand() != null ? playerIn.getHeldItemMainhand().getItem() : null;
	if (worldIn.isRemote) {
		return true;
	} else {
		if (compliment == 0) {
			player.addChatMessage(new TextComponentString("§6Welcome§r §e" + player.getDisplayNameString() + "§r§6, have fun!§r"));
		}
		return true;
	}
}
}

Please help!

Those where from another thing i tried, forgot to remove them

When i right-click on the machine it doesn't do anything

package com.Egietje.degeweldigemod.init;

import java.util.Random;

import javax.annotation.Nullable;

import net.minecraft.block.Block;
import net.minecraft.block.BlockChest;
import net.minecraft.block.material.Material;
import net.minecraft.block.state.IBlockState;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.stats.StatList;
import net.minecraft.util.EnumFacing;
import net.minecraft.util.EnumHand;
import net.minecraft.util.math.BlockPos;
import net.minecraft.util.text.TextComponentString;
import net.minecraft.world.ILockableContainer;
import net.minecraft.world.World;

public class ComplimentsMachine extends Block {
public Random random = new Random();
public EntityPlayer player;
public int compliment = random.nextInt(2);

public ComplimentsMachine(Material materialIn) {
	super(materialIn);
}

public boolean onBlockActivated(World worldIn, BlockPos pos, IBlockState state, EntityPlayer playerIn, EnumHand hand, @Nullable ItemStack heldItem, EnumFacing side, float hitX, float hitY, float hitZ) {
	Item equipped = playerIn.getHeldItemMainhand() != null ? playerIn.getHeldItemMainhand().getItem() : null;
	if (worldIn.isRemote) {
		return true;
	} else {
		if (compliment == 0) {
			player.addChatMessage(new TextComponentString("§6Welcome§r §e" + player.getDisplayNameString() + "§r§6, have fun!§r"));
		}
		return true;
	}
}
}

Oh and btw, that welcome PLAYERNAME, have fun was as a test

 

I've found the wrong thing, at the if (compliment == 0) part i have player.addChatMessage and not playerIn.addChatMessage