From 040f8396dd85e1691f4e2929e0a23397d8849c87 Mon Sep 17 00:00:00 2001
From: kiyreload27 <xtrickline@gmail.com>
Date: Wed, 31 Dec 2025 23:18:32 +0000
Subject: [PATCH] feat: Implement database update/sync from seed, add schema
 migrations, and introduce deck skills view.

---
 database/umamusume_seed.db | Bin 1581056 -> 1581056 bytes
 db/db_queries.py           |  42 +++++++++++++++++++++++++++++--------
 gui/deck_skills_view.py    |   4 +++-
 version.py                 |   2 +-
 4 files changed, 37 insertions(+), 11 deletions(-)

diff --git a/database/umamusume_seed.db b/database/umamusume_seed.db
index 98e2438b26868ef6940eb8cabf5b44b296883f7c..473e962ce28891dce6d610118b273570e7594bef 100644
GIT binary patch
delta 83
zcmV~$NfCoU006-voZ(CeBtD`92`gob$4KFg1ROAqpgSHR9$~-E<iCmuJ}|`$b1bmL
a3ZGbGgDrO0<A5)G<A@)eaK`WEZ@B@fgeQUk

delta 83
zcmV~$NfCoU006-voZ(CeBsL-i2`gob$4KFg1ROAqpgSHR9$~-E<iCmuJ}|`$b1bmL
a3ZGbGgDrO0<A5)G<A@)eaK`WEZ@B@e@+W`*

diff --git a/db/db_queries.py b/db/db_queries.py
index 7381795..3a1eb3a 100644
--- a/db/db_queries.py
+++ b/db/db_queries.py
@@ -75,10 +75,41 @@ def get_conn():
     # Check for updates and migrate if needed (only once per session)
     if not _updates_checked:
         _updates_checked = True
+        run_migrations()
         check_for_updates()
         
     return sqlite3.connect(DB_PATH)
 
+def run_migrations():
+    """Ensure database schema is up to date by adding missing columns"""
+    print("Checking for database migrations...")
+    conn = sqlite3.connect(DB_PATH)
+    cur = conn.cursor()
+    
+    # 1. Add is_gold to event_skills
+    try:
+        cur.execute("ALTER TABLE event_skills ADD COLUMN is_gold INTEGER DEFAULT 0")
+        print("Added is_gold column to event_skills")
+    except sqlite3.OperationalError:
+        pass # Column already exists
+        
+    # 2. Add is_or to event_skills
+    try:
+        cur.execute("ALTER TABLE event_skills ADD COLUMN is_or INTEGER DEFAULT 0")
+        print("Added is_or column to event_skills")
+    except sqlite3.OperationalError:
+        pass # Column already exists
+    
+    # 3. Add image_path to support_cards
+    try:
+        cur.execute("ALTER TABLE support_cards ADD COLUMN image_path TEXT")
+        print("Added image_path column to support_cards")
+    except sqlite3.OperationalError:
+        pass # Column already exists
+        
+    conn.commit()
+    conn.close()
+
 def check_for_updates():
     """Check if database version matches app version, sync if outdated"""
     if getattr(sys, 'frozen', False):
@@ -299,15 +330,8 @@ def init_database():
         )
     """)
     
-    # Migration: Add columns to event_skills if they don't exist
-    try:
-        cur.execute("ALTER TABLE event_skills ADD COLUMN is_gold INTEGER DEFAULT 0")
-    except sqlite3.OperationalError:
-        pass # Column already exists
-    try:
-        cur.execute("ALTER TABLE event_skills ADD COLUMN is_or INTEGER DEFAULT 0")
-    except sqlite3.OperationalError:
-        pass # Column already exists
+    # Run migrations to ensure all columns exist
+    run_migrations()
     
     # User tables
     cur.execute("""
diff --git a/gui/deck_skills_view.py b/gui/deck_skills_view.py
index 8390434..3da530d 100644
--- a/gui/deck_skills_view.py
+++ b/gui/deck_skills_view.py
@@ -161,8 +161,10 @@ class DeckSkillsFrame(ttk.Frame):
         display_text = f"{owned_mark}{name}"
         
         # Parent row only needs display text and rarity (optional)
-        iid = self.tree.insert('', tk.END, text=display_text, image=img, open=True,
+        # We remove open=True from insert and set it using item() to avoid Tcl Errors on some systems
+        iid = self.tree.insert('', tk.END, text=display_text, image=img if img else "",
                                values=("", rarity, f"{type_icon} {card_type}", ""))
+        self.tree.item(iid, open=True)
         return iid
 
     def add_skill_row(self, parent_id, skill_name, source, details):
diff --git a/version.py b/version.py
index b01c3c8..fe59d97 100644
--- a/version.py
+++ b/version.py
@@ -4,7 +4,7 @@ This file is the single source of truth for the application version.
 """
 
 # Semantic versioning: MAJOR.MINOR.PATCH
-VERSION: str = "13.0.0"
+VERSION: str = "13.0.2"
 
 # Application metadata
 APP_NAME: str = "UmamusumeCardManager"