From 96bbe0a51059a3f51f1185e542fa168d9f9a9e8c Mon Sep 17 00:00:00 2001
From: Mikhail Chusavitin <mchusavitin@mchusmbp.local>
Date: Mon, 2 Feb 2026 12:32:44 +0300
Subject: [PATCH] fix: use originalHTML to restore button state after sync

- Pass originalHTML through syncAction function chain
- Simplify finally block by restoring original button innerHTML
- Remove hardcoded button HTML values (5 lines reduction)
- Improve maintainability: button text changes won't break code
- Preserve any custom classes, attributes, or nested elements

This fixes the issue where originalHTML was declared but never used.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
---
 web/templates/base.html | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/web/templates/base.html b/web/templates/base.html
index 06ad0f2..f290f6d 100644
--- a/web/templates/base.html
+++ b/web/templates/base.html
@@ -103,9 +103,9 @@
                 button.innerHTML = '<svg class="animate-spin w-4 h-4 inline mr-2" fill="none" viewBox="0 0 24 24"><circle class="opacity-25" cx="12" cy="12" r="10" stroke="currentColor" stroke-width="4"></circle><path class="opacity-75" fill="currentColor" d="M4 12a8 8 0 018-8V0C5.373 0 0 5.373 0 12h4zm2 5.291A7.962 7.962 0 014 12H0c0 3.042 1.135 5.824 3 7.938l3-2.647z"></path></svg> Синхронизация...';
 
                 if (action === 'push-changes') {
-                    pushPendingChanges(button);
+                    pushPendingChanges(button, originalHTML);
                 } else if (action === 'full-sync') {
-                    fullSync(button);
+                    fullSync(button, originalHTML);
                 }
             }
         });
@@ -126,7 +126,7 @@
         });
 
         // Refactored sync action function to reduce duplication
-        async function syncAction(endpoint, successMessage, button) {
+        async function syncAction(endpoint, successMessage, button, originalHTML) {
             try {
                 const resp = await fetch(endpoint, { method: 'POST' });
                 const data = await resp.json();
@@ -146,21 +146,17 @@
                 // Reset button state
                 if (button) {
                     button.disabled = false;
-                    if (endpoint === '/api/sync/push') {
-                        button.innerHTML = '<svg class="w-4 h-4 inline mr-2" fill="none" stroke="currentColor" viewBox="0 0 24 24"><path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M4 4v5h.582m15.356 2A8.001 8.001 0 004.582 9m0 0H9m11 11v-5h-.581m0 0a8.003 8.003 0 01-15.357-2m15.357 2H15"></path></svg> Push changes';
-                    } else {
-                        button.innerHTML = '<svg class="w-4 h-4 inline mr-2" fill="none" stroke="currentColor" viewBox="0 0 24 24"><path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M4 4v5h.582m15.356 2A8.001 8.001 0 004.582 9m0 0H9m11 11v-5h-.581m0 0a8.003 8.003 0 01-15.357-2m15.357 2H15"></path></svg> Full sync';
-                    }
+                    button.innerHTML = originalHTML;
                 }
             }
         }
 
-        function pushPendingChanges(button) {
-            syncAction('/api/sync/push', 'Изменения синхронизированы', button);
+        function pushPendingChanges(button, originalHTML) {
+            syncAction('/api/sync/push', 'Изменения синхронизированы', button, originalHTML);
         }
 
-        function fullSync(button) {
-            syncAction('/api/sync/all', 'Полная синхронизация завершена', button);
+        function fullSync(button, originalHTML) {
+            syncAction('/api/sync/all', 'Полная синхронизация завершена', button, originalHTML);
         }
 
         async function checkDbStatus() {