Warning in actor viewer when previewing attack animations

[historic_bruno]

Another symptom of actor viewer lacking player assignments (see #1499). Attack animations use the Attack component which tries to apply techs:

WARNING: GetPlayerByID: no player defined for id '0'
(0)@simulation/components/PlayerManager.js:28
QueryOwnerInterface(5,28)@simulation/helpers/Player.js:168
("Melee")@simulation/components/Attack.js:298 @(eval):1

Everything works because Attack handles this failure gracefully. I don't know if we should add proper player data to actor viewer's simulation (and keep it synced with the editor's simulation) or just squash the warning.

[historic_bruno]

Other random ideas:

• Add some kind of global game option component, that allows turning off techs, territories, etc., and add the logic wherever these components would be used (more complicated but may be beneficial later when we want different game types) - in this case actor viewer could disable techs in its local simulation.
• If we remove the GetPlayerByID warning, we should check every use of GetPlayerByID including indirect calls with QueryOwnerInterface and handle the return value properly (INVALID_ENTITY or null). But this is really just covering up the problem in actor viewer.

[historic_bruno]

The warning has been fixed.