From 80719febaff87bc070b59043034e44d6dbcfdad4 Mon Sep 17 00:00:00 2001
From: ebbit1q <ebbit1q@gmail.com>
Date: Sat, 11 Apr 2026 18:45:25 +0200
Subject: [PATCH] remove break statement when closing views

it's possible that multiple views need to be closed
this is an oversight of #4570
---
 cockatrice/src/game/player/player_event_handler.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/cockatrice/src/game/player/player_event_handler.cpp b/cockatrice/src/game/player/player_event_handler.cpp
index 5571010d1..6fb1ff19a 100644
--- a/cockatrice/src/game/player/player_event_handler.cpp
+++ b/cockatrice/src/game/player/player_event_handler.cpp
@@ -60,7 +60,6 @@ void PlayerEventHandler::eventShuffle(const Event_Shuffle &event)
             // we want to close empty views as well
             if (length == 0 || length > absStart) { // note this assumes views always start at the top of the library
                 view->close();
-                break;
             }
         } else {
             qWarning() << zone->getName() << "of" << player->getPlayerInfo()->getName() << "holds empty zoneview!";
@@ -595,4 +594,4 @@ void PlayerEventHandler::processGameEvent(GameEvent::GameEventType type,
             qWarning() << "unhandled game event" << type;
         }
     }
-}
\ No newline at end of file
+}