Problem with Dialog Boxes that gives rewards

[ROBBYDOBBY]

Hey,
I'm having trouble with my Dialog Boxes at the moment.
I've been following AceHart's helpful guide to Dialog Boxes and I've managed to create a dialog box to choose what item you want for helping the soldier.
When you click on an item it goes to another short cinematic, then should give the item to the player. BUT it doesn't Give the item to the player, and now im very confused.
Thanks in advance,
Robbydobby
Trigger: (I've only included one of the rewards in the trigger because the triggers aren't different.)

Reclaiming Milda post 5
    Events
        Unit - A unit enters Quest Range Mildas post 1 <gen>
    Conditions
        ((Owner of (Entering unit)) controller) Equal to User
    Actions
        Special Effect - Destroy Reclaiming_Milda_Post
        Cinematic - Turn cinematic mode On for (All players)
        Camera - Pan camera for Player 1 (Red) to (Position of Captain 0008 <gen>) over 0.00 seconds
        Cinematic - Send transmission to (Player group((Owner of (Entering unit)))) from (Entering unit) named (Name of (Owner of (Entering unit))): Play No sound and display I've done it. Ive k....  Modify duration: Add 0.00 seconds and Wait
        Cinematic - Send transmission to (Player group((Owner of (Entering unit)))) from Captain 0008 <gen> named Captain: Play No sound and display Thanks! You are in ....  Modify duration: Set to 10.00 seconds and Wait
        Cinematic - Turn cinematic mode Off for (Player group((Owner of (Entering unit))))
        Dialog - Clear Reclaiming_MP_Box
        Dialog - Change the title of Reclaiming_MP_Box to Choose your reward!
        Dialog - Create a dialog button for Reclaiming_MP_Box labelled Club
        Set Reclaiming_MP_Button1 = (Last created dialog Button)
        Dialog - Create a dialog button for Reclaiming_MP_Box labelled Staff
        Set Reclaiming_MP_Button2 = (Last created dialog Button)
        Dialog - Create a dialog button for Reclaiming_MP_Box labelled Dagger
        Set Reclaiming_MP_Button3 = (Last created dialog Button)
        Dialog - Create a dialog button for Reclaiming_MP_Box labelled Sword
        Set Reclaiming_MP_Button_4 = (Last created dialog Button)
        Dialog - Show Reclaiming_MP_Box for (Owner of (Entering unit))
        Trigger - Turn off (This trigger)
 RMP 1
    Events
        Dialog - A dialog button is clicked for Reclaiming_MP_Box
    Conditions
        (Clicked dialog button) Equal to Reclaiming_MP_Button1
    Actions
        Hero - Create Battered Club and give it to (Triggering unit)
        Hero - Add 400 experience to (Triggering unit), Show level-up graphics
        Game - Display to (Player group((Owner of (Triggering unit)))) the text: |cff25525540Congrat...
        Cinematic - Turn cinematic mode On for (All players)
        Cinematic - Send transmission to (Player group((Triggering player))) from Captain 0008 <gen> named Soldier:: Play No sound and display  Oh, you want that?....  Modify duration: Set to 9.00 seconds and Wait
        Cinematic - Turn cinematic mode Off for (All players)

 

[Menelaus]

Actions
Hero - Create Battered Club and give it to (Triggering unit)
Hero - Add 400 experience to (Triggering unit), Show level-up graphics
Game - Display to (Player group((Owner of (Triggering unit)))) the text: |cff25525540Congrat...
Cinematic - Turn cinematic mode On for (All players)
Cinematic - Send transmission to (Player group((Triggering player))) from Captain 0008 <gen> named Soldier:: Play No sound and display Oh, you want that?.... Modify duration: Set to 9.00 seconds and Wait
Cinematic - Turn cinematic mode Off for (All players)[/CODE]

theres no unit that clicked a dialog button

Solution (assuming each player only has one hero on the map, if not i would upload ur map so people can see the situation)

Unit Group - Pick every unit in (Units owned by (Triggering player) matching (((Picked unit) is A Hero) Equal to True)) and do (Actions)
    Loop - Actions
        Hero - Give (Last created item) to (Picked unit)

 

[ROBBYDOBBY]

Thanks i was thinking that was the problem
Argh, I've just tried it but it wont give the item!!! I'll upload the map :

 

[ROBBYDOBBY]

Sorry about double post, but here it is!

 

[Drunken_God]

you can use the EDIT Button
and then push "go advanced"

Solution (assuming each player only has one hero on the map, if not i would upload ur map so people can see the situation)
Code:

Unit Group - Pick every unit in (Units owned by (Triggering player) matching (((Picked unit)

Spoiler

Replace this with matching unit

is A Hero) Equal to True)) and do (Actions)
Loop - Actions
Hero - Give (Last created item) to (Picked unit)

there is a mistake ( a very large one)

 

[ROBBYDOBBY]

Yay, It worked!!!!
Thank you!

 

[Menelaus]

lol sry that was a typo