can you please review this code

[Machine]

what i am trying to achieve is when killer kill another player he will get all the killed player money , is this functional or i messed up the parameters  

function rewardOnWasted ( ammo, killer, killerweapon, bodypart )
	local money = getPlayerMoney(source)
	if ( killer ) and ( killer ~= source ) then 
		givePlayerMoney ( killer, source )
			end
end
addEventHandler ( "onPlayerWasted", getRootElement(), rewardOnWasted )

 

[IIYAMA]

27 minutes ago, Machine said:

is this functional or i messed up the parameters  

You can't give a player to another player.

givePlayerMoney ( killer, source )

 

Some extra validation might be required, not only a player can kill another player.

if ( killer ) and ( killer ~= source ) and getElementType(killer) == "player" and money > 0 then 
	givePlayerMoney ( killer, money )
end

 

 

[Tekken]

Replace givePlayerMoney(killer, source) with 

givePlayerMoney(killer, (getPlayerMoney(source) or 0));

 

As you need to get the killed player money before you can give it to the killer.

Good luck.

[TAPL]

Also use setPlayerMoney to remove the killed player money after giving the amount to the killer, assuming that's what you're trying to do.