Duel Problem

[Marc Mamales]

Hey guys i need help again!

• Duel Timers
  • Events
    • Time - Elapsed game time is 1.00 seconds
  • Conditions
  • Actions
    • -------- Start the timer --------
    • Countdown Timer - Start Duel_Timer as a One-shot timer that will expire in 10.00 seconds
    • -------- Create a window --------
    • Countdown Timer - Create a timer window for Duel_Timer with title Duel in
    • Countdown Timer - Show (Last created timer window)
    • -------- Change color and transparency --------
    • Countdown Timer - Change the color of the title for (Last created timer window) to (0.00%, 60.00%, 70.00%) with 40.00% transparency

• Duel Start
  • Events
    • Time - Duel_Timer expires
  • Conditions
  • Actions
    • -------- Setting variables so that memory leaks can be removed at the end of this trigger --------
    • Set Duel_Unit[1] = (Random 1 units from (Units owned by (Random player from (All allies of Player 1 (Red))) matching (((Matching unit) is A Hero) Equal to True)))
    • Set Duel_Unit[2] = (Random 1 units from (Units owned by (Random player from (All allies of Player 7 (Green))) matching (((Matching unit) is A Hero) Equal to True)))
    • Set Duel_Point_Leak[1] = (Center of Arena Team 1 Teleport <gen>)
    • Set Duel_Point_Leak[2] = (Center of Arena Team 2 Teleport <gen>)
    • Set Duel_Point_Leak[3] = (Center of Hero Teleport When Pick T1 <gen>)
    • Set Duel_Point_Leak[4] = (Center of Hero Teleport When Pick T2 <gen>)
    • Unit Group - Pick every unit in Duel_Unit[2] and do (Actions)
      • Loop - Actions
        • -------- Be sure to check where i put the Duel Region --------
        • -------- Check if the unit is belongs to which team --------
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • ((Picked unit) belongs to an ally of Player 7 (Green)) Equal to True
          • Then - Actions
            • Unit - Move (Picked unit) instantly to Duel_Point_Leak[2]
            • -------- add unit to group --------
            • Unit Group - Add (Picked unit) to Duel_Group[2]
          • Else - Actions
    • -------- Pick the unit that will Duel --------
    • Unit Group - Pick every unit in Duel_Unit[1] and do (Actions)
      • Loop - Actions
        • -------- Be sure to check where i put the Duel Region --------
        • -------- Check if the unit is belongs to which team --------
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • ((Picked unit) belongs to an ally of Player 1 (Red)) Equal to True
          • Then - Actions
            • Unit - Move (Picked unit) instantly to Duel_Point_Leak[1]
            • -------- add unit to group --------
            • Unit Group - Add (Picked unit) to Duel_Group[1]
          • Else - Actions
    • Wait 5.00 seconds
    • Trigger - Turn on Duel End <gen>
    • Custom script: call RemoveLocation( udg_Duel_Point_Leak[1] )
    • Custom script: call RemoveLocation( udg_Duel_Point_Leak[2] )
    • Custom script: call DestroyGroup( udg_Duel_Unit[1] )
    • Custom script: call DestroyGroup( udg_Duel_Unit[2] )

• Duel Hero
  • Events
    • Unit - A unit Dies
  • Conditions
    • ((Dying unit) is A Hero) Equal to True
    • (Arena <gen> contains (Dying unit)) Equal to True
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • ((Dying unit) belongs to an ally of Player 1 (Red)) Equal to True
      • Then - Actions
        • Unit Group - Remove (Dying unit) from Duel_Group[1]
      • Else - Actions
        • Unit Group - Remove (Dying unit) from Duel_Group[2]

• Duel End
  • Events
    • Time - Every 5.00 seconds of game time
  • Conditions
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Number of units in Duel_Group[2]) Equal to 0
      • Then - Actions
        • Quest - Display to (All players) the Quest Completed message: Team 1 wins the duel
        • Wait 2.00 seconds
        • Unit Group - Pick every unit in Duel_Group[1] and do (Actions)
          • Loop - Actions
            • Unit - Move (Picked unit) instantly to Duel_Point_Leak[3]
        • Custom script: call RemoveLocation(udg_Duel_Point_Leak[3])
        • Unit Group - Remove all units from Duel_Group[1]
        • Unit Group - Remove all units from Duel_Group[2]
        • -------- countdown timer can be changed --------
        • Countdown Timer - Start Duel_Timer as a One-shot timer that will expire in 10.00 seconds
        • Trigger - Turn off (This trigger)
      • Else - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Number of units in Duel_Group[1]) Equal to 0
          • Then - Actions
            • Quest - Display to (All players) the Quest Completed message: Team 2 wins the duel
            • Wait 2.00 seconds
            • Unit Group - Pick every unit in Duel_Group[2] and do (Actions)
              • Loop - Actions
                • Unit - Move (Picked unit) instantly to Duel_Point_Leak[4]
            • Custom script: call RemoveLocation(udg_Duel_Point_Leak[4])
            • Unit Group - Remove all units from Duel_Group[1]
            • Unit Group - Remove all units from Duel_Group[2]
            • -------- countdown timer can be changed --------
            • Countdown Timer - Start Duel_Timer as a One-shot timer that will expire in 10.00 seconds
            • Trigger - Turn off (This trigger)
          • Else - Actions
            • Do nothing

when my trigger runs it ends automatically without killing the enemy hero, idk what really is the problem because im testing it with computers...

 

[Maker]

Use Game - Display text message to show the name of DuelUnit[1] and DuelUnit[2]. Also display the number of units in DuelUnits[1] and DuelUnits[2]. Do these in Duel Start. right before the wait.

You have a leak with these lines:

Trig

• Set Duel_Unit[1] = (Random 1 units from (Units owned by (Random player from (All allies of Player 1 (Red))) matching (((Matching unit) is A Hero) Equal to True)))

Random 1 units is a unit group, units owned by x is another.

 

[GIANT_CRAB]

• Actions
  • Custom script: call DestroyGroup(udg_Duel_Unit[1])
  • Custom script: call DestroyGroup(udg_Duel_Unit[2])

Very funneh, when you destroy the group, group will not exist and theres no units in it.

 

[Marc Mamales]

Use Game - Display text message to show the name of DuelUnit[1] and DuelUnit[2]. Also display the number of units in DuelUnits[1] and DuelUnits[2]. Do these in Duel Start. right before the wait.

You have a leak with these lines:

Trig

• Set Duel_Unit[1] = (Random 1 units from (Units owned by (Random player from (All allies of Player 1 (Red))) matching (((Matching unit) is A Hero) Equal to True)))

Random 1 units is a unit group, units owned by x is another.

i dont really get it, can you explain it to me?

 

[GIANT_CRAB]

Clear the leaks of the unit group at the Duel End trigger

 

[Marc Mamales]

Clear the leaks of the unit group at the Duel End trigger

wait, i'll give you the map.