From 80e70f4980d2447dd47bc9d565e4b659eb9ae9b4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E9=9B=A8?= <luotianyi@luotianyi.me>
Date: Fri, 27 Sep 2024 02:07:35 +0800
Subject: [PATCH] fix: reset admin page save button in catch (#3963)

---
 framework/core/js/src/admin/components/AdminPage.tsx | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/framework/core/js/src/admin/components/AdminPage.tsx b/framework/core/js/src/admin/components/AdminPage.tsx
index 4d1d47add..460d9037e 100644
--- a/framework/core/js/src/admin/components/AdminPage.tsx
+++ b/framework/core/js/src/admin/components/AdminPage.tsx
@@ -351,6 +351,13 @@ export default abstract class AdminPage<CustomAttrs extends IPageAttrs = IPageAt
     app.alerts.show({ type: 'success' }, app.translator.trans('core.admin.settings.saved_message'));
   }
 
+  /**
+   * Called when `saveSettings` completes with errors.
+   */
+  onsavefailed(): void {
+    this.loading = false;
+  }
+
   /**
    * Returns a function that fetches the setting from the `app` global.
    */
@@ -394,6 +401,6 @@ export default abstract class AdminPage<CustomAttrs extends IPageAttrs = IPageAt
 
     this.loading = true;
 
-    return saveSettings(this.dirty()).then(this.onsaved.bind(this));
+    return saveSettings(this.dirty()).then(this.onsaved.bind(this)).catch(this.onsavefailed.bind(this));
   }
 }