Is this trigger not good enough?

[Dinodin]

Here is a trigger to make players gain gold when destroying an enemy's tower but players complains that it sometimes doesn't give any gold at all. It doesn't happening often but sometimes it does... What could be the problem?

• Good Kill Tower Evil
  • Events
    • Unit - A unit owned by Player 12 (Brown) Dies
  • Conditions
    • (Owner of (Killing unit)) Equal to Player 10 (Light Blue)
    • Or - Any (Conditions) are true
      • Conditions
        • (Unit-type of (Dying unit)) Equal to Orc Tower
        • (Unit-type of (Dying unit)) Equal to Ballista Tower
  • Actions
    • Set VariableSet BonusGoldTower = 300
    • Game - Display to PlayersGood the text: (The |cff0080FF + (Good|r + (|rforces destroyed a tower, awarding you + (|cffffcc00 + ((String((BonusGoldTower / (Number of players in PlayersGood)))) + |r gold!)))))
    • Player Group - Pick every player in PlayersGood and do (Actions)
      • Loop - Actions
        • Player - Add (BonusGoldTower / (Number of players in PlayersGood)) to (Picked player).Current gold

 

[Duckfarter]

If the killing unit is null or belongs to another player than player 10 the trigger conditions fail.

 

[A]mun]

I think you should simply check if owner of killing unit is in playersgood instead?!