[greentarch]

Looks nice, btw ( noticed a typo )

pawn Code:

CMD:harvest(playerid, params[])
{
    if(PlayerInfo[playerid][pJob] == 1)
    {
        new vehicleid = GetPlayerVehicleID(playerid);
        if(IsAFarverCar(vehicleid) && GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
        {
            if(gPlayerCheckpointStatus[playerid] == CHECKPOINT_NONE)
            {
                SendClientMessage(playerid, COLOR_GREY, "[FarmerJob]:Follow The Checkpoints and get your reward.");
                SetPlayerCheckpoint(playerid, -309.0990,-1381.1797,10.7049, 10);
            }
        }
        else return SendClientMessage(playerid, COLOR_GREY, "[FarmerJob]:You are not at Farm!!");
    }
    else return SendClientMessage(playerid, COLOR_GREY, "Y[FarmerJob]:ou are not a Farmer!");
    return 1;
}

Look at this :

pawn Code:

else return SendClientMessage(playerid, COLOR_GREY, "Y[FarmerJob]:ou are not a Farmer!");

Shouldn't it be :

pawn Code:

else return SendClientMessage(playerid, COLOR_GREY, "[FarmerJob]: You are not a Farmer!");