[Client] Drop dead player event handler extension points

The second PlayerLogic constructor taking a custom PlayerEventHandler was UB
(converting this to PlayerLogic* in the base initializer, and connecting a
freshly-built handler against a not-yet-constructed object), and its abstract
extension surface — virtual initializeZones, virtual getPlayerEventHandler,
virtual processGameEvent, protected player member — has no subclass consumer in
the stack. Revert to the single primary constructor.

Also:
- Drop the unused GameMetaInfo::setIsTournament setter. Tournament status is
  authoritative from Event_GameJoined's ServerInfo_Game; the redundant
  is_tournament field on Event_GameStateChanged is removed in [Protocol].
- Gate the parent-game routing on has_parent_game_id() instead of comparing the
  proto default, so a future drop of the default keeps working.

libcockatrice_game/libcockatrice/game/player/player_event_handler.h

@@ -93,10 +93,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);
 
     /** @} */
 
@@ -277,7 +277,7 @@ signals:
      */
     void cardViewRefreshRequested(CardState *card, bool zoneChanged, PlayerLogic *newOwner);
 
-protected:
+private:
     /** Owning player instance. */
     PlayerLogic *player;
 

libcockatrice_game/libcockatrice/game/player/player_logic.cpp

@@ -49,19 +49,6 @@ PlayerLogic::PlayerLogic(const ServerInfo_User &info,
     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));

libcockatrice_game/libcockatrice/game/player/player_logic.h

@@ -103,30 +103,7 @@ public:
                 ICardPreferenceProvider *cardPreferences = nullptr);
     ~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();