[FIX] add safety check to prevent reload on wrong page after navigation

Author: emjay0921

spp_custom_fields_ui/CHANGELOG.md

@@ -2,10 +2,17 @@
 
 ## 2025-11-20
 
+### 2025-11-20 10:24:55 - [FIX] add safety check to prevent reload on wrong page after navigation
+
+- Added URL verification before executing scheduled page reload
+- Prevents reload on wrong page if user navigates away during 100ms timeout
+- Only reloads if user is still on ir.model.fields page
+- Protects against data loss on unrelated pages
+
 ### 2025-11-20 10:12:34 - [FIX] remove meaningless return after page reload in custom fields UI
+
 - Fixed bug where return statement after window.location.reload() was unreachable
 - For existing records, reload happens immediately before return, destroying page context
 - For new records, setTimeout creates race condition where return value is meaningless
 - Added proper handling for result === false case without reload
 - Added comments explaining control flow in reload scenarios
-

spp_custom_fields_ui/static/src/js/custom_fields_ui_reload.js

@@ -29,7 +29,11 @@ patch(FormController.prototype, {
                     // For new records, wait a bit for URL to update, then reload
                     // This ensures the URL contains the new record ID
                     setTimeout(() => {
-                        window.location.reload();
+                        // Safety check: only reload if still on ir.model.fields page
+                        // Prevents unwanted reloads if user navigated away during timeout
+                        if (window.location.href.includes("ir.model.fields")) {
+                            window.location.reload();
+                        }
                     }, 100);
                     // Don't return - page will reload soon, making return value meaningless
                 } else {