[Game][Arrows] Don't request deletion on inbound arrow deletion signal again but react to it locally (#6927)

* [Game][Arrows] Deleting arrows has no acknowledgement command so we have to delete locally as well.

Took 22 minutes

* Fix properly.

Took 15 minutes

---------

Co-authored-by: Lukas Brübach <Bruebach.Lukas@bdosecurity.de>

cockatrice/src/game/game_scene.cpp

@@ -93,6 +93,7 @@ void GameScene::addPlayer(PlayerLogic *player)
         rearrange();
     });
 
+    connect(player, &PlayerLogic::arrowDeleted, this, &GameScene::onArrowDeleted);
     connect(player, &PlayerLogic::arrowCreateRequested, this, &GameScene::onArrowCreateRequested);
     connect(player, &PlayerLogic::arrowDeleteRequested, this, &GameScene::onArrowDeleteRequested);
     connect(player, &PlayerLogic::arrowsCleared, this,
@@ -404,6 +405,13 @@ void GameScene::onArrowCreateRequested(const ArrowData &data)
     connect(arrow, &QObject::destroyed, this, [this, id = data.id]() { arrowRegistry.remove(id); });
 }
 
+void GameScene::onArrowDeleted(int arrowId)
+{
+    if (arrowRegistry.contains(arrowId)) {
+        arrowRegistry.take(arrowId)->delArrow();
+    }
+}
+
 void GameScene::onArrowDeleteRequested(int arrowId)
 {
     if (arrowRegistry.contains(arrowId)) {

cockatrice/src/game/game_scene.h

@@ -202,6 +202,7 @@ public slots:
     QTransform getViewportTransform() const;
 
     void onArrowCreateRequested(const ArrowData &data);
+    void onArrowDeleted(int arrowId);
     void onArrowDeleteRequested(int arrowId);
     void onCardZoneChanged(CardItem *card, bool sameZone);
     void clearArrowsForPlayer(int playerId);

cockatrice/src/game/player/player_event_handler.cpp

@@ -128,7 +128,7 @@ void PlayerEventHandler::eventCreateArrow(const Event_CreateArrow &event)
 
 void PlayerEventHandler::eventDeleteArrow(const Event_DeleteArrow &event)
 {
-    emit player->arrowDeleteRequested(event.arrow_id());
+    emit player->arrowDeleted(event.arrow_id());
 }
 
 void PlayerEventHandler::eventCreateToken(const Event_CreateToken &event)

cockatrice/src/game/player/player_logic.h

@@ -80,6 +80,7 @@ signals:
     void resetTopCardMenuActions();
     void arrowCreateRequested(ArrowData data);
     void arrowDeleteRequested(int arrowId);
+    void arrowDeleted(int arrowId);
     void arrowsCleared(); // fires on clear() and processPlayerInfo
 
 public slots: