PlayerGraphicsItem is responsible for retranslateUi, not Player.

Took 14 seconds

cockatrice/src/client/tabs/tab_game.cpp

@@ -169,7 +169,6 @@ void TabGame::connectToGameEventHandler()
 
 void TabGame::connectMessageLogToGameEventHandler()
 {
-    // connect(game->getGameEventHandler(), &GameEventHandler:: , messageLog, &MessageLogWidget::);
     connect(game->getGameEventHandler(), &GameEventHandler::gameFlooded, messageLog, &MessageLogWidget::logGameFlooded);
     connect(game->getGameEventHandler(), &GameEventHandler::containerProcessingStarted, messageLog,
             &MessageLogWidget::containerProcessingStarted);
@@ -366,7 +365,7 @@ void TabGame::retranslateUi()
     QMapIterator<int, Player *> i(game->getPlayerManager()->getPlayers());
 
     while (i.hasNext())
-        i.next().value()->retranslateUi();
+        i.next().value()->getGraphicsItem()->retranslateUi();
     QMapIterator<int, TabbedDeckViewContainer *> j(deckViewContainers);
     while (j.hasNext())
         j.next().value()->playerDeckView->retranslateUi();

cockatrice/src/game/player/player.cpp

@@ -71,19 +71,6 @@ Player::~Player()
     delete getPlayerInfo()->userInfo;
 }
 
-void Player::retranslateUi()
-{
-    QMapIterator<QString, CardZoneLogic *> zoneIterator(zones);
-    while (zoneIterator.hasNext()) {
-        emit zoneIterator.next().value()->retranslateUi();
-    }
-
-    QMapIterator<int, AbstractCounter *> counterIterator(counters);
-    while (counterIterator.hasNext()) {
-        counterIterator.next().value()->retranslateUi();
-    }
-}
-
 void Player::clear()
 {
     clearArrows();

cockatrice/src/game/player/player.h

@@ -76,7 +76,6 @@ public slots:
 public:
     Player(const ServerInfo_User &info, int _id, bool _local, bool _judge, Game *_parent);
     ~Player() override;
-    void retranslateUi();
 
     void initializeZones();
     void updateZones();

cockatrice/src/game/player/player_graphics_item.cpp

@@ -24,7 +24,23 @@ PlayerGraphicsItem::PlayerGraphicsItem(Player *_player) : player(_player)
     updateBoundingRect();
 
     rearrangeZones();
-    // retranslateUi();
+    retranslateUi();
+}
+
+void PlayerGraphicsItem::retranslateUi()
+{
+        player->getPlayerMenu()->retranslateUi();
+
+        QMapIterator<QString, CardZoneLogic *> zoneIterator(player->getZones());
+        while (zoneIterator.hasNext()) {
+            emit zoneIterator.next().value()->retranslateUi();
+        }
+
+        QMapIterator<int, AbstractCounter *> counterIterator(player->getCounters());
+        while (counterIterator.hasNext()) {
+            counterIterator.next().value()->retranslateUi();
+        }
+
 }
 
 void PlayerGraphicsItem::onPlayerActiveChanged(bool _active)

cockatrice/src/game/player/player_graphics_item.h

@@ -91,6 +91,7 @@ public:
 
 public slots:
     void onPlayerActiveChanged(bool _active);
+    void retranslateUi();
 
 signals:
     void sizeChanged();