[Include] OnVehicleRespray for pay 'n' spray

[MP2]

Makes OnVehicleRespray be called when a player uses pay 'n' spray.

DOWNLOAD
http://pastebin.com/zX96D4dA

Works by setting vehicles' health to 999 when created and detecting an increase to 1000.

Simples.

NOTE: 'color1' and 'color2' values will always be -1 in OnVehicleRespray.

[FireCat]

Good, I guess?

[Vince]

Requires Y_Hooks .. Not very user friendly for those who decide not to use that.

Works by setting vehicles' health to 999 when created and detecting an increase to 1000

i didn't get that part ?

does that mean it is going to work if an admin used "/fix [playerid]" to fix the specified player's vehicle ?

[KyleSmith]

You should make it call for when it's resprayed in transfender etc.

[Hanger]

Have you heard about vehicle health hack? Hackers may spam health hack therefore server will be lagged.

[MP2]

Originally Posted by FireCat

Good, I guess?

You guess correct.

Originally Posted by Vince

Requires Y_Hooks .. Not very user friendly for those who decide not to use that.

I agree, I may change it.

Originally Posted by Lexi'

i didn't get that part ?

does that mean it is going to work if an admin used "/fix [playerid]" to fix the specified player's vehicle ?

If you set a vehicle's health to 1000 it will just set it back to 999. It only calls it if you are in range of a pay 'n' spray for this very reason.

Originally Posted by KyleSmith

You should make it call for when it's resprayed in transfender etc.

It already is natively - this makes it work for pay 'n' spray also.

Originally Posted by Hanger

Have you heard about vehicle health hack? Hackers may spam health hack therefore server will be lagged.

Then ban them. This actually makes it a lot easier to detect that hack.

[Niko_boy]

preety much usefull all around SAMP =)

[CaptainMactavish]

So you used this in OnPlayerUpdate being conscious that this callback is called around 30 times per second and per player?

[MP2]

Originally Posted by CaptainMactavish

So you used this in OnPlayerUpdate being conscious that this callback is called around 30 times per second and per player?

All it does is check vehicle health - it's not 'damaging'. It's probably more efficient than a timer and loop.

People have it drilled in to their heads that using OnPlayerUpdate is BAD. It's not. Having a loop through 2000 vehicles in there is bad yes, but a single statement is not. It was added to be used, not to just sit there un-used. People complain when they use memory, and when they use CPU, but they're there to be used. I'm not saying efficiency is bad, I just think people go over the top.