[lelemaster]

@Stu1, seriously, if you don't know how to script, please don't try to help people with random codes.

You are miss using the IsVehicleOccupied

use it that way:

if(IsVehicleOccupied(GetPlayerVehicleID(playerid)) )

Will be:

Код:

//=============================Fuel Stations==============================// 
if(pickupid == LVFuelPickup1) 
{ 
     if(IsVehicleOccupied(GetPlayerVehicleID(playerid)))
     { 
         SendClientMessage(playerid, COLOR_YELLOW,"[GAS STATION] /refuel  To refuel your vehile. "); 
     } 
} 
if(pickupid == LVFuelPickup2) 
{ 
     if(IsVehicleOccupied(GetPlayerVehicleID(playerid)))
     { 
          SendClientMessage(playerid, COLOR_YELLOW,"[GAS STATION] /refuel  To refuel your vehile. "); 
     } 
}
//====================================================// 
//Stock IsVehicleOccupied. 
stock IsVehicleOccupied(vehicleid) 
{ 
    for(new i = 0; i < MAX_PLAYERS; i++) 
    { 
        if(IsPlayerInVehicle(i, vehicleid) && GetPlayerState(i) == PLAYER_STATE_DRIVER) 
        { 
            return 1; 
        } 
    } 
  return 0; 
}

Why don't you do: if(GetPlayerVehicleSeat(playerid) == 0)

It will be the same function as IsVehicleOccupied, but less buggy.

Will be:

Код:

//=============================Fuel Stations==============================// 
if(pickupid == LVFuelPickup1) 
{ 
     if(GetPlayerVehicleSeat(playerid) == 0)
     { 
         SendClientMessage(playerid, COLOR_YELLOW,"[GAS STATION] /refuel  To refuel your vehile. "); 
     } 
} 
if(pickupid == LVFuelPickup2) 
{ 
     if(GetPlayerVehicleSeat(playerid) == 0)
     { 
         SendClientMessage(playerid, COLOR_YELLOW,"[GAS STATION] /refuel  To refuel your vehile. "); 
     } 
}