empathy: Don't force active:true when adding state for a new player
authorCarl Worth <cworth@cworth.org>
Sat, 4 Jul 2020 14:53:32 +0000 (07:53 -0700)
committerCarl Worth <cworth@cworth.org>
Sat, 4 Jul 2020 14:53:32 +0000 (07:53 -0700)
commitd3ae8ad65b770d0774dfffd0d853b15335f73e63
treea43555adb2780dacc41ca55d34979fc8bdd47ef7
parent6e3daa0768dafe57ad8f045b8314502316dc7c66
empathy: Don't force active:true when adding state for a new player

This fixes the bug where reloading the interface would cause inactive,
grayed-out player names to appear solid instead of grayed out.

What I had been thinking when writing this code is that this function
would be called only when a new player joins the game, (and by
definition, a newly joined player must be active). But this function
can also be called when a player reloads their interface, and at that
point player-join events are received for all players whether active
or not.

Meanwhile, the server is now explicitly providing the active bit for
players, so we get the right information without doing anything.
empathy/empathy.jsx