[Paladin]

So .. I've read almost everything I could until I was blue in the face about this key story.. basically I'm wanting to create something that when someone presses their up key, it sends them a message saying "Woop woop", can some one assist me and tell me where I'm going wrong:

Code:

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys)
{
	if(newkeys == KEY_UP)
	{
	SendClientMessage(playerid, COLOR_YELLOW, "Woop Woop");
  }
}

Thanks in advanced!