[Trigger] Problems with trigger !

[operator]

For some reason, the game will not move on to the following actions after the first one in this trigger:

• Movie1
  • Events
  • Conditions
  • Actions
    • Unit Group - Pick every unit in CinemaPeople and do (Actions)
      • Loop - Actions
        • Camera - Apply Movie1 <gen> for (Owner of (Picked unit)) over 0.00 seconds
        • Wait 4.00 seconds
        • Cinematic - Send transmission to (All players) from No unit named <Empty String>: Play No sound and display It was just another.... Modify duration: Add 0.00 seconds and Wait
        • Wait 2.00 seconds
        • Camera - Apply Movie3 <gen> for (Owner of (Picked unit)) over 0.00 seconds
        • Cinematic - Send transmission to (All players) from No unit named <Empty String>: Play No sound and display An evil force was p.... Modify duration: Add 0.00 seconds and Wait
        • Wait 2.00 seconds
        • Camera - Apply Movie5 <gen> for (Owner of (Picked unit)) over 6.00 seconds
        • Wait 6.00 seconds
        • Camera - Apply Movie4 <gen> for (Owner of (Picked unit)) over 0.00 seconds
        • Camera - Shake the camera for (Owner of (Picked unit)) with magnitude 2.00
        • Wait 5.00 seconds
        • Set MovieLoc = (Target of Movie4 <gen>)
        • Sound - Play RiflemanAttack1 <gen> at 100.00% volume, located at MovieLoc with Z offset 0.00
        • Custom script: call RemoveLocation(udg_MovieLoc)
        • Camera - Stop swaying/shaking the camera for (Owner of (Picked unit))
        • Camera - Apply Movie2 <gen> for Player 1 (Red) over 0.40 seconds
        • Wait 0.30 seconds
        • Animation - Play Child 0023 <gen>'s death animation
        • Set MovieLoc = (Target of Movie6 <gen>)
        • Wait 1.50 seconds
        • Sound - Play RiflemanAttack1 <gen> at 100.00% volume, located at MovieLoc with Z offset 0.00
        • Animation - Play Child 0024 <gen>'s death animation
        • Wait 0.50 seconds
        • Sound - Play RiflemanAttack1 <gen> at 100.00% volume, located at MovieLoc with Z offset 0.00
        • Animation - Play Child (2) 0160 <gen>'s death animation
        • Custom script: call RemoveLocation(udg_MovieLoc)
        • Cinematic - Turn cinematic mode Off for (All players)

The very first action is working as it should, but it will not move on to the following ones. Why ?
I've tried to use both real time and game time seconds.

Thank you,

~Operator

 

[Huurka]

Wouldn't it be possible to add it in a playergroup and do the same?

The only thing I can come up with now.

 

[operator]

Nope.
Would take me ages to explain, but bottom line is that it has to remain a unitgroup.
Thank you for your time though

EDIT:
Sorry, I didn't get what you where saying at first. But when I did change it to operate with a playergroup, it didn't work either.

• Movie1
  • Events
  • Conditions
  • Actions
    • Player Group - Pick every player in Cinemagroup and do (Actions)
      • Loop - Actions
        • Camera - Apply Movie1 <gen> for (Picked player) over 0.00 seconds
        • Wait 4.00 seconds
        • Cinematic - Send transmission to (Player group((Picked player))) from No unit named <Empty String>: Play No sound and display It was just another.... Modify duration: Add 0.00 seconds and Wait
        • Wait 2.00 seconds
        • Camera - Apply Movie3 <gen> for (Picked player) over 0.00 seconds
        • Cinematic - Send transmission to (Player group((Picked player))) from No unit named <Empty String>: Play No sound and display An evil force was p.... Modify duration: Add 0.00 seconds and Wait
        • Wait 2.00 seconds
        • Camera - Apply Movie5 <gen> for (Picked player) over 6.00 seconds
        • Wait 6.00 seconds
        • Camera - Apply Movie4 <gen> for (Picked player) over 0.00 seconds
        • Camera - Shake the camera for (Picked player) with magnitude 2.00
        • Wait 5.00 seconds
        • Set MovieLoc = (Target of Movie4 <gen>)
        • Sound - Play RiflemanAttack1 <gen> at 100.00% volume, located at MovieLoc with Z offset 0.00
        • Custom script: call RemoveLocation(udg_MovieLoc)
        • Camera - Stop swaying/shaking the camera for (Picked player)
        • Camera - Apply Movie2 <gen> for (Picked player) over 0.40 seconds
        • Wait 0.30 seconds
        • Animation - Play Child 0023 <gen>'s death animation
        • Set MovieLoc = (Target of Movie6 <gen>)
        • Wait 1.50 seconds
        • Sound - Play RiflemanAttack1 <gen> at 100.00% volume, located at MovieLoc with Z offset 0.00
        • Animation - Play Child 0024 <gen>'s death animation
        • Wait 0.50 seconds
        • Sound - Play RiflemanAttack1 <gen> at 100.00% volume, located at MovieLoc with Z offset 0.00
        • Animation - Play Child (2) 0160 <gen>'s death animation
        • Custom script: call RemoveLocation(udg_MovieLoc)
        • Cinematic - Turn cinematic mode Off for (Player group((Picked player)))

 

[daxtreme]

You cannot insert waits inside a player group or unit group loop.

If you wish to know why I could explain it to you but it involves a little bit of JASS knowledge.

Anyhow, the trigger editor mentions it when you loop through these kind of groups.

 

[operator]

Damnit !
You see, I'm making a map where players will be able to go to the cinema and actually watch a cinematic. Any idea of how I can make it otherwise ?

_____

+rep, daxtreme

 

[Dr Super Good]

The problem is that it loops though them via a callback, waiting breaks the thread.
What you are after is threading it (or as close to WC3 lets you) or looping though each player for each action at a time and having the waits outside the loop (I would recomend this).

 

[daxtreme]

You can also loop with an integer variable and wait inside the loop. If this loop is the only time you are using the variable, it won't collide with anything else.

The correct action is: "For each (Integer variable), do multiple actions..."

 

[Squiggy]

Pick every player in CinemaPeople and do ...
Wait 4 seconds
Pick every player in CinemaPeople and do ...
Wait 80 seconds..
would be my solution