Separate concerns of Player object

[xeruf]

The Player object right now is a big mess, it handles color, name, listeners and reporting invalid player moves. I want to separate these concerns with this PR.

[anarchuser]

Do you think it makes sense to update this PR?
the plugin2020 shouldn't matter anymore anyways

[xeruf]

Well the general premise should be implemented - having separate player classes for network client vs an entity that plays. But I guess starting from scratch would be faster. Until then we can keep this as reference.

[anarchuser]

I'm looking into what can be done right now.

As I see it, we need to move the Team/PlayerColor stuff into the game plugin (since the way that teams / colours are defined differs between the games) and only leave the player representing the client in the sdk

[xeruf]

Well, for all games so far the color was an identifying feature of the player, I think this is the first one where it is different - so the perfect time to rethink the architecture ;)

[anarchuser]

yes, as I already mentioned.
I'll create a new PR where I just rework the colors; afterwards I'll look into what's needed to rework the player