Garryware minigame "RocketHop"

First lua that’s made it’s self out of my pc

[lua]WARE.Author = “Nannal”
WARE.Room = “empty”

function WARE:Initialize()
GAMEMODE:SetWareWindupAndLength(1,9)

GAMEMODE:DrawPlayersTextAndInitialStatus("Look Down !",0)
return

end

function WARE:StartAction()
local a = math.random(2,10)
GAMEMODE:DrawPlayersTextAndInitialStatus(“Rocket jump “…a…” times !”,0)

for k,v in pairs(team.GetPlayers(TEAM_HUMANS)) do 
	v:Give( "ware_weap_rocketjump" )
	end 
	return
	end 
	 function WARE:EndAction()  

function WARE:Think( )  
for k,v in pairs(team.GetPlayers(TEAM_HUMANS)) do   
    if (PLAYER SHOOT > a) then v:WarePlayerDestinyWin( )
    else v:WarePlayerDestinyLose( ) end  
end  

end
end [/lua]

PLAYER SHOOT is a place holder for whatever should be in there. Basically I’m checking if the player has shot X number of times

please look over it in as much depth as is possible and tell me if/where else I’ve gone wrong. also i’ll change up the numbers so they go from easy to hard as opposed to arbitrary numbers like they are at the minute

also about 05% of the credit goes to everyone who’s garryware minigames have been added to the svn, code was of great help, thank you ('specially you Ha3)

[editline]EDIT[/editline]

argh, i did something to cock it up again, “error with minigame rockethop!” and it was working so nicely before ( you know, aside from not working)

[editline]EDIT AGAIN:[/editline]

[lua] function WARE:Think( )
for k,v in pairs(team.GetPlayers(TEAM_HUMANS)) do
if (v:GetVelocity():height() > a) then v:WarePlayerDestinyWin( ) end
end [/lua]

makes it playable but unwinable