[Trigger] What's wrong with this?

[Dinodin]

I'm trying to make a timer that shows long time it is for the player Hero to revive but the timer window is shown for all other players too and I want it to only be shown for the player with the dying Hero. I can't see what's wrong. Plz help!

• Events
  • Unit - A unit owned by Player 1 (Red) Dies
• Conditions
  • ((Triggering unit) is A Hero) Equal to True
• Actions
  • Game - Display to Player Group - Player 1 (Red) the text: DON´T LEAVE! Revi...
  • Countdown Timer - Start death[1] as a One-shot timer that will expire in 50.00 seconds
  • Countdown Timer - Create a timer window for (Last started timer) with title Reviving Hero in...
  • Set deathtimer[1] = (Last created timer window)
  • Countdown Timer - Show deathtimer[1] for Player 1 (Red)
  • Wait 50.00 seconds
  • Countdown Timer - Destroy deathtimer[1]
  • Hero - Instantly revive (Triggering unit) at (Center of Revive Hero Good <gen>), Show revival graphics
  • Camera - Pan camera for Player 1 (Red) to (Center of Revive Hero Good <gen>) over 0.50 seconds

 

[Arnadath]

• For each (Integer A) from 1 to 12, do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Integer A) Not equal to (Player number of (Owner of (Triggering unit)))
      • Then - Actions
        • Countdown Timer - Hide (Last created timer window) for (Player((Integer A)))
      • Else - Actions

Might need to modify it

also if it works, click on my sig for a free cookie

 

[apocalypse_dude]

ok first this will not work if you have more then one hero

ok and now to your problem
you need tho hide the countdown timer for all players and then afther hiding you show it just to owner of dying unit

 

[Zakamutt]

You might be able to use GetLocalPlayer() to do it to just one player straight away - I don't know if this would cause a desync or not, so beware! Might be more efficient than other ways.

 

[Arnadath]

GetLocalPlayer() is jass and i hate it. Anyway hiding the timer for every one else except the triggering player, is ok way

Also click down to have a cookie

 

[Diehard@Azeroth]

i can see the link xD

 

[Noobsmoke]

Action "Wait 50 secounds" leaks ... try this
PS:i don't test it ...

 

[Deuterium]

the best way is to use GetLocalPlayer()...
i could show you how but im on mac now so i don't have warcraft... maybe once I'm back home in case no one raced me to it