Why won't GetClickedButton work?

[storyyeller]

I've been trying to add a vote system, but it won't work. The trigger executes just fine, but it can't detect which button was pressed because
GetClickedButton() always returns null for some reason. Does anyone know what is going on?

Here is what my code looks like

Code: Select all

  set mydialog = DialogCreate()
  call DialogSetMessage(mydialog,"You now have 30 seconds\n to vote on game modes")
  set mybuttons[0]=DialogAddButton(mydialog,"Don't Care",13)
  set mybuttons[1]=DialogAddButton(mydialog,"Normal Mode",78)
  set mybuttons[2]=DialogAddButton(mydialog,"Nuke Mode",75)
  set mybuttons[3]=DialogAddButton(mydialog,"Space Mode",83)
  call TriggerRegisterDialogButtonEvent( TrigDialogClick, mybuttons[0])
  call TriggerRegisterDialogButtonEvent( TrigDialogClick, mybuttons[1])
  call TriggerRegisterDialogButtonEvent( TrigDialogClick, mybuttons[2])
  call TriggerRegisterDialogButtonEvent( TrigDialogClick, mybuttons[3])
  
  loop
    exitwhen i>5
    call DialogDisplay( Player(i), mydialog, true)
    set i=i+1
  endloop

Code: Select all

function funcDialog takes nothing returns nothing
    local integer i=0
    call DisplayTextToPlayer(GetLocalPlayer(),0,0,"A dialog was clicked")
    loop
        exitwhen i>3                 
        if GetClickedButton() == mybuttons[i] then
            call DisplayTextToPlayer(GetLocalPlayer(),0,0,"Button " + I2S(i) + " was selected")  
        endif
        set i=i+1
    endloop
endfunction

[Ken]

Code: Select all

  loop
    exitwhen i>5
    call DialogDisplay( Player(i), mydialog, true)
    set i=i+1
  endloopn

endloopn?

[storyyeller]

That's not in the actual map. (Obviously if it was, it wouldn't run at all)

I must have accidentally added that while pasting my code in here

[Ken]

I'm going to assume you do have a 'call TriggerAddAction' for this, right?

That's all I can see that would possibly be a problem... Or maybe changing the GetLocalPlayer() to GetTriggerPlayer().

[storyyeller]

Yeah I have all that. You can also safely assume that I declared all my variables and stuff like that. Like I said, the trigger runs perfectly.

The problem is that GetClickedButton() is returning NULL for some reason

[Ken]

This is a guess based on looking at a map...

Code: Select all

call DialogAddButton(mydialog,"Don't Care",13)
set mybuttons[0]=GetLastCreatedButtonBJ()

And do that for each of the buttons. Or try TriggerRegisterDialogEvent.

[storyyeller]

This is a guess based on looking at a map...

Code: Select all

call DialogAddButton(mydialog,"Don't Care",13)
set mybuttons[0]=GetLastCreatedButtonBJ()

And do that for each of the buttons. Or try TriggerRegisterDialogEvent.

Actually, that wouldn't work because DialogAddButton does not set bj_lastCreatedButton, so mybuttons[] would end up as null.
You were probably thinking of DialogAddButtonBJ, in which case, you would omit the last argument

Anyway, that is just a more complicated and inefficient way of doing the same thing my code does already.


As for TriggerRegisterDialogEvent, I already tried that

[Risen]

Actually, I kinda wrote a dialog library a few mins ago, Here's the pastebin for it, It works completely so just rename it or w/e.
http://pastebin.com/d2de352a5
http://pastebin.com/d5abeaa3a

[storyyeller]

I tried your code, and it didn't work either
Not terribly surprising, since it is very similar to what I wrote

As far as I can tell, neither TriggerRegisterDialogEvent, nor GetButtonClicked are working

Edit: I attached the war3map.j file in case it helps

[Ozzapoo]

Show us your TriggerReisterEventButtonClicked line.