From 91b88aed85cc37f0b2f01cbd92327573dea7ef0b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Thu, 28 Aug 2025 19:20:03 +0200
Subject: [PATCH] Prevent the CSV import and export windows from being clicked
 on multiple times

Closes #14205
---
 sao/src/window.js                      | 32 ++++++++++++++++----------
 tryton/tryton/gui/window/win_csv.py    |  6 ++++-
 tryton/tryton/gui/window/win_export.py |  1 +
 tryton/tryton/gui/window/win_import.py |  1 +
 4 files changed, 27 insertions(+), 13 deletions(-)

diff --git a/sao/src/window.js b/sao/src/window.js
index f5829fd1e6..f9f4a5c26f 100644
--- a/sao/src/window.js
+++ b/sao/src/window.js
@@ -1466,17 +1466,16 @@
                 'class': 'btn btn-link',
                 'type': 'button',
                 'title': Sao.i18n.gettext("Cancel"),
-            }).text(Sao.i18n.gettext("Cancel")).click(() => {
-                this.response('RESPONSE_CANCEL');
+            }).text(Sao.i18n.gettext("Cancel")).on('click', (evt) => {
+                this.response(evt, 'RESPONSE_CANCEL');
             }).appendTo(this.dialog.footer);
 
             jQuery('<button/>', {
                 'class': 'btn btn-primary',
                 'type': 'submit',
                 'title': Sao.i18n.gettext("Import"),
-            }).text(Sao.i18n.gettext("Import")).click(e => {
-                e.preventDefault();
-                this.response('RESPONSE_OK');
+            }).text(Sao.i18n.gettext("Import")).on('click', (evt) => {
+                this.response(evt, 'RESPONSE_OK');
             }).appendTo(this.dialog.footer);
 
             jQuery('<button/>', {
@@ -1758,7 +1757,10 @@
                 }
             }
         },
-        response: function(response_id) {
+        response: function(evt, response_id) {
+            evt.preventDefault();
+            let button = jQuery(evt.currentTarget);
+            button.prop('disabled', true);
             if(response_id == 'RESPONSE_OK') {
                 var fields = [];
                 this.fields_selected.children('li').each((i, field_el) => {
@@ -1768,6 +1770,8 @@
                 if(fname) {
                     this.import_csv(fname, fields).then(() => {
                         this.destroy();
+                    }, () => {
+                        button.prop('disabled', false);
                     });
                 } else {
                     this.destroy();
@@ -1821,17 +1825,16 @@
                 'class': 'btn btn-link',
                 'type': 'button',
                 'title': Sao.i18n.gettext("Close"),
-            }).text(Sao.i18n.gettext("Close")).click(() => {
-                this.response('RESPONSE_CLOSE');
+            }).text(Sao.i18n.gettext("Close")).on('click', (evt) => {
+                this.response(evt, 'RESPONSE_CLOSE');
             }).appendTo(this.dialog.footer);
 
             jQuery('<button/>', {
                 'class': 'btn btn-primary',
                 'type': 'submit',
                 'title': Sao.i18n.gettext("Save As..."),
-            }).text(Sao.i18n.gettext("Save As...")).click(e => {
-                e.preventDefault();
-                this.response('RESPONSE_OK');
+            }).text(Sao.i18n.gettext("Save As...")).on('click', (evt) => {
+                this.response(evt, 'RESPONSE_OK');
             }).appendTo(this.dialog.footer);
 
             this.info_bar = new Sao.Window.InfoBar();
@@ -2286,7 +2289,10 @@
                 Sao.common.ICONFACTORY.get_icon_img('tryton-drag')
             ).appendTo(this.fields_selected);
         },
-        response: function(response_id) {
+        response: function(evt, response_id) {
+            evt.preventDefault();
+            let button = jQuery(evt.currentTarget);
+            button.prop('disabled', true);
             this.info_bar.clear();
             if(response_id == 'RESPONSE_OK') {
                 var fields = [];
@@ -2339,6 +2345,8 @@
                 }
                 prm.then(data => {
                     this.export_csv(data, paths, header);
+                }).always(() => {
+                    button.prop('disabled', false);
                 });
             } else {
                 this.destroy();
diff --git a/tryton/tryton/gui/window/win_csv.py b/tryton/tryton/gui/window/win_csv.py
index 360a3d4892..b37c530516 100644
--- a/tryton/tryton/gui/window/win_csv.py
+++ b/tryton/tryton/gui/window/win_csv.py
@@ -5,7 +5,7 @@ import locale
 import os
 import sys
 
-from gi.repository import Gdk, GObject, Gtk
+from gi.repository import Gdk, GLib, GObject, Gtk
 
 from tryton.common import IconFactory, RPCExecute
 from tryton.config import TRYTON_ICON
@@ -318,3 +318,7 @@ class WinCSV(NoModal):
 
     def hide(self):
         self.dialog.hide()
+
+    def response(self, dialog, response):
+        dialog.set_sensitive(False)
+        GLib.idle_add(lambda: dialog.set_sensitive(True) or False)
diff --git a/tryton/tryton/gui/window/win_export.py b/tryton/tryton/gui/window/win_export.py
index 6f283b133c..1e728a907c 100644
--- a/tryton/tryton/gui/window/win_export.py
+++ b/tryton/tryton/gui/window/win_export.py
@@ -360,6 +360,7 @@ class WinExport(WinCSV, InfoBar):
         self.model2.append((string_, name))
 
     def response(self, dialog, response):
+        super().response(dialog, response)
         self.info_bar_clear()
         if response in {Gtk.ResponseType.OK, Gtk.ResponseType.ACCEPT}:
             fields = []
diff --git a/tryton/tryton/gui/window/win_import.py b/tryton/tryton/gui/window/win_import.py
index dd63567058..4cdc898044 100644
--- a/tryton/tryton/gui/window/win_import.py
+++ b/tryton/tryton/gui/window/win_import.py
@@ -200,6 +200,7 @@ class WinImport(WinCSV):
         self.model2.clear()
 
     def response(self, dialog, response):
+        super().response(dialog, response)
         if response == Gtk.ResponseType.OK:
             fields = []
             iter = self.model2.get_iter_first()