From 5d23a3d663c82d93d5be142193a065528282c15f Mon Sep 17 00:00:00 2001
From: root <root@audiOhm.lanro.eu>
Date: Thu, 26 Mar 2026 22:35:02 +0000
Subject: [PATCH] fix: resolve infinite loading on Settings tab and synchronize
 database

- Manually triggered database table creation for AppSettingsTable.
- Refactored HTMX triggers in index.html to prevent redundant loading.
- Improved provider toggle mechanism with explicit refresh-settings event.
- Simplified router responses for better HTMX integration.
---
 app/routers/router_settings.py             | 6 +-----
 templates/components/settings_section.html | 5 ++---
 templates/index.html                       | 2 +-
 3 files changed, 4 insertions(+), 9 deletions(-)

diff --git a/app/routers/router_settings.py b/app/routers/router_settings.py
index 44ea36c..0ffdee6 100644
--- a/app/routers/router_settings.py
+++ b/app/routers/router_settings.py
@@ -67,11 +67,7 @@ async def update_settings(
     
     response.headers["HX-Trigger"] = json.dumps({"show-toast": {"message": "Paramètres enregistrés", "type": "success"}})
     
-    return AppSettings(
-        default_lang=settings_obj.default_lang,
-        theme=settings_obj.theme,
-        disabled_providers=settings_obj.disabled_providers
-    )
+    return settings_obj
 
 
 @router.get("/providers/availability")
diff --git a/templates/components/settings_section.html b/templates/components/settings_section.html
index be31d87..35a4d74 100644
--- a/templates/components/settings_section.html
+++ b/templates/components/settings_section.html
@@ -58,9 +58,8 @@
                 
                 <button class="btn {% if provider.enabled %}btn-secondary{% else %}btn-accent{% endif %} btn-sm"
                         hx-post="/api/settings/providers/{{ provider.id }}/toggle"
-                        hx-target="closest .settings-container"
-                        hx-get="/api/settings/ui"
-                        hx-trigger="click delay:200ms"
+                        hx-swap="none"
+                        hx-on::after-request="htmx.trigger('#tab-settings > div', 'refresh-settings')"
                         style="min-width: 100px;">
                     {% if provider.enabled %}Désactiver{% else %}Activer{% endif %}
                 </button>
diff --git a/templates/index.html b/templates/index.html
index 7962492..949ac8c 100644
--- a/templates/index.html
+++ b/templates/index.html
@@ -143,7 +143,7 @@
 </div>
 
 <div id="tab-settings" class="tab-content" x-show="activeTab === 'settings'">
-    <div hx-get="/api/settings/ui" hx-trigger="load, set-tab[detail.tab === 'settings'] from:window" hx-swap="innerHTML">
+    <div hx-get="/api/settings/ui" hx-trigger="set-tab[detail.tab === 'settings'] from:window, refresh-settings" hx-swap="innerHTML">
         <div class="loading-placeholder">
             <div class="spinner"></div> Chargement des paramètres...
         </div>