[Trigger] How to remove a player's pre-placed units if player is not playing?

[Lightstalker]

This is my current trigger that occurs at map ini. I've also tried many different versions of it, but nothing has worked so far.

• Player Group - Pick every player in (All players) and do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • ((Picked player) slot status) Not equal to Is playing
      • Then - Actions
        • Set NotPlaying_Group = (Units owned by (Picked player))
        • Unit Group - Pick every unit in NotPlaying_Group and do (Actions)
          • Loop - Actions
            • Unit - Remove (Picked unit) from the game
        • Custom script: call DestroyGroup(udg_NotPlaying_Group)
      • Else - Actions
        • Player - Limit training of Heroes to 1 for (Picked player)
        • Player - Set (Picked player) Current gold to 1500
        • Player - Turn Gives bounty On for (Picked player)
        • Hero - Make (Picked player) Heroes gain 85.00% experience from future kills

 

[Eleandor]

Try: elapsed game time = 0.5 seconds?

 

[xXMephistoXx]

Well, if you have 4 players it would be like this:

• Player Init
  • Events
    • Map initialization
  • Conditions
  • Actions
    • For each (Integer A) from 1 to 4, do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • ((Player((Integer A))) slot status) Not equal to Is playing
          • Then - Actions
            • Unit Group - Pick every unit in (Units owned by (Player((Integer A)))) and do (Actions)
              • Loop - Actions
                • Unit - Remove (Picked unit) from the game
          • Else - Actions
            • Player - Limit training of Heroes to 1 for (Player((Integer A)))
            • Player - Set (Player((Integer A))) Current gold to 1500
            • Player - Turn Gives bounty Off for (Player((Integer A)))
            • Hero - Make (Player((Integer A))) Heroes gain 85.00% experience from future kills

I think it should work. Just replace "4" with however many players you would like to have and add or edit the information to suit your map.
Although it might just be easier to add the units via a trigger, based on whether the player is playing or not and remove the "Create starting units for player" trigger..

 

[Lord_BoNes]

Wait until you've finished looping though all the players before destroying the units, put all the units in NotPlaying_Group into another group, then destroy NotPlaying_Group.

After you've looped through the players, all the "non-playing" units should be in one group... DESTROY THEM, then DESTROY THE GROUP!!

Sorry, got a little carried away there

 

[idodik]

Trigger Correction

• Player Group - Pick every player in (Players matching (Matching Player is playing) equal to True) and do (Actions)
  • Loop - Actions
    • Set NotPlaying_Group = (Units owned by (Picked player))
    • Unit Group - Pick every unit in NotPlaying_Group and do (Actions)
      • Loop - Actions
        • Unit - Remove (Picked unit) from the
    • Custom script: call DestroyGroup(udg_NotPlaying_Group)

Corrected your trigger.
Make one for your filled player slots, you should.
May the Force be with you!
EDIT: Mephisto's solution is better. The Force is strong with you!