Trigger not working

[Manix]

CaptureFlags1
    Events
        Unit - A unit enters Region 006 <gen>
    Conditions
    Actions
        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            If - Conditions
                ((Triggering unit) has an item of type Leather Chest) Equal to True
            Then - Actions
                Set VillageCaptureUnit[1] = (Triggering unit)
                Unit - Pause VillageCaptureUnit[1]
                Countdown Timer - Start AssociateTimers[1] as a One-shot timer that will expire in 20.00 seconds
                Countdown Timer - Create a timer window for AssociateTimers[1] with title Capturing:
                Set AssociateTimersWindows[1] = (Last created timer window)
                Trigger - Turn off (This trigger)
            Else - Actions

CaptureFlags1Check
    Events
        Time - AssociateTimers[1] expires
    Conditions
    Actions
        Countdown Timer - Destroy AssociateTimersWindows[1]
        Set Village[1] = (Units within 200.00 of VillagePoint[1])
        Unit Group - Pick every unit in Village[1] and do (Actions)
            Loop - Actions
                If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                    If - Conditions
                        (Region 006 <gen> contains VillageCaptureUnit[1]) Equal to True
                    Then - Actions
                        Unit - Unpause VillageCaptureUnit[1]
                        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                            If - Conditions
                                (Owner of Circle of Power 0052 <gen>) Equal to Player 6 (Orange)
                                (VillageCaptureUnit[1] belongs to an ally of Player 5 (Yellow)) Equal to True
                            Then - Actions
                                Set AGoldGivenSyndicate = (AGoldGivenSyndicate - 1)
                                Set AGoldGivenAssociate = (AGoldGivenAssociate + 1)
                                Unit - Change ownership of Circle of Power 0052 <gen> to Player 5 (Yellow) and Change color
                            Else - Actions
                                If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                                    If - Conditions
                                        (VillageCaptureUnit[1] belongs to an ally of Player 5 (Yellow)) Equal to True
                                    Then - Actions
                                        Set AGoldGivenAssociate = (AGoldGivenAssociate + 1)
                                        Unit - Change ownership of Circle of Power 0052 <gen> to Player 5 (Yellow) and Change color
                                    Else - Actions
                        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                            If - Conditions
                                (Owner of Circle of Power 0052 <gen>) Equal to Player 5 (Yellow)
                                (VillageCaptureUnit[1] belongs to an ally of Player 6 (Orange)) Equal to True
                            Then - Actions
                                Set AGoldGivenSyndicate = (AGoldGivenSyndicate + 1)
                                Set AGoldGivenAssociate = (AGoldGivenAssociate - 1)
                                Unit - Change ownership of Circle of Power 0052 <gen> to Player 6 (Orange) and Change color
                            Else - Actions
                                If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                                    If - Conditions
                                        (VillageCaptureUnit[1] belongs to an ally of Player 6 (Orange)) Equal to True
                                    Then - Actions
                                        Set AGoldGivenSyndicate = (AGoldGivenSyndicate + 1)
                                        Unit - Change ownership of Circle of Power 0052 <gen> to Player 6 (Orange) and Change color
                                    Else - Actions
                    Else - Actions
        Custom script:   call DestroyGroup (udg_Village[1])
        Wait 1.00 seconds
        Trigger - Turn on CaptureFlags1 <gen>

I've set that VillagePoint[1] in an initialization trigger:

    Set Village[1] = (Units within 200.00 of VillagePoint[1])

But after 20 sec of "capturing" nothing happens :/

 

[Manix]

Ah i changed it so it changes the owner but it doesn't increase the gold given per second :/ here's the main trigger:

Free Gold
    Events
        Time - Every 1.00 seconds of game time
    Conditions
    Actions
        Set AGoldGivenAssociate = 1
        Set AGoldGivenSyndicate = 1
        Player Group - Pick every player in PlayersAssociate and do (Actions)
            Loop - Actions
                Player - Add AGoldGivenAssociate to (Picked player) Current gold
        Player Group - Pick every player in PlayersSyndicate and do (Actions)
            Loop - Actions
                Player - Add AGoldGivenSyndicate to (Picked player) Current gold

EDIT: Oh crap im setting the gold to 1 every 1 seconds... Sorry, it seems like i have to post a trigger here so i can see my mistakes xD

EDIT2: It works some times, sometimes not and i don't know why

 

