[Red_Dragon.]

Hello, i have a small problem with this command in my server, i tried to make it but here its define

pawn Код:

#define VSTORAGE 19104

and here is the command

pawn Код:

CMD:vstorage(playerid, params[])

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
if(dialogid == VSTORAGE 19104)
    new vstring[1024];
    for(new i, iModelID; i < MAX_PLAYERVEHICLES; i++) {
        if((iModelID = PlayerVehicleInfo[playerid][i][pvModelId] - 400) >= 0) {
            if(PlayerVehicleInfo[playerid][i][pvImpounded]) {
                format(vstring, sizeof(vstring), "%s\n%s (impounded)", vstring, VehicleName[iModelID]);
            }
            else if(PlayerVehicleInfo[playerid][i][pvDisabled]) {
                format(vstring, sizeof(vstring), "%s\n%s (disabled)", vstring, VehicleName[iModelID]);
            }
            else if(!PlayerVehicleInfo[playerid][i][pvSpawned]) {
                format(vstring, sizeof(vstring), "%s\n%s (stored)", vstring, VehicleName[iModelID]);
            }
            else format(vstring, sizeof(vstring), "%s\n%s (spawned)", vstring, VehicleName[iModelID]);
        }
        else strcat(vstring, "\nEmpty");
    }
    ShowPlayerDialog(playerid, VSTORAGE, DIALOG_STYLE_LIST, "Vehicle storage", vstring, "(De)spawn", "Cancel");
    return 1;
}

Where to put it if i want it in a filterscript not a gamemode so where to place it ? and if anyone see something wrong in that code please tell

[teomakedonija]

OnPlayerCommandText or OnPlayerCommandPerformed

[Faisal_khan]

There is nothing under the command. If its ZCMD, then you don't need to put it under any callbacks.

[Red_Dragon.]

Do you mean that the syntax is wrong ?

[thefatshizms]

As faisal said, there is nothing under the command ie the command is not finished. Can you show the rest of it? This surely cant be it as this would give errors.

[Faisal_khan]

The command should be something like this:

pawn Код:

CMD:vstorage(playerid, params[])
{
    //your code here
    return 1;
}

[Red_Dragon.]

pawn Код:

CMD:vstorage(playerid, params[])
{
    new vstring[1024];
    for(new i, iModelID; i < MAX_PLAYERVEHICLES; i++) {
        if((iModelID = PlayerVehicleInfo[playerid][i][pvModelId] - 400) >= 0) {
            if(PlayerVehicleInfo[playerid][i][pvImpounded]) {
                format(vstring, sizeof(vstring), "%s\n%s (impounded)", vstring, VehicleName[iModelID]);
            }
            else if(PlayerVehicleInfo[playerid][i][pvDisabled]) {
                format(vstring, sizeof(vstring), "%s\n%s (disabled)", vstring, VehicleName[iModelID]);
            }
            else if(!PlayerVehicleInfo[playerid][i][pvSpawned]) {
                format(vstring, sizeof(vstring), "%s\n%s (stored)", vstring, VehicleName[iModelID]);
            }
            else format(vstring, sizeof(vstring), "%s\n%s (spawned)", vstring, VehicleName[iModelID]);
        }
        else strcat(vstring, "\nEmpty");
    }
    ShowPlayerDialog(playerid, VSTORAGE, DIALOG_STYLE_LIST, "Vehicle storage", vstring, "(De)spawn", "Cancel");
    return 1;
}

Like this one ?

[thefatshizms]

EDIT:
dont worry

[Plovix]

Don't put it on any callback

[NumbSkull]

if(dialogid == VSTORAGE 19104) <---------
just ------------>if(dialogid == VSTORAGE)