[dcmd_crash]

Lol, you only define one callback, and put everything under it

No need for several, do it like below:

pawn Код:

//Health pickups
public OnPlayerPickUpPickup(playerid, pickupid)
{
    if (pickupid == Health)
    {
    SetPlayerHealth(playerid,100);
    GameTextForPlayer(playerid, "~y~You have found some health!", 5000, 3);
    }
    if (pickupid == Armour)
    {
    SetPlayerArmour(playerid,100);
    GameTextForPlayer(playerid, "~y~You have found an armor!", 5000, 3);
    }
    return 1;
}