You have now posted two versions of your trigger. I guess your second post shows the updated one?
Look at your last action. It ends the game for player "Number of players in (Allies of player (Killing player))", which is obviously not a usual player ID. For example, if the killing player has two allies, then this action would end the game for player 2! Instead, loop through all allies of the killing player and call the "End Game" action in every loop.
Another thing I noticed is that, in your second version, there is no action which lets a player end in defeat.
You have now posted two versions of your trigger. I guess your second post shows the updated one?
Look at your last action. It ends the game for player "Number of players in (Allies of player (Killing player))", which is obviously not a usual player ID. For example, if the killing player has two allies, then this action would end the game for player 2! Instead, loop through all allies of the killing player and call the "End Game" action in every loop.
Another thing I noticed is that, in your second version, there is no action which lets a player end in defeat.
When and how do you display the victory screen? Could you post the trigger please?