Even local clients needs the player list synced

Could also have just done `client.setPlayer()` when starting the server, but that's more special-cased logic we don't
need.

src/main/java/net/rptools/maptool/server/MapToolServer.java

@@ -238,6 +238,15 @@ private void installConnection(Connection conn, Player player) {
     router.addConnection(conn);
 
     assetManagerMap.put(conn.getId(), new AssetTransferManager());
+
+    synchronized (playerMap) {
+      for (Player remotePlayer : playerMap.values()) {
+        var msg = PlayerConnectedMsg.newBuilder().setPlayer(remotePlayer.toDto());
+        sendMessage(conn.getId(), Message.newBuilder().setPlayerConnectedMsg(msg).build());
+      }
+    }
+    var msg = PlayerConnectedMsg.newBuilder().setPlayer(player.getTransferablePlayer().toDto());
+    broadcastMessage(Message.newBuilder().setPlayerConnectedMsg(msg).build());
   }
 
   public void addLocalConnection(Connection conn, Player localPlayer) {
@@ -247,15 +256,6 @@ public void addLocalConnection(Connection conn, Player localPlayer) {
   private void addRemoteConnection(Connection conn, Player connPlayer) {
     installConnection(conn, connPlayer);
 
-    synchronized (playerMap) {
-      for (Player player : playerMap.values()) {
-        var msg = PlayerConnectedMsg.newBuilder().setPlayer(player.toDto());
-        sendMessage(conn.getId(), Message.newBuilder().setPlayerConnectedMsg(msg).build());
-      }
-    }
-    var msg = PlayerConnectedMsg.newBuilder().setPlayer(connPlayer.getTransferablePlayer().toDto());
-    broadcastMessage(Message.newBuilder().setPlayerConnectedMsg(msg).build());
-
     var msg2 = SetCampaignMsg.newBuilder().setCampaign(campaign.toDto());
     sendMessage(conn.getId(), Message.newBuilder().setSetCampaignMsg(msg2).build());
   }