diff --git a/cockatrice/src/game/game_event_handler.cpp b/cockatrice/src/game/game_event_handler.cpp
index 03930b0de..b7f687030 100644
--- a/cockatrice/src/game/game_event_handler.cpp
+++ b/cockatrice/src/game/game_event_handler.cpp
@@ -268,8 +268,9 @@ void GameEventHandler::eventGameStateChanged(const Event_GameStateChanged &event
                                              const GameEventContext & /*context*/)
 {
     // Sub-games of a tournament report their parent hub game so the client can
-    // route "close game" back to the parent tab.
-    if (event.parent_game_id() != -1) {
+    // route "close game" back to the parent tab. Use has_parent_game_id rather
+    // than comparing the default so a future drop of the proto default still works.
+    if (event.has_parent_game_id()) {
         game->getGameMetaInfo()->setParentGameId(event.parent_game_id());
     }
 
diff --git a/cockatrice/src/game/game_meta_info.h b/cockatrice/src/game/game_meta_info.h
index 8585238bd..6b3569e64 100644
--- a/cockatrice/src/game/game_meta_info.h
+++ b/cockatrice/src/game/game_meta_info.h
@@ -89,11 +89,6 @@ public:
         return gameInfo_.is_tournament();
     }
 
-    void setIsTournament(bool t)
-    {
-        gameInfo_.set_is_tournament(t);
-    }
-
     int parentGameId() const
     {
         return parentGameId_;
diff --git a/cockatrice/src/game/player/player_event_handler.h b/cockatrice/src/game/player/player_event_handler.h
index 1e9814ddb..48ad85e88 100644
--- a/cockatrice/src/game/player/player_event_handler.h
+++ b/cockatrice/src/game/player/player_event_handler.h
@@ -90,10 +90,10 @@ public:
      * @param context Additional context (undo, judge, etc.).
      * @param options Processing options (UI suppression, reveal behavior).
      */
-    virtual void processGameEvent(GameEvent::GameEventType type,
-                                  const GameEvent &event,
-                                  const GameEventContext &context,
-                                  EventProcessingOptions options);
+    void processGameEvent(GameEvent::GameEventType type,
+                          const GameEvent &event,
+                          const GameEventContext &context,
+                          EventProcessingOptions options);
 
     /** @} */
 
@@ -266,7 +266,7 @@ signals:
     void cardZoneChanged(CardItem *card, bool sameZone);
     void requestCardMenuUpdate(const CardItem *card);
 
-protected:
+private:
     /** Owning player instance. */
     PlayerLogic *player;
 
diff --git a/cockatrice/src/game/player/player_logic.cpp b/cockatrice/src/game/player/player_logic.cpp
index 084f3df30..45ba09aac 100644
--- a/cockatrice/src/game/player/player_logic.cpp
+++ b/cockatrice/src/game/player/player_logic.cpp
@@ -37,19 +37,6 @@ PlayerLogic::PlayerLogic(const ServerInfo_User &info, int _id, bool _local, bool
     initializeZones();
 }
 
-PlayerLogic::PlayerLogic(const ServerInfo_User &info,
-                         int _id,
-                         bool _local,
-                         bool _judge,
-                         AbstractGame *_parent,
-                         PlayerEventHandler *customEventHandler)
-    : QObject(_parent), game(_parent), playerInfo(new PlayerInfo(info, _id, _local, _judge)),
-      playerEventHandler(customEventHandler), playerActions(new PlayerActions(this)), active(false), conceded(false),
-      zoneId(0), dialogSemaphore(false)
-{
-    initializeZones();
-}
-
 void PlayerLogic::initializeZones()
 {
     addZone(new PileZoneLogic(this, ZoneNames::DECK, false, true, false, this));
diff --git a/cockatrice/src/game/player/player_logic.h b/cockatrice/src/game/player/player_logic.h
index 26cd63f50..6923b3afe 100644
--- a/cockatrice/src/game/player/player_logic.h
+++ b/cockatrice/src/game/player/player_logic.h
@@ -99,30 +99,7 @@ public:
     PlayerLogic(const ServerInfo_User &info, int _id, bool _local, bool _judge, AbstractGame *_parent);
     ~PlayerLogic() override;
 
-protected:
-    /**
-     * @brief Constructor for subclasses that need a custom event handler (e.g. DraftPlayerLogic).
-     *
-     * @p customEventHandler must be non-null and either QObject-parented to this PlayerLogic
-     * or deleted externally; it is not owned by PlayerLogic. The handler connects to @c player
-     * during its own constructor, so passing a freshly built subclass handler from an
-     * initializer list is safe.
-     */
-    PlayerLogic(const ServerInfo_User &info,
-                int _id,
-                bool _local,
-                bool _judge,
-                AbstractGame *_parent,
-                PlayerEventHandler *customEventHandler);
-
-public:
-    /**
-     * @brief Creates the standard zone set.
-     *
-     * Not virtually dispatched from constructors — subclasses overriding this must add
-     * their extra zones in their own constructor body.
-     */
-    virtual void initializeZones();
+    void initializeZones();
     void updateZones();
     void clear();