need help

[Unholy_Vendetta]

• Event Finish
  • Events
    • Unit - A unit Dies
  • Conditions
    • (String((Player number of (Owner of (Dying unit))))) Equal to String_Loc
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • ((Killing unit) is A Hero) Equal to True
      • Then - Actions
        • Player - Add 200 to (Owner of (Killing unit)) Current gold
        • Game - Display to (All players) the text: (PlayerName[(Player number of (Owner of (Killing unit)))] + has win the event.)
        • Quest - Destroy Quest
        • Trigger - Turn on Event <gen>
        • Trigger - Turn off (This trigger)
      • Else - Actions

why my trigger not working?

 

[Maker]

You could display the string loc and the player number strings separately for debugging purposes.

 

[Unholy_Vendetta]

• Actions
  • Set PlayerName[1] = (|CFFFF0303 + ((Name of Player 1 (Red)) + |R))
  • Set PlayerName[2] = (|CFF0042FF + ((Name of Player 2 (Blue)) + |R))
  • Set PlayerName[3] = (|CFF1BE6D8 + ((Name of Player 3 (Teal)) + |R))
  • Set PlayerName[4] = (|CFF540081 + ((Name of Player 4 (Purple)) + |R))
  • Set String_Loc = PlayerName[(Random integer number between 1 and 4)]

srry, here you go

 

[Chaosy]

try this instead.

• Melee Initialization
  • Events
    • Unit - A unit Dies
  • Conditions
    • (Player number of (Owner of (Triggering unit))) Equal to (Player number of (Player((Random integer number between 1 and 4))))
  • Actions
    • -------- do your stuff --------

 

[Unholy_Vendetta]

its work, thx u so much +rep

 

[Dr Super Good]

Game - Display to (All players) the text: (PlayerName[(Player number of (Owner of (Killing unit)))] + has win the event.)

Somebody set us up the bomb!

I mean the grammar is incorrect.
Wrong: DrSuperGood has win the event.
Correct: DrSuperGood won the event.
Alternative: DrSuperGood is the winner of the event.

 

[Meticulous]

FYI, why the original doesn't work:

• Conditions
  • (String((Player number of (Owner of (Dying unit))))) Equal to String_Loc

• Actions
  • Set PlayerName[1] = (|CFFFF0303 + ((Name of Player 1 (Red)) + |R))
  • Set PlayerName[2] = (|CFF0042FF + ((Name of Player 2 (Blue)) + |R))
  • Set PlayerName[3] = (|CFF1BE6D8 + ((Name of Player 3 (Teal)) + |R))
  • Set PlayerName[4] = (|CFF540081 + ((Name of Player 4 (Purple)) + |R))
  • Set String_Loc = PlayerName[(Random integer number between 1 and 4)]

You set the variable to the player's name, but the condition uses the player number.