[Kahiera]

                Countdown Timer - Start AssociateTimers[1] as a One-shot timer that will expire in 20.00 seconds
                Countdown Timer - Create a timer window for AssociateTimers[1] with title Capturing:

your starting a timer before even creating it yet

 

[martix]

Thats wrong... He is only creating a timer window. The timer - its declared in globals...
Question: What are you trying to do.

 

[Manix]

Im trying to create a place where a hero with a flag can go to, get paused for 20 sec and after the 20 sec if the hero is alive and carrying the flag the team captures the village and get gold given per second increased by 1

 

[The_Prophet]

You need to learn to use conditions. I'm not exactly sure what your doing with AGoldGivenSyndicate and AGoldGivenAssociate. Everything with a * next it it means i just wrote out the trigger, cause I did all this without my WE open. May come back later when I have more time and look your trigger over.

CaptureFlags1
    Events
        Unit - A unit enters Region 006 <gen>
    Conditions
        *Unit has leather chest == true*
    Actions
         Set VillageCaptureUnit[1] = (Triggering unit)
         Unit - Pause VillageCaptureUnit[1]
         Countdown Timer - Start AssociateTimers[1] as a One-shot timer that will expire in 20.00 seconds
         Countdown Timer - Create a timer window for AssociateTimers[1] with title Capturing:
         Set AssociateTimersWindows[1] = (Last created timer window)
         Trigger - Turn off (This trigger)
            
            

CaptureFlags1Check
    Events
        Time - AssociateTimers[1] expires
    Conditions
        *Check if VillageCaptureUnit[1] is alive*
    Actions
        Countdown Timer - Destroy AssociateTimersWindows[1]
        Unit - Unpause VillageCaptureUnit[1]        
        Set Village[1] = (Units within 200.00 of VillagePoint[1]) *Whats village Point 1?*
        
        Unit Group - Pick every unit in Village[1] and do (Actions)
            If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                    If - Conditions
                          (Owner of Circle of Power 0052 <gen>) Equal to Player 6 (Orange)
                          (VillageCaptureUnit[1] belongs to an ally of Player 5 (Yellow)) Equal to True
                    Then - Actions
                           Set AGoldGivenSyndicate = (AGoldGivenSyndicate - 1)
                           Set AGoldGivenAssociate = (AGoldGivenAssociate + 1)
                           Unit - Change ownership of Circle of Power 0052 <gen> to Player 5 (Yellow) and Change color
                    Else - Actions
                        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                                    If - Conditions
                                        (VillageCaptureUnit[1] belongs to an ally of Player 5 (Yellow)) Equal to True
                                    Then - Actions
                                        Set AGoldGivenAssociate = (AGoldGivenAssociate + 1)
                                        Unit - Change ownership of Circle of Power 0052 <gen> to Player 5 (Yellow) and Change color
                                    Else - Actions
                        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                            If - Conditions
                                (Owner of Circle of Power 0052 <gen>) Equal to Player 5 (Yellow)
                                (VillageCaptureUnit[1] belongs to an ally of Player 6 (Orange)) Equal to True
                            Then - Actions
                                Set AGoldGivenSyndicate = (AGoldGivenSyndicate + 1)
                                Set AGoldGivenAssociate = (AGoldGivenAssociate - 1)
                                Unit - Change ownership of Circle of Power 0052 <gen> to Player 6 (Orange) and Change color
                            Else - Actions
                                If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                                    If - Conditions
                                        (VillageCaptureUnit[1] belongs to an ally of Player 6 (Orange)) Equal to True
                                    Then - Actions
                                        Set AGoldGivenSyndicate = (AGoldGivenSyndicate + 1)
                                        Unit - Change ownership of Circle of Power 0052 <gen> to Player 6 (Orange) and Change color
                                    Else - Actions
                    Else - Actions
        Custom script:   call DestroyGroup (udg_Village[1])
        Wait 1.00 seconds
        Trigger - Turn on CaptureFlags1 <gen>

 

[HappyPeasant]

idk whats wrong but that trigger has a shit load of leaks. after you get it working think about cleaning it up cause after a few times of that trigger running theres going to be serious lag.


Edit: found 19 leaks in the original which are mostly around the circle of power and players 5 and 6

also your VillageCaptureUnit[1] and your countdown timer

 

[Manix]

As you said, first i need to get it to work, then i'll think about the leaks