05.11.2012, 21:19
Thanks! However for the LONGEST time I didnt realize something...started scripting in 2007 and only realized this in 2012 once it was pointed out:
Vechicle ID CAN be 0!!! I had to change everything in Everystuff from:
if(!GetPlayerVehicleID(playerid))
to
if(GetPlayerVehicleID(playerid) == INVALID_VEHICLE_ID)
The first one checks to see if vehicleid is 0...assuming that there is no 'vehicle 0'. That assumption was wrong...the best way to check if a vehicleid is valid is to use things like:
if(GetPlayerVehicleID(playerid) == INVALID_VEHICLE_ID) SendClientMessage(playerid,COLOR_WHATEVER,"Thats not a real car!");
if(GetPlayerVehicleID(playerid) != INVALID_VEHICLE_ID) SendClientMessage(playerid,COLOR_WHATEVER,"That IS a real car!");
Vechicle ID CAN be 0!!! I had to change everything in Everystuff from:
if(!GetPlayerVehicleID(playerid))
to
if(GetPlayerVehicleID(playerid) == INVALID_VEHICLE_ID)
The first one checks to see if vehicleid is 0...assuming that there is no 'vehicle 0'. That assumption was wrong...the best way to check if a vehicleid is valid is to use things like:
if(GetPlayerVehicleID(playerid) == INVALID_VEHICLE_ID) SendClientMessage(playerid,COLOR_WHATEVER,"Thats not a real car!");
if(GetPlayerVehicleID(playerid) != INVALID_VEHICLE_ID) SendClientMessage(playerid,COLOR_WHATEVER,"That IS a real car!");