Merge pull request 'profile' (#6) from profile into main
Reviewed-on: #6
This commit was merged in pull request #6.
This commit is contained in:
102
engine.py
102
engine.py
@@ -32,7 +32,11 @@ class SorterEngine:
|
|||||||
(folder_path TEXT, filename TEXT, category TEXT, tag_index INTEGER,
|
(folder_path TEXT, filename TEXT, category TEXT, tag_index INTEGER,
|
||||||
PRIMARY KEY (folder_path, filename))''')
|
PRIMARY KEY (folder_path, filename))''')
|
||||||
|
|
||||||
# Seed categories if empty
|
# --- NEW: PROFILE CATEGORIES TABLE (each profile has its own categories) ---
|
||||||
|
cursor.execute('''CREATE TABLE IF NOT EXISTS profile_categories
|
||||||
|
(profile TEXT, category TEXT, PRIMARY KEY (profile, category))''')
|
||||||
|
|
||||||
|
# Seed categories if empty (legacy table)
|
||||||
cursor.execute("SELECT COUNT(*) FROM categories")
|
cursor.execute("SELECT COUNT(*) FROM categories")
|
||||||
if cursor.fetchone()[0] == 0:
|
if cursor.fetchone()[0] == 0:
|
||||||
for cat in ["_TRASH", "control", "Default", "Action", "Solo"]:
|
for cat in ["_TRASH", "control", "Default", "Action", "Solo"]:
|
||||||
@@ -108,21 +112,45 @@ class SorterEngine:
|
|||||||
"tab5_source": r[7], "tab5_out": r[8]
|
"tab5_source": r[7], "tab5_out": r[8]
|
||||||
} for r in rows}
|
} for r in rows}
|
||||||
|
|
||||||
# --- 3. CATEGORY MANAGEMENT (Sorted A-Z) ---
|
# --- 3. CATEGORY MANAGEMENT (Profile-based) ---
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def get_categories():
|
def get_categories(profile=None):
|
||||||
conn = sqlite3.connect(SorterEngine.DB_PATH)
|
conn = sqlite3.connect(SorterEngine.DB_PATH)
|
||||||
cursor = conn.cursor()
|
cursor = conn.cursor()
|
||||||
|
|
||||||
|
# Ensure table exists
|
||||||
|
cursor.execute('''CREATE TABLE IF NOT EXISTS profile_categories
|
||||||
|
(profile TEXT, category TEXT, PRIMARY KEY (profile, category))''')
|
||||||
|
|
||||||
|
if profile:
|
||||||
|
cursor.execute("SELECT category FROM profile_categories WHERE profile = ? ORDER BY category COLLATE NOCASE ASC", (profile,))
|
||||||
|
cats = [r[0] for r in cursor.fetchall()]
|
||||||
|
# If no categories for this profile, seed with defaults
|
||||||
|
if not cats:
|
||||||
|
for cat in ["_TRASH", "control"]:
|
||||||
|
cursor.execute("INSERT OR IGNORE INTO profile_categories VALUES (?, ?)", (profile, cat))
|
||||||
|
conn.commit()
|
||||||
|
cats = ["_TRASH", "control"]
|
||||||
|
else:
|
||||||
|
# Fallback to legacy table
|
||||||
cursor.execute("SELECT name FROM categories ORDER BY name COLLATE NOCASE ASC")
|
cursor.execute("SELECT name FROM categories ORDER BY name COLLATE NOCASE ASC")
|
||||||
cats = [r[0] for r in cursor.fetchall()]
|
cats = [r[0] for r in cursor.fetchall()]
|
||||||
|
|
||||||
conn.close()
|
conn.close()
|
||||||
return cats
|
return cats
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def add_category(name):
|
def add_category(name, profile=None):
|
||||||
conn = sqlite3.connect(SorterEngine.DB_PATH)
|
conn = sqlite3.connect(SorterEngine.DB_PATH)
|
||||||
cursor = conn.cursor()
|
cursor = conn.cursor()
|
||||||
|
|
||||||
|
if profile:
|
||||||
|
cursor.execute('''CREATE TABLE IF NOT EXISTS profile_categories
|
||||||
|
(profile TEXT, category TEXT, PRIMARY KEY (profile, category))''')
|
||||||
|
cursor.execute("INSERT OR IGNORE INTO profile_categories VALUES (?, ?)", (profile, name))
|
||||||
|
else:
|
||||||
cursor.execute("INSERT OR IGNORE INTO categories VALUES (?)", (name,))
|
cursor.execute("INSERT OR IGNORE INTO categories VALUES (?)", (name,))
|
||||||
|
|
||||||
conn.commit()
|
conn.commit()
|
||||||
conn.close()
|
conn.close()
|
||||||
|
|
||||||
@@ -267,13 +295,13 @@ class SorterEngine:
|
|||||||
return {r[0]: {"cat": r[1], "name": r[2], "marked": r[3]} for r in rows}
|
return {r[0]: {"cat": r[1], "name": r[2], "marked": r[3]} for r in rows}
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def commit_global(output_root, cleanup_mode, operation="Copy", source_root=None):
|
def commit_global(output_root, cleanup_mode, operation="Copy", source_root=None, profile=None):
|
||||||
"""Commits ALL staged files and fixes permissions."""
|
"""Commits ALL staged files and fixes permissions."""
|
||||||
data = SorterEngine.get_staged_data()
|
data = SorterEngine.get_staged_data()
|
||||||
|
|
||||||
# Save folder tags BEFORE processing (so we can restore them later)
|
# Save folder tags BEFORE processing (so we can restore them later)
|
||||||
if source_root:
|
if source_root:
|
||||||
SorterEngine.save_folder_tags(source_root)
|
SorterEngine.save_folder_tags(source_root, profile)
|
||||||
|
|
||||||
conn = sqlite3.connect(SorterEngine.DB_PATH)
|
conn = sqlite3.connect(SorterEngine.DB_PATH)
|
||||||
cursor = conn.cursor()
|
cursor = conn.cursor()
|
||||||
@@ -513,11 +541,16 @@ class SorterEngine:
|
|||||||
conn.close()
|
conn.close()
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def delete_category(name):
|
def delete_category(name, profile=None):
|
||||||
"""Deletes a category and clears any staged tags associated with it."""
|
"""Deletes a category and clears any staged tags associated with it."""
|
||||||
conn = sqlite3.connect(SorterEngine.DB_PATH)
|
conn = sqlite3.connect(SorterEngine.DB_PATH)
|
||||||
cursor = conn.cursor()
|
cursor = conn.cursor()
|
||||||
|
|
||||||
|
if profile:
|
||||||
|
cursor.execute("DELETE FROM profile_categories WHERE profile = ? AND category = ?", (profile, name))
|
||||||
|
else:
|
||||||
cursor.execute("DELETE FROM categories WHERE name = ?", (name,))
|
cursor.execute("DELETE FROM categories WHERE name = ?", (name,))
|
||||||
|
|
||||||
cursor.execute("DELETE FROM staging_area WHERE target_category = ?", (name,))
|
cursor.execute("DELETE FROM staging_area WHERE target_category = ?", (name,))
|
||||||
conn.commit()
|
conn.commit()
|
||||||
conn.close()
|
conn.close()
|
||||||
@@ -536,7 +569,7 @@ class SorterEngine:
|
|||||||
|
|
||||||
# --- 7. FOLDER TAG PERSISTENCE ---
|
# --- 7. FOLDER TAG PERSISTENCE ---
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def save_folder_tags(folder_path):
|
def save_folder_tags(folder_path, profile=None):
|
||||||
"""
|
"""
|
||||||
Saves current staging data associated with a folder for later restoration.
|
Saves current staging data associated with a folder for later restoration.
|
||||||
Call this BEFORE clearing the staging area.
|
Call this BEFORE clearing the staging area.
|
||||||
@@ -549,11 +582,22 @@ class SorterEngine:
|
|||||||
conn = sqlite3.connect(SorterEngine.DB_PATH)
|
conn = sqlite3.connect(SorterEngine.DB_PATH)
|
||||||
cursor = conn.cursor()
|
cursor = conn.cursor()
|
||||||
|
|
||||||
# Ensure table exists (for existing databases)
|
# Ensure table exists with profile column
|
||||||
cursor.execute('''CREATE TABLE IF NOT EXISTS folder_tags
|
cursor.execute('''CREATE TABLE IF NOT EXISTS folder_tags
|
||||||
(folder_path TEXT, filename TEXT, category TEXT, tag_index INTEGER,
|
(profile TEXT, folder_path TEXT, filename TEXT, category TEXT, tag_index INTEGER,
|
||||||
PRIMARY KEY (folder_path, filename))''')
|
PRIMARY KEY (profile, folder_path, filename))''')
|
||||||
|
|
||||||
|
# Check if old schema (without profile) - migrate if needed
|
||||||
|
cursor.execute("PRAGMA table_info(folder_tags)")
|
||||||
|
columns = [row[1] for row in cursor.fetchall()]
|
||||||
|
if 'profile' not in columns:
|
||||||
|
cursor.execute("DROP TABLE folder_tags")
|
||||||
|
cursor.execute('''CREATE TABLE folder_tags
|
||||||
|
(profile TEXT, folder_path TEXT, filename TEXT, category TEXT, tag_index INTEGER,
|
||||||
|
PRIMARY KEY (profile, folder_path, filename))''')
|
||||||
|
conn.commit()
|
||||||
|
|
||||||
|
profile = profile or "Default"
|
||||||
saved_count = 0
|
saved_count = 0
|
||||||
for orig_path, info in staged.items():
|
for orig_path, info in staged.items():
|
||||||
# Only save tags for files that are in this folder (or subfolders)
|
# Only save tags for files that are in this folder (or subfolders)
|
||||||
@@ -567,8 +611,8 @@ class SorterEngine:
|
|||||||
tag_index = int(match.group(1)) if match else 0
|
tag_index = int(match.group(1)) if match else 0
|
||||||
|
|
||||||
cursor.execute(
|
cursor.execute(
|
||||||
"INSERT OR REPLACE INTO folder_tags VALUES (?, ?, ?, ?)",
|
"INSERT OR REPLACE INTO folder_tags VALUES (?, ?, ?, ?, ?)",
|
||||||
(folder_path, filename, category, tag_index)
|
(profile, folder_path, filename, category, tag_index)
|
||||||
)
|
)
|
||||||
saved_count += 1
|
saved_count += 1
|
||||||
|
|
||||||
@@ -577,7 +621,7 @@ class SorterEngine:
|
|||||||
return saved_count
|
return saved_count
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def restore_folder_tags(folder_path, all_images):
|
def restore_folder_tags(folder_path, all_images, profile=None):
|
||||||
"""
|
"""
|
||||||
Restores previously saved tags for a folder back into the staging area.
|
Restores previously saved tags for a folder back into the staging area.
|
||||||
Call this when loading/reloading a folder.
|
Call this when loading/reloading a folder.
|
||||||
@@ -587,27 +631,27 @@ class SorterEngine:
|
|||||||
conn = sqlite3.connect(SorterEngine.DB_PATH)
|
conn = sqlite3.connect(SorterEngine.DB_PATH)
|
||||||
cursor = conn.cursor()
|
cursor = conn.cursor()
|
||||||
|
|
||||||
# Check if table exists and has correct schema
|
# Ensure table exists with profile column
|
||||||
cursor.execute("SELECT name FROM sqlite_master WHERE type='table' AND name='folder_tags'")
|
cursor.execute('''CREATE TABLE IF NOT EXISTS folder_tags
|
||||||
if cursor.fetchone():
|
(profile TEXT, folder_path TEXT, filename TEXT, category TEXT, tag_index INTEGER,
|
||||||
# Table exists - check if it has the filename column
|
PRIMARY KEY (profile, folder_path, filename))''')
|
||||||
|
|
||||||
|
# Check if old schema (without profile) - migrate if needed
|
||||||
cursor.execute("PRAGMA table_info(folder_tags)")
|
cursor.execute("PRAGMA table_info(folder_tags)")
|
||||||
columns = [row[1] for row in cursor.fetchall()]
|
columns = [row[1] for row in cursor.fetchall()]
|
||||||
if 'filename' not in columns:
|
if 'profile' not in columns:
|
||||||
# Wrong schema - drop and recreate
|
|
||||||
cursor.execute("DROP TABLE folder_tags")
|
cursor.execute("DROP TABLE folder_tags")
|
||||||
|
cursor.execute('''CREATE TABLE folder_tags
|
||||||
|
(profile TEXT, folder_path TEXT, filename TEXT, category TEXT, tag_index INTEGER,
|
||||||
|
PRIMARY KEY (profile, folder_path, filename))''')
|
||||||
conn.commit()
|
conn.commit()
|
||||||
|
|
||||||
# Create table with correct schema
|
profile = profile or "Default"
|
||||||
cursor.execute('''CREATE TABLE IF NOT EXISTS folder_tags
|
|
||||||
(folder_path TEXT, filename TEXT, category TEXT, tag_index INTEGER,
|
|
||||||
PRIMARY KEY (folder_path, filename))''')
|
|
||||||
conn.commit()
|
|
||||||
|
|
||||||
# Get saved tags for this folder
|
# Get saved tags for this folder and profile
|
||||||
cursor.execute(
|
cursor.execute(
|
||||||
"SELECT filename, category, tag_index FROM folder_tags WHERE folder_path = ?",
|
"SELECT filename, category, tag_index FROM folder_tags WHERE profile = ? AND folder_path = ?",
|
||||||
(folder_path,)
|
(profile, folder_path)
|
||||||
)
|
)
|
||||||
saved_tags = {row[0]: {"cat": row[1], "index": row[2]} for row in cursor.fetchall()}
|
saved_tags = {row[0]: {"cat": row[1], "index": row[2]} for row in cursor.fetchall()}
|
||||||
|
|
||||||
@@ -619,7 +663,6 @@ class SorterEngine:
|
|||||||
filename_to_path = {}
|
filename_to_path = {}
|
||||||
for img_path in all_images:
|
for img_path in all_images:
|
||||||
fname = os.path.basename(img_path)
|
fname = os.path.basename(img_path)
|
||||||
# Only map files that haven't been mapped yet (handles duplicates by using first occurrence)
|
|
||||||
if fname not in filename_to_path:
|
if fname not in filename_to_path:
|
||||||
filename_to_path[fname] = img_path
|
filename_to_path[fname] = img_path
|
||||||
|
|
||||||
@@ -628,7 +671,6 @@ class SorterEngine:
|
|||||||
for filename, tag_info in saved_tags.items():
|
for filename, tag_info in saved_tags.items():
|
||||||
if filename in filename_to_path:
|
if filename in filename_to_path:
|
||||||
full_path = filename_to_path[filename]
|
full_path = filename_to_path[filename]
|
||||||
# Check if not already staged
|
|
||||||
cursor.execute("SELECT 1 FROM staging_area WHERE original_path = ?", (full_path,))
|
cursor.execute("SELECT 1 FROM staging_area WHERE original_path = ?", (full_path,))
|
||||||
if not cursor.fetchone():
|
if not cursor.fetchone():
|
||||||
ext = os.path.splitext(filename)[1]
|
ext = os.path.splitext(filename)[1]
|
||||||
|
|||||||
@@ -78,7 +78,7 @@ class AppState:
|
|||||||
|
|
||||||
def get_categories(self) -> List[str]:
|
def get_categories(self) -> List[str]:
|
||||||
"""Get list of categories, ensuring active_cat exists."""
|
"""Get list of categories, ensuring active_cat exists."""
|
||||||
cats = SorterEngine.get_categories() or ["Default"]
|
cats = SorterEngine.get_categories(self.profile_name) or ["control"]
|
||||||
if self.active_cat not in cats:
|
if self.active_cat not in cats:
|
||||||
self.active_cat = cats[0]
|
self.active_cat = cats[0]
|
||||||
return cats
|
return cats
|
||||||
@@ -132,8 +132,8 @@ def load_images():
|
|||||||
|
|
||||||
state.all_images = SorterEngine.get_images(state.source_dir, recursive=True)
|
state.all_images = SorterEngine.get_images(state.source_dir, recursive=True)
|
||||||
|
|
||||||
# Restore previously saved tags for this folder
|
# Restore previously saved tags for this folder and profile
|
||||||
restored = SorterEngine.restore_folder_tags(state.source_dir, state.all_images)
|
restored = SorterEngine.restore_folder_tags(state.source_dir, state.all_images, state.profile_name)
|
||||||
if restored > 0:
|
if restored > 0:
|
||||||
ui.notify(f"Restored {restored} tags from previous session", type='info')
|
ui.notify(f"Restored {restored} tags from previous session", type='info')
|
||||||
|
|
||||||
@@ -238,7 +238,8 @@ async def action_apply_global():
|
|||||||
state.output_dir,
|
state.output_dir,
|
||||||
state.cleanup_mode,
|
state.cleanup_mode,
|
||||||
state.batch_mode,
|
state.batch_mode,
|
||||||
state.source_dir
|
state.source_dir,
|
||||||
|
state.profile_name
|
||||||
)
|
)
|
||||||
load_images()
|
load_images()
|
||||||
ui.notify("Global apply complete!", type='positive')
|
ui.notify("Global apply complete!", type='positive')
|
||||||
@@ -340,7 +341,7 @@ def render_sidebar():
|
|||||||
|
|
||||||
def add_category():
|
def add_category():
|
||||||
if new_cat_input.value:
|
if new_cat_input.value:
|
||||||
SorterEngine.add_category(new_cat_input.value)
|
SorterEngine.add_category(new_cat_input.value, state.profile_name)
|
||||||
state.active_cat = new_cat_input.value
|
state.active_cat = new_cat_input.value
|
||||||
refresh_staged_info()
|
refresh_staged_info()
|
||||||
render_sidebar()
|
render_sidebar()
|
||||||
@@ -350,7 +351,7 @@ def render_sidebar():
|
|||||||
# Delete category
|
# Delete category
|
||||||
with ui.expansion('Danger Zone', icon='warning').classes('w-full text-red-400 mt-2'):
|
with ui.expansion('Danger Zone', icon='warning').classes('w-full text-red-400 mt-2'):
|
||||||
def delete_category():
|
def delete_category():
|
||||||
SorterEngine.delete_category(state.active_cat)
|
SorterEngine.delete_category(state.active_cat, state.profile_name)
|
||||||
refresh_staged_info()
|
refresh_staged_info()
|
||||||
render_sidebar()
|
render_sidebar()
|
||||||
|
|
||||||
@@ -492,16 +493,57 @@ def build_header():
|
|||||||
with ui.row().classes('w-full items-center gap-4 no-wrap px-4'):
|
with ui.row().classes('w-full items-center gap-4 no-wrap px-4'):
|
||||||
ui.label('🖼️ NiceSorter').classes('text-xl font-bold shrink-0 text-green-400')
|
ui.label('🖼️ NiceSorter').classes('text-xl font-bold shrink-0 text-green-400')
|
||||||
|
|
||||||
# Profile selector
|
# Profile selector with add/delete
|
||||||
profile_names = list(state.profiles.keys())
|
|
||||||
|
|
||||||
def change_profile(e):
|
def change_profile(e):
|
||||||
state.profile_name = e.value
|
state.profile_name = e.value
|
||||||
state.load_active_profile()
|
state.load_active_profile()
|
||||||
load_images()
|
state.active_cat = "control" # Reset to default category
|
||||||
|
refresh_staged_info()
|
||||||
|
refresh_ui()
|
||||||
|
|
||||||
ui.select(profile_names, value=state.profile_name, on_change=change_profile) \
|
profile_select = ui.select(
|
||||||
.props('dark dense options-dense borderless').classes('w-32')
|
list(state.profiles.keys()),
|
||||||
|
value=state.profile_name,
|
||||||
|
on_change=change_profile
|
||||||
|
).props('dark dense options-dense borderless').classes('w-32')
|
||||||
|
|
||||||
|
def add_profile():
|
||||||
|
with ui.dialog() as dialog, ui.card().classes('p-4'):
|
||||||
|
ui.label('New Profile Name').classes('font-bold')
|
||||||
|
name_input = ui.input(placeholder='Profile name').props('autofocus')
|
||||||
|
|
||||||
|
def do_create():
|
||||||
|
name = name_input.value
|
||||||
|
if name and name not in state.profiles:
|
||||||
|
state.profiles[name] = {"tab5_source": "/storage", "tab5_out": "/storage"}
|
||||||
|
SorterEngine.save_tab_paths(name, t5_s="/storage", t5_o="/storage")
|
||||||
|
state.profile_name = name
|
||||||
|
state.load_active_profile()
|
||||||
|
dialog.close()
|
||||||
|
ui.notify(f"Profile '{name}' created", type='positive')
|
||||||
|
# Rebuild header to update profile list
|
||||||
|
ui.navigate.reload()
|
||||||
|
elif name in state.profiles:
|
||||||
|
ui.notify("Profile already exists", type='warning')
|
||||||
|
|
||||||
|
with ui.row().classes('w-full justify-end gap-2 mt-2'):
|
||||||
|
ui.button('Cancel', on_click=dialog.close).props('flat')
|
||||||
|
ui.button('Create', on_click=do_create).props('color=green')
|
||||||
|
dialog.open()
|
||||||
|
|
||||||
|
def delete_profile():
|
||||||
|
if len(state.profiles) <= 1:
|
||||||
|
ui.notify("Cannot delete the last profile", type='warning')
|
||||||
|
return
|
||||||
|
deleted_name = state.profile_name
|
||||||
|
del state.profiles[state.profile_name]
|
||||||
|
state.profile_name = list(state.profiles.keys())[0]
|
||||||
|
state.load_active_profile()
|
||||||
|
ui.notify(f"Profile '{deleted_name}' deleted", type='info')
|
||||||
|
ui.navigate.reload()
|
||||||
|
|
||||||
|
ui.button(icon='add', on_click=add_profile).props('flat round dense color=green').tooltip('New profile')
|
||||||
|
ui.button(icon='delete', on_click=delete_profile).props('flat round dense color=red').tooltip('Delete profile')
|
||||||
|
|
||||||
# Source and output paths
|
# Source and output paths
|
||||||
with ui.row().classes('flex-grow gap-2'):
|
with ui.row().classes('flex-grow gap-2'):
|
||||||
|
|||||||
Reference in New Issue
Block a user