need help with weapon dialog

Why when i pickup the case it doesnt show me the dialog.This is the code i have:



public OnGameModeInit()

{

weapon = CreatePickup(1210, 2, 2521.3564,-1681.3511,14.8824,-1);

weapon = CreatePickup(1210, 2, 1905.4609,-1352.9421,13.4692,-1);

weapon = CreatePickup(1210, 2, 2216.8201,-1171.2583,25.7266,-1);

weapon = CreatePickup(1210, 2, 2312.2629,-1203.1252,24.8876,-1);

weapon = CreatePickup(1210, 2, 2352.2600,-1507.4819,24.0000,-1);

weapon = CreatePickup(1210, 2, 2503.9373,-1758.9484,13.4601,-1);

weapon = CreatePickup(1210, 2, 1816.2310,-1556.9333,13.5052,-1);

weapon = CreatePickup(1210, 2, 1762.9436,-1906.0568,13.5670,-1);

weapon = CreatePickup(1210, 2, 1543.4761,-1619.3920,13.5546,-1);





public OnPlayerPickUpPickup(playerid, pickupid)

{

if(pickupid == weapon)

{

ShowPlayerDialog(playerid, DIALOG_WEAPONS, DIALOG_STYLE_LIST, "{6EF83C}Reckless Ammunitions", "\n{F81414}Armour($3000)\nHealth($2000)\n{FFFFFF}S niper Rifle($3500)\nTec-9($3000)\nMP5($3000)\nUzi($3000)\nConbat Shotgun($2300)\nSawnoff($2200)\nShotgun($2200)\nDe sert Eagle($1900)\nSilenced 9mm($1500)\n9mm($1500)\nM4($4000)", "buy", "close");

}







if(dialogid ==DIALOG_WEAPONS)

{

if(response)

{

if(listitem == 0)

{

if(GetPlayerMoney(playerid) < 3000) return SendClientMessage(playerid, 0xFF0000FF,"You do not have enough money to buy this weapon!");

GivePlayerMoney(playerid, -3000);

SetPlayerArmour(playerid, 100);

}

if(listitem == 1)

{

if(GetPlayerMoney(playerid) < 2000) return SendClientMessage(playerid, 0xFF0000FF,"You do not have enough money to buy this weapon!");

GivePlayerMoney(playerid, -2000);

SetPlayerHealth(playerid, 100);

}

if(listitem == 2)

{

if(GetPlayerMoney(playerid) < 4000) return SendClientMessage(playerid, 0xFF0000FF,"You do not have enough money to buy this weapon!");

GivePlayerMoney(playerid, -4000);

GivePlayerWeapon(playerid, 34, 100);

}

if(listitem == 3)

{

if(GetPlayerMoney(playerid) < 3000) return SendClientMessage(playerid, 0xFF0000FF,"You do not have enough money to buy this weapon!");

GivePlayerMoney(playerid, -3000);

GivePlayerWeapon(playerid, 32, 100);

}

if(listitem == 4)

{

if(GetPlayerMoney(playerid) < 3000) return SendClientMessage(playerid, 0xFF0000FF,"You do not have enough money to buy this weapon!");

GivePlayerMoney(playerid, -3000);

GivePlayerWeapon(playerid, 29, 100);

}

if(listitem == 5)

{

if(GetPlayerMoney(playerid) < 3000) return SendClientMessage(playerid, 0xFF0000FF,"You do not have enough money to buy this weapon!");

GivePlayerMoney(playerid, -3000);

GivePlayerWeapon(playerid, 28, 100);

}

if(listitem == 6)

{

if(GetPlayerMoney(playerid) < 2200) return SendClientMessage(playerid, 0xFF0000FF,"You do not have enough money to buy this weapon!");

GivePlayerMoney(playerid, -2200);

GivePlayerWeapon(playerid, 27, 100);

}

if(listitem == 7)

{

if(GetPlayerMoney(playerid) < 2300) return SendClientMessage(playerid, 0xFF0000FF,"You do not have enough money to buy this weapon!");

GivePlayerMoney(playerid, -2300);

GivePlayerWeapon(playerid, 26, 100);

}

if(listitem == 8)

{

if(GetPlayerMoney(playerid) < 2200) return SendClientMessage(playerid, 0xFF0000FF,"You do not have enough money to buy this weapon!");

GivePlayerMoney(playerid, -2200);

GivePlayerWeapon(playerid, 25, 100);

}

if(listitem == 9)

{

if(GetPlayerMoney(playerid) < 1900) return SendClientMessage(playerid, 0xFF0000FF,"You do not have enough money to buy this weapon!");

GivePlayerMoney(playerid, -1900);

GivePlayerWeapon(playerid, 24, 100);

}

if(listitem == 10)

{

if(GetPlayerMoney(playerid) < 1500) return SendClientMessage(playerid, 0xFF0000FF,"You do not have enough money to buy this weapon!");

GivePlayerMoney(playerid, -1500);

GivePlayerWeapon(playerid, 23, 100);

}

if(listitem == 11)

{

if(GetPlayerMoney(playerid) < 1500) return SendClientMessage(playerid, 0xFF0000FF,"You do not have enough money to buy this weapon!");

GivePlayerMoney(playerid, -1500);

GivePlayerWeapon(playerid, 22, 100);

}

if(listitem == 12)

{

if(GetPlayerMoney(playerid) < 4000) return SendClientMessage(playerid, 0xFF0000FF,"You do not have enough money to buy this weapon!");

GivePlayerMoney(playerid, -4000);

GivePlayerWeapon(playerid, 31, 100);

}

}

}