lets make a list of issues.
1. You need to package this
2. You need to match up each { with a }
3. %name doesn't really exist yet. I think you want %client.name
4. Where the heck does the string control come from?
5. Your function doesn't do anything
6. I got bored making this list, but you have more things done wrong than correctly
package messageonjoin
{
function gameconnection::oncliententergame(%client)
{
messageClient(%client, "", "You have entered the game!");
parent::oncliententergame(%client);
}
};
activatePackage(messageonjoin);