[Tanush123]

For some reason when i click passport, it works. When i click Gun License and after click buy gun licenses or Guns, nothing pops out.

Dialog

pawn Код:

ShowPlayerDialog(playerid,47,DIALOG_STYLE_LIST,"City hall","Passport\nGun License","Open","Close");

Response to dialog (the problem)

pawn Код:

if(dialogid == 47)
        {
            if(response)
            {
                if(listitem == 0)
                {
                    ShowPlayerDialog(playerid,48,DIALOG_STYLE_LIST,"Passport","Montgomery $1000\nJefferson $1000\nAll Saints $1000","Buy","Close");
                }
                if(listitem == 1)
                {
                    ShowPlayerDialog(playerid,49,DIALOG_STYLE_LIST,"Gun License","Buy Gun Licenses $1000\nGuns","Open","Close");
                }
            }
        }

dialogid 49 and 50

pawn Код:

if(listitem == 49)
        {
            if(response)
            {
                if(listitem == 0)
                {
                    if(PlayerData[playerid][GunLicense] == 1) return SCM(playerid,red,"ERROR: You already have gun licenses");
                    PlayerData[playerid][GunLicense] = 1;
                    SCM(playerid,COLOR_LIGHTBLUE,"You have bought gun licenses, select Guns under Buy Gun Licenses to get a gun everytime");
                    ShowPlayerDialog(playerid,47,DIALOG_STYLE_LIST,"City hall","Passport\nGun License","Open","Close");
                }
                if(listitem == 1)
                {
                    if(PlayerData[playerid][GunLicense] == 0) return SCM(playerid,red,"ERROR: You don't have gun licenses");
                    ShowPlayerDialog(playerid,50,DIALOG_STYLE_LIST,"Guns","Deagle $750\nShotgun $500","Buy","Close");
                }
            }
        }
        if(listitem == 50)
        {
            if(response)
            {
                if(listitem == 0)
                {
                    if(PlayerData[playerid][GunID] == 24) return SCM(playerid,red,"ERROR: You already have a deagle as a gun");
                    SCM(playerid,COLOR_LIGHTBLUE,"You will now spawn with a free deagle");
                    PlayerData[playerid][GunID] = 24;
                    GivePlayerWeapon(playerid,24,100);
                    ShowPlayerDialog(playerid,47,DIALOG_STYLE_LIST,"City hall","Passport\nGun License","Open","Close");
                }
                if(listitem == 1)
                {
                    if(PlayerData[playerid][GunID] == 25) return SCM(playerid,red,"ERROR: You already have a shotgun as a gun");
                    SCM(playerid,COLOR_LIGHTBLUE,"You will now spawn with a free shotgun");
                    PlayerData[playerid][GunID] = 25;
                    GivePlayerWeapon(playerid,25,100);
                    ShowPlayerDialog(playerid,47,DIALOG_STYLE_LIST,"City hall","Passport\nGun License","Open","Close");
                }
            }
        }

[ricardo178]

This should work.. Not tested.

pawn Код:

ShowPlayerDialog(playerid, 47, DIALOG_STYLE_LIST, "City hall", "\n1 Passport \n2 Gun License", "Open", "Close");

[Tanush123]

doesn't work still the same, well the problem is only with dialogid == 49

[ricardo178]

I am sure it will work now:

pawn Код:

if(listitem == 49)
        {
            if(response)
            {
                if(listitem == 0)
                {
                    if(PlayerData[playerid][GunLicense] == 1) return SCM(playerid,red,"ERROR: You already have gun licenses");
                    PlayerData[playerid][GunLicense] = 1;
                    SCM(playerid,COLOR_LIGHTBLUE,"You have bought gun licenses, select Guns under Buy Gun Licenses to get a gun everytime");
                    ShowPlayerDialog(playerid,47,DIALOG_STYLE_LIST,"City hall","Passport\nGun License","Open","Close");
                    return 1;
                }
                if(listitem == 1)
                {
                    if(PlayerData[playerid][GunLicense] == 0) return SCM(playerid,red,"ERROR: You don't have gun licenses");
                    ShowPlayerDialog(playerid,50,DIALOG_STYLE_LIST,"Guns","Deagle $750\nShotgun $500","Buy","Close");
                    return 1;
                }
            }
        }
        if(listitem == 50)
        {
            if(response)
            {
                if(listitem == 0)
                {
                    if(PlayerData[playerid][GunID] == 24) return SCM(playerid,red,"ERROR: You already have a deagle as a gun");
                    SCM(playerid,COLOR_LIGHTBLUE,"You will now spawn with a free deagle");
                    PlayerData[playerid][GunID] = 24;
                    GivePlayerWeapon(playerid,24,100);
                    ShowPlayerDialog(playerid,47,DIALOG_STYLE_LIST,"City hall","Passport\nGun License","Open","Close");
                    return 1;
                }
                if(listitem == 1)
                {
                    if(PlayerData[playerid][GunID] == 25) return SCM(playerid,red,"ERROR: You already have a shotgun as a gun");
                    SCM(playerid,COLOR_LIGHTBLUE,"You will now spawn with a free shotgun");
                    PlayerData[playerid][GunID] = 25;
                    GivePlayerWeapon(playerid,25,100);
                    ShowPlayerDialog(playerid,47,DIALOG_STYLE_LIST,"City hall","Passport\nGun License","Open","Close");
                    return 1;
                }
            }
        }

Some returns are needed.

[Tanush123]

Wow, it still doesn't work like seriously whats the problem...

[ricardo178]

Ok, made it with cases.. Much better.
Whole public:

pawn Код:

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    switch(dialogid)
    {
        case 49:
        {
            if(response)
            {
                if(PlayerData[playerid][GunID] == 24) return SCM(playerid,red,"ERROR: You already have a deagle as a gun");
                SCM(playerid,COLOR_LIGHTBLUE,"You will now spawn with a free deagle");
                PlayerData[playerid][GunID] = 24;
                GivePlayerWeapon(playerid,24,100);
                ShowPlayerDialog(playerid,47,DIALOG_STYLE_LIST,"City hall","Passport\n Gun License","Open","Close");
                return 1;
            }
        }
        case 50:
        {
            if(response)
            {
                if(PlayerData[playerid][GunID] == 25) return SCM(playerid,red,"ERROR: You already have a shotgun as a gun");
                SCM(playerid,COLOR_LIGHTBLUE,"You will now spawn with a free shotgun");
                PlayerData[playerid][GunID] = 25;
                GivePlayerWeapon(playerid,25,100);
                ShowPlayerDialog(playerid,47,DIALOG_STYLE_LIST,"City hall","Passport\n Gun License","Open","Close")
                return 1;
            }
        }
    }
    return 0;
}

[Tanush123]

dialogid 49 is not buying guns, dialog 50 is.

[ricardo178]

if(PlayerData[playerid][GunID] == 24)

If PlayerData[playerid][GunID] == 24) it will not work of-course.. Are you sure it isn't 24 when you try?

[Tanush123]

My GunID is 0, i just created Gun Licenses and GunID then i made a checkpoint and updated the city hall cp i made and for some reason Buy Gun Licenses and Guns won't do anything

[ricardo178]

Any Error/Warmning?