From ebe5ac923f0a19df6d4f8d17e404160999288497 Mon Sep 17 00:00:00 2001
From: Christian Kuhn <lolli@schwarzbu.ch>
Date: Wed, 29 Nov 2017 18:06:54 +0100
Subject: [PATCH] [BUGFIX] Install tool: Racy clear tables
JS ajax call to reload stats must be called after
clear table finished and not in parallel.
Change-Id: Ifdaa6f89eafe042b5c57b130e189001b1ca0a3fe
Resolves: #83166
Releases: master
Reviewed-on: https://review.typo3.org/54869
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
---
.../install/Resources/Public/JavaScript/Modules/ClearTables.js | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/typo3/sysext/install/Resources/Public/JavaScript/Modules/ClearTables.js b/typo3/sysext/install/Resources/Public/JavaScript/Modules/ClearTables.js
index 21fc01d77cfd..4119181609bd 100644
--- a/typo3/sysext/install/Resources/Public/JavaScript/Modules/ClearTables.js
+++ b/typo3/sysext/install/Resources/Public/JavaScript/Modules/ClearTables.js
@@ -58,7 +58,6 @@ define([
var table = $(e.target).closest(self.selectorClearTrigger).data('table');
e.preventDefault();
self.clear(table);
- self.getStats();
});
},
@@ -107,6 +106,7 @@ define([
$.ajax({
url: Router.getUrl(),
method: 'POST',
+ context: this,
data: {
'install': {
'action': 'clearTablesClear',
@@ -126,6 +126,7 @@ define([
var message = FlashMessage.render(Severity.error, 'Something went wrong', '');
$outputContainer.append(message);
}
+ this.getStats();
},
error: function(xhr) {
Router.handleAjaxError(xhr);
--
GitLab