score counter programming problems

I'm having a bit of trouble with a program I made for a game. In the game, there are ten items that I want the player to collect, and when he collects them, I want the item to disappear and add 1 to the score counter. Until of course he collects all 10, then a message will pop up. I made the score counter in the gui editor and it is on the top right of the screen. It simply says before any of the items are collected: "Items: 0". And the score counters name is simply: "ScoreCounter". The program works fine except, when the player bumps into the item, and the item dissapears, it does not add 1 to the score counter. I would like to know why it won't add 1 to the score counter every time the player gets an item. here is the code.



function Item::onCollision(%this, %obj, %col)
{
if(%col.getClassName() $= "Player")
{
%client = %col.client;
%client.score++;
commandToClient(%client, 'SetScoreCounter', %client.score);
obj.delete();
%itemCount = Items.getCount();
if(%itemCount > 0)
return;
//otherwise display victory screen
commandToClient(%client, 'ShowVictory', %client.score);
}
}


Can anyone see why this code may be faulty?