[[DOG]irinel1996]

Give a try, I think everything looks ok:

pawn Code:

dcmd_vcall(playerid, params[])
{
    #pragma unused params
    new Slot;
    if(sscanf(params,"d",Slot)) return SendClientMessage(playerid, Red, "[ERROR]Usage: /vcall [Slot]");
    if(Slot == 1)
    {
        if(Car1(playerid) == 0) return SendClientMessage(playerid, Red, "You didn't have a car in slot 1");
        SetTimerEx("Timer1",5000,false,"i",playerid);
        SetVehicleToRespawn(Car1(playerid));
        SendClientMessage(playerid, Green, "You has call you vehicle. Please wait 5 second");
    }
    else if(Slot == 2)
    {
        if(Car2(playerid) == 0) return SendClientMessage(playerid, Red, "You didn't have a car in slot 2");
        SetTimerEx("Timer1",5000,false,"i",playerid);
        SetVehicleToRespawn(Car2(playerid));
        SendClientMessage(playerid, Green, "You has call you vehicle. Please wait 5 second");
    }
    else if(Slot == 3)
    {
        if(Car3(playerid) == 0) return SendClientMessage(playerid, Red, "You didn't have a car in slot 3");
        SetTimerEx("Timer1",5000,false,"i",playerid);
        SetVehicleToRespawn(Car3(playerid)); //I added this line.
        SendClientMessage(playerid, Green, "You has call you vehicle. Please wait 5 second");
    }
    return 1;
}

dcmd_vunbuyable(playerid, params[])
{
    #pragma unused params
    new vehicleid = GetPlayerVehicleID(playerid), string[50];
    if(!IsPlayerAdmin(playerid)) return SendClientMessage(playerid,Red,"[ERROR]Only RCON Admin can use this command");
    if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid,Red,"[ERROR]You must in vehicle to use this command");
    if(VehicleInfo[vehicleid][ToSell] == 0) return SendClientMessage(playerid,Red,"[ERROR]This is not a buyable");
    if(dini_Int(VFile(vehicleid),"UnBuyable") == 1) return SendClientMessage(playerid, Red, "[ERROR]This vehicle already in unbuyable");
    if(!dini_Exists(VFile(vehicleid)))
    {
        dini_Create(VFile(vehicleid));
        SetUnbuyableVehicle(vehicleid, 1);
    }
    else
    {
        SetUnbuyableVehicle(vehicleid, 1
    }
    format(string, sizeof(string), "Now this %s is in unbuyable", VehNames[GetVehicleModel(vehicleid)-400]);
    SendClientMessage(playerid, Green, string);
    return 1;
}

And define your functions and colors.