[JASS] A (most likely) Simple Question

[Colegreen]

This has no actions yet so ignor that part, but id like to know what i put for the second condition in the TriggerRegisterDialogEButtonEvent?

function Trig_Call_Inventory_Menu_Actions takes nothing returns nothing
endfunction

//===========================================================================
function InitTrig_Call_Inventory_Menu takes nothing returns nothing
    set gg_trg_Call_Inventory_Menu = CreateTrigger(  )
    call TriggerRegisterDialogEButtonEvent( gg_trg_Call_Inventory_Menu, )
    call TriggerAddAction( gg_trg_Call_Inventory_Menu, function Trig_Call_Inventory_Menu_Actions )
endfunction

It is originally from here

function Trig_Call_Menu_Actions takes nothing returns nothing
    call DialogClear( udg_ESC_Menu_Main )
    call DialogSetMessageBJ( udg_ESC_Menu_Main, "Main Menu" )
    call DialogAddButton( udg_ESC_Menu_Main, "Inventory", 1 )
    call DialogAddButton( udg_ESC_Menu_Main, "Change Menu", 2 )
    call DialogAddButton( udg_ESC_Menu_Main, "LogBook", 3 )
    call DialogAddButton( udg_ESC_Menu_Main, "Return", 4 )
    call DialogDisplayBJ( true, udg_ESC_Menu_Main, Player(0) )
endfunction

function InitTrig_Call_Menu takes nothing returns nothing
    set gg_trg_Call_Menu = CreateTrigger(  )
    call TriggerRegisterPlayerEventEndCinematic( gg_trg_Call_Menu, Player(0) )
    call TriggerAddAction( gg_trg_Call_Menu, function Trig_Call_Menu_Actions )
    set gg_trg_Call_Menu = null
endfunction

I am completely new to JASS, havnt made maps in a while, saw some tutorials, read em, and am trying to learn it. Any help will be appreciated.

 

[Homicide13]

well first of all im assuming you ment
"TriggerRegisterDialogButtonEvent"
instead of
"TriggerRegisterDialogEButtonEvent"

ok what this does is takes a specific button and returns an even, ie when it is clicked. assuming you have the dialog button as a global variable, you just put in the variable for the button that you want to trigger this spell with. If its a local variable, just use that one, but as far as i know you cannot make dialog buttons without a variable.

for example:

call TriggerRegisterDialogButtonEvent( gg_trg_Call_Inventory_Menu, udg_Chose_Orcs)

when a person clicks the button related to this variable, it will set off the trigger.

btw please use

tags, though i know Hive Workshop dosent have buttons at the top for them (or anything O_O)

 

[PurplePoot]

It used to be fine, pask. There's just a weird bug with JASS tags right now that makes them indent an extra line per line

 

[Colegreen]

Alright thanks for the help, i got The 2nd screen working now. I added some variable but i need to know how to get the condnition to work. It brings it up if any of the buttons on the first screen are clicked.

function Trig_Call_Inventory_Menu_Conditions takes nothing returns boolean
    if ( GetClickedButtonBJ() == udg_Main_Menu_Button_Inventory ) then
        return true
    endif
    return true
endfunction

//==========================================================================
function Trig_Call_Inventory_Menu_Actions takes nothing returns nothing
    call DialogClear( udg_ESC_Menu_Main )
    call DialogSetMessageBJ( udg_ESC_Inventory_Menu, "Inventory" )
    call DialogAddButton( udg_ESC_Inventory_Menu, "Store Items", 1 )
    set udg_Inv_Menu_Button_Store_Item = GetLastCreatedButtonBJ()
    call DialogAddButton( udg_ESC_Inventory_Menu, "Retrieve Items", 2 )
    call DialogAddButton( udg_ESC_Inventory_Menu, "Equip Items", 3 )
    call DialogAddButton( udg_ESC_Inventory_Menu, "Unequip Items", 4 )
    call DialogAddButton( udg_ESC_Inventory_Menu, "Return", 5 )
    call DialogDisplayBJ( true, udg_ESC_Inventory_Menu, Player(0) )
endfunction

//===========================================================================
function InitTrig_Call_Inventory_Menu takes nothing returns nothing
    set gg_trg_Call_Inventory_Menu = CreateTrigger(  )
    call TriggerRegisterDialogEvent( gg_trg_Call_Inventory_Menu, udg_ESC_Menu_Main )
    call TriggerAddCondition( gg_trg_Call_Inventory_Menu, Condition( function Trig_Call_Inventory_Menu_Conditions ) )
    call TriggerAddAction( gg_trg_Call_Inventory_Menu, function Trig_Call_Inventory_Menu_Actions )
    set gg_trg_Call_Inventory_Menu = null
endfunction

 

[PurplePoot]

function Trig_Call_Inventory_Menu_Conditions takes nothing returns boolean
if ( GetClickedButtonBJ() == udg_Main_Menu_Button_Inventory ) then
return true
endif
return true
endfunction

Please spare me ><. That condition type sucks in general, and the way you wrote it has a 100% chance to return true =O. Also, GetClickedButtonBJ() is equivelant to GetClickedButton(). Use the following;

function Trig_Call_Inventory_Menu_Conditions takes nothing returns boolean
    return GetClickedButton() == udg_Main_Menu_Button_Inventory
endfunction

 

[Colegreen]

I rewrote the condition like you said and it dosen't appear to ever bring back true now, do i have to change anything else?

Thxs for the help tho.

 

[PurplePoot]

If it's never bringing back true, you're never clicking that button.

The reason yours would return true always was because both returns were 'return true'.

 

[Homicide13]

>_> im assuming youv set udg_Main_Menu_Button_Inventory to something, right?

and instead of:

call DialogAddButton( udg_ESC_Inventory_Menu, "Store Items", 1 )
set udg_Inv_Menu_Button_Store_Item = GetLastCreatedButtonBJ()

try something like this:

call DialogAddButton ( udg_Inv_Menu_Button_Store_Item, "Store Items", 1 )

gl with this ^^

 

[SentryIII]

I'm pretty sure he meant this:

set udg_Inv_Menu_Button_Store_Item = DialogAddButton( udg_ESC_Inventory_Menu, "Store Items", 1 )

 

[Homicide13]

err, yea, O_O
lol im not to great with dialogs