[willsuckformoney]

Things are getting annoying now. I'm now getting dialog problems and I can't find a fix because the code looks fine. It will show the dialog, but when I select something listed it won't work. It will just close the dialog and change nothing. It don't call it as a response or no response. Anyone know a fix?

[BizzyD]

Post us your code, But check if your dialogid isnt disturbing any other dialog(look if a dialog has the same ID has the one you have)

Lets say you have dialogid 12. Press CTRL + F & search dialogid == 12

[willsuckformoney]

Only Dialog Used

pawn Код:

CMD:weather(playerid,params[])
{
    if(IsPlayerAdmin(playerid))
    {
        if(GetPVarInt(playerid,"WeatherView") == 0)
        {
            ShowPlayerDialog(playerid,9912,DIALOG_STYLE_LIST,
            "Select A Weather Type To Preview","Blue Skies \nStormy \nCloudy And Foggy \nClear Blue Sky \nHeatwave \nSandstorm \nFoggy, Greenish \nVery Dark, Gradiented Skyline, Purple \nPale Orange \nFresh Blue \nDark, Cloudy, Teal \nDark, Cloudy, Brown \nBlue/Purple, Regular \nDull Brown \nBright, Foggy, Orange \nVery Bright \nBlue/Purple, Cloudy \nToxic Clouds \nBlack/White Sky \nAmazing Draw Distance \nStormy Weather, Pink Sky, Crystal Water \nDarkest Weather Ever","View","Cancel");
            SendClientMessage(playerid,lightblue,"Weather Selection Menu: Select A Weather To View");
        } else return SendClientMessage(playerid,red,"You Are Already Viewing The Weather Selections");
    }
    return 1;
}

pawn Код:

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    if(dialogid == 9912)
    {
        if(!response) return SendClientMessage(playerid,red,"Weather Selection Menu Closed");
        switch(listitem)
        {
            case 0:
            {
                SetPlayerWeather(playerid,5);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",5);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 1:
            {
                SetPlayerWeather(playerid,8);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",8);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 2:
            {
                SetPlayerWeather(playerid,9);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",9);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 3:
            {
                SetPlayerWeather(playerid,10);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",10);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 4:
            {
                SetPlayerWeather(playerid,11);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",11);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 5:
            {
                SetPlayerWeather(playerid,13);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",13);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 6:
            {
                SetPlayerWeather(playerid,16);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",16);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 7:
            {
                SetPlayerWeather(playerid,18);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",18);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 8:
            {
                SetPlayerWeather(playerid,19);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",19);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 9:
            {
                SetPlayerWeather(playerid,20);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",20);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 10:
            {
                SetPlayerWeather(playerid,21);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",21);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 11:
            {
                SetPlayerWeather(playerid,22);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",22);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 12:
            {
                SetPlayerWeather(playerid,26);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",26);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 13:
            {
                SetPlayerWeather(playerid,29);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",29);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 14:
            {
                SetPlayerWeather(playerid,31);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",31);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 15:
            {
                SetPlayerWeather(playerid,33);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",33);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 16:
            {
                SetPlayerWeather(playerid,34);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",34);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 17:
            {
                SetPlayerWeather(playerid,35);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",35);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 18:
            {
                SetPlayerWeather(playerid,37);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",37);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 19:
            {
                SetPlayerWeather(playerid,39);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",39);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 20:
            {
                SetPlayerWeather(playerid,40);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",40);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 21:
            {
                SetPlayerWeather(playerid,43);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",43);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 22:
            {
                SetPlayerWeather(playerid,44);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",44);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 23:
            {
                SetPlayerWeather(playerid,52);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",50);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 24:
            {
                SetPlayerWeather(playerid,700);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",700);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
            case 25:
            {
                SetPlayerWeather(playerid,150);
                SetPVarInt(playerid,"WeatherView",1);
                SetPVarInt(playerid,"Weather",150);
                SendClientMessage(playerid,lightblue,"Press 'Shift' To Select. Press 'LMB' To Go Back");
            }
        }
    }
    return 1;
}

[=WoR=Varth]

What about putting else:

pawn Код:

if(!response) return SendClientMessage(playerid,red,"Weather Selection Menu Closed");
else
{

[dowster]

Quote:

Originally Posted by willsuckformoney Only Dialog Used pawn Код: CMD:weather(playerid,params[]) {     if(IsPlayerAdmin(playerid))     {         if(GetPVarInt(playerid,"WeatherView") == 0)         {             ShowPlayerDialog(playerid,9912,DIALOG_STYLE_LIST,             "Select A Weather Type To Preview","Blue Skies \nStormy \nCloudy And Foggy \nClear Blue Sky \nHeatwave \nSandstorm \nFoggy, Greenish \nVery Dark, Gradiented Skyline, Purple \nPale Orange \nFresh Blue \nDark, Cloudy, Teal \nDark, Cloudy, Brown \nBlue/Purple, Regular \nDull Brown \nBright, Foggy, Orange \nVery Bright \nBlue/Purple, Cloudy \nToxic Clouds \nBlack/White Sky \nAmazing Draw Distance \nStormy Weather, Pink Sky, Crystal Water \nDarkest Weather Ever","View","Cancel");             SendClientMessage(playerid,lightblue,"Weather Selection Menu: Select A Weather To View");         } else return SendClientMessage(playerid,red,"You Are Already Viewing The Weather Selections");     }     return 1; }

try having a \r before every \n, like this

pawn Код:

CMD:weather(playerid,params[])
{
    if(IsPlayerAdmin(playerid))
    {
        if(GetPVarInt(playerid,"WeatherView") == 0)
        {
            ShowPlayerDialog(playerid,9912,DIALOG_STYLE_LIST,
            "Select A Weather Type To Preview","Blue Skies \r\nStormy \r\nCloudy And Foggy \r\nClear Blue Sky \r\nHeatwave \r\nSandstorm \r\nFoggy, Greenish \r\nVery Dark, Gradiented Skyline, Purple \r\nPale Orange \r\nFresh Blue \r\nDark, Cloudy, Teal \r\nDark, Cloudy, Brown \r\nBlue/Purple, Regular \r\nDull Brown \r\nBright, Foggy, Orange \r\nVery Bright \r\nBlue/Purple, Cloudy \r\nToxic Clouds \r\nBlack/White Sky \r\nAmazing Draw Distance \r\nStormy Weather, Pink Sky, Crystal Water \r\nDarkest Weather Ever","View","Cancel");
            SendClientMessage(playerid,lightblue,"Weather Selection Menu: Select A Weather To View");
        } else return SendClientMessage(playerid,red,"You Are Already Viewing The Weather Selections");
    }
    return 1;
}

[Seven_of_Nine]

That's why I hate dialogs!
The code looks fine.

Maybe the dialogid is conflicted with some other? : S

[cessil]

this is how I do dialogs and I don't have any problems
http://forum.sa-mp.com/showpost.php?...95&postcount=3

for debugging yours add this

pawn Код:

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
printf("playerid: %d dialogid: %d response: %d listitem: %d inputtext: %s",playerid,dialogid,response,listitem,insputtext);