1. Search the forum for an example of armor::ontrigger
2. If you have the brick object, I believe it's just .getName()
As for "activating" the brick, do you mean an output event to be triggered by an onActivate input event? If so, just do similar (but fixed) to what you tried in #3. If not, you'll need to also use ontrigger, as well as a raycast (look up an example of that), and a few checks to make sure it's a brick/right type of brick/right name
3. Your client variable is actually a player. Rename it %player to better describe what it contains, and then use %player.client when you need the client.
%this is also an incorrect name. Standards-wise, it should only be used to describe the object a method is being called on, and as such it should only ever be the first argument
So in summary, change
function Player::addLap(%client, %this)
To
function Player::addLap(%this, %laps)
Change any reference to %client to %this.client, and any reference from %this to %laps