Fix NPE when cloning Adventure creatures on mobile (#10349)

When a clone (e.g. Superior Spider-Man) copies an Adventure creature,
the CardView's AlternateState was null because getAlternateState() only
checks original card states, not clone states. The mobile renderer then
crashes calling getName() on the null state when rendering with card
images disabled.

Two fixes: CardView.updateState() falls back to the Secondary clone
state, and CardImageRenderer falls back from the backup to the in-game
card when the backup lacks the adventure state.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: Hans Mackowiak <hanmac@gmx.de>

Author: 3 people

forge-game/src/main/java/forge/game/card/CardView.java

@@ -1144,6 +1144,12 @@ void updateState(Card c) {
             alternateState = c.getState(CardStateName.Original);
         }
 
+        // When a card is cloned as an Adventure creature, getAlternateState() returns null
+        // because it only checks original states. Fall back to the Secondary clone state.
+        if (alternateState == null && c.hasState(CardStateName.Secondary)) {
+            alternateState = c.getState(CardStateName.Secondary);
+        }
+
         if (alternateState == null) {
             set(TrackableProperty.AlternateState, null);
         } else {

forge-gui-mobile/src/forge/card/CardImageRenderer.java

@@ -656,6 +656,10 @@ private static void setTextBox(Graphics g, CardView card, CardStateView state, C
                     if (cv == null || isFaceDown)
                         cv = card;
                     CardStateView csv = cv.getState(true);
+                    if (csv == null) { // backup may not have adventure state (e.g. clone of adventure)
+                        cv = card;
+                        csv = cv.getState(true);
+                    }
                     text = cv.getText(csv, needTranslation && csv != null ? CardTranslation.getTranslationTexts(csv) : null);
 
                 } else {