[BUG] Match Timeline: Viego interactions with his passive and item destroy events

[Degcube]

Bug Description This occurs when attempting to request a match timeline wherein one of the participants picks Viego. It becomes impossible to process item events for Viego, as all his items are marked as "ITEM_DESTROYED" in the timeline frame events when he takes over a "soul" with his passive and becomes the enemy champion. Even worse, when he switches back to Viego or to another soul, all their new items in his inventory also generate these events.

Expected Result I would expect that when Viego takes over a soul or when he drops the soul and becomes Viego again, no events should be created at all. These should be ignored or filtered out, or at least differentiated from other "ITEM_DESTROYED" events.

Developer Impact Attempting to write logic that processes item events for Viego is impossible.

• Frequency: Occurs in every Viego game after every passive use.
• Severity: Viego has an 8-10% pick rate in nearly every elo, causing problems for the majority of games.

Player Impact Excluding Viego from our features is a significant loss for the users of the service. The champ has a high pick rate, and this problem should not be ignored.

[bangingheads]

Duplicate of #568

[Degcube]

Duplicate of #568

2.5 years.... this is painful