Odd raiding bug

[XXXconanXXX]

The system is like so:

You go into a pedestal to capture the expansion, your paused along with the base, then your un-paused. Thing is, during certain games and at random times, the bases will bug out and won't pause or switch over. The Tidal Guardian that guards it, is though. So, I'm thinking it has to do with the picking of the units. Any thoughts on why this isn't working?

Expansion base Human Middle
    Events
        Unit - A unit enters Expansion Human Middle Raid <gen>
    Conditions
        ((Entering unit) is A Hero) Equal to True
    Actions
        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            If - Conditions
                Or - Any (Conditions) are true
                    Conditions
                        (Owner of (Entering unit)) Equal to Player 1 (Red)
                        (Owner of (Entering unit)) Equal to Player 2 (Blue)
                        (Owner of (Entering unit)) Equal to Player 3 (Teal)
                        (Owner of (Entering unit)) Equal to Player 4 (Purple)
                        (Owner of (Entering unit)) Equal to Player 5 (Yellow)
                        (Owner of (Entering unit)) Equal to Player 6 (Orange)
            Then - Actions
                Unit Group - Pick every unit in ExpBase_UnitGroup_HM and do (Actions)
                    Loop - Actions
                        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                            If - Conditions
                                (Owner of (Picked unit)) Equal to Player 6 (Orange)
                            Then - Actions
                                Skip remaining actions
                            Else - Actions
                Unit Group - Pick every unit in ExpBase_UnitGroup_HM and do (Unit - Pause (Picked unit))
                Unit - Pause (Entering unit)
                Game - Display to (All players) for 3.00 seconds the text: |cffff0000Control o...
                Wait 2.00 seconds
                Game - Display to (All players) for 1.00 seconds the text: |cffff00003...|r
                Sound - Play BattleNetTick <gen>
                Wait 1.00 seconds
                Game - Display to (All players) for 1.00 seconds the text: |cffff00002...|r
                Sound - Play BattleNetTick <gen>
                Wait 1.00 seconds
                Game - Display to (All players) for 1.00 seconds the text: |cffff00001...|r
                Sound - Play BattleNetTick <gen>
                Wait 1.00 seconds
                Game - Display to (All players) for 3.00 seconds the text: |cffff0000Control o...
                Sound - Play GoodJob <gen>
                Unit - Unpause (Entering unit)
                Unit - Change ownership of Tidal Guardian 0007 <gen> to Player 6 (Orange) and Change color
                Unit Group - Pick every unit in ExpBase_UnitGroup_HM and do (Unit - Unpause (Picked unit))
                Unit Group - Pick every unit in ExpBase_UnitGroup_HM and do (Unit - Change ownership of (Picked unit) to Player 6 (Orange) and Change color)
            Else - Actions
        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            If - Conditions
                Or - Any (Conditions) are true
                    Conditions
                        (Owner of (Entering unit)) Equal to Player 7 (Green)
                        (Owner of (Entering unit)) Equal to Player 8 (Pink)
                        (Owner of (Entering unit)) Equal to Player 9 (Gray)
                        (Owner of (Entering unit)) Equal to Player 10 (Light Blue)
                        (Owner of (Entering unit)) Equal to Player 11 (Dark Green)
                        (Owner of (Entering unit)) Equal to Player 12 (Brown)
            Then - Actions
                Unit Group - Pick every unit in ExpBase_UnitGroup_HM and do (Actions)
                    Loop - Actions
                        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                            If - Conditions
                                (Owner of (Picked unit)) Equal to Player 12 (Brown)
                            Then - Actions
                                Skip remaining actions
                            Else - Actions
                Unit Group - Pick every unit in ExpBase_UnitGroup_HM and do (Unit - Pause (Picked unit))
                Unit - Pause (Entering unit)
                Game - Display to (All players) for 3.00 seconds the text: |cffff0000Control o...
                Wait 2.00 seconds
                Game - Display to (All players) for 1.00 seconds the text: |cffff00003...|r
                Sound - Play BattleNetTick <gen>
                Wait 1.00 seconds
                Game - Display to (All players) for 1.00 seconds the text: |cffff00002...|r
                Sound - Play BattleNetTick <gen>
                Wait 1.00 seconds
                Game - Display to (All players) for 1.00 seconds the text: |cffff00001...|r
                Sound - Play BattleNetTick <gen>
                Wait 1.00 seconds
                Game - Display to (All players) for 3.00 seconds the text: Red: |cffff0000Cont...
                Sound - Play GoodJob <gen>
                Unit - Unpause (Entering unit)
                Unit - Change ownership of Tidal Guardian 0007 <gen> to Player 12 (Brown) and Change color
                Unit Group - Pick every unit in ExpBase_UnitGroup_HM and do (Unit - Unpause (Picked unit))
                Unit Group - Pick every unit in ExpBase_UnitGroup_HM and do (Unit - Change ownership of (Picked unit) to Player 12 (Brown) and Change color)
            Else - Actions

 

[Shadowy Fear]

~In an unrelated-to-the-current-problem analysis, you will need to change your sound - play good job or whatever it was to be a 3-D sound; otherwise, the people will never here it (I never did, at least ).

Yes, I believe it is related to the pick every unit thing. You might want to double check that, and make sure that no other codes have a chance to overlap and override that information. Also, check the replays to see if you can determine when and why this problem occurs.

Occasionally, 2 people, sometimes on the same team and sometimes not, will step on the CoP at the same time (pretty much) and the clock will start a dual countdown. Just thought I'd mention this, too

You might just want to pick every unit in area matching - is a building. It would be more redundant, but might stop the problem from occuring. :>)

 

[Darg]

I guess I can only say what's obvious - your ExpBase_UnitGroup_HM variable is being changed or cleared or something like that.

What you should do is add the following test trigger action in:

Unit Group - Pick every unit in ExpBase_UnitGroup_HM and do
Cinematic - Flash a speech indicator for (Picked unit) of color (100.00%, 100.00%, 100.00%) with 0.00% transparency

So when that trigger of yours executes, you will see all the units in the unit group have white circles flash around them. So, if the theory holds correct, the next time units don't switch over, you will see no units with white flashing circles.

 

[Shadowy Fear]

Eh, that is good, Darg, but it is redundant, as the units in question are TOWERS, with big long color strips down the sides, and a sizable portion of the minimap . It is pretty easy to tell which ones have and haven't converted over, rofl

 

[XXXconanXXX]

His point is to see if the unit group is holding the correct units or not, and thank you Darg, I'll test it out in a second.

 

[SilverHawk]

Well, I'm not sure if this is your problem, but if you have nested pick every unit loops, you need to store the outer one into a variable; only the innermost one will be referred to as picked unit.

Just general info.

 

[NRinpun]

And I thought I'd suggest less lines for the condition.

(Owner of Unit is ally of (Human Side CPU)) and (Owner of Unit is not Neutral Passive)

instead of your long lines

You can also make a Player variable for whoever controls the base. Set the Player equal to Neutral Hostile at first and then to either the Human or Undead CPU Player as they go for control of the base. Then, just have (Owner of Unit is enemy of (Owner of Base variable)).

 

[XXXconanXXX]

Saving maybe a few bytes of memory is not worth it for every part I have to change.

Let's just leave it at this, since there's no point in doing it if it doesn't make a significant change.

(And yes, I know it leaks pretty badly...)