From 3d1c42c6f7662228c8f6d2ed50d2f07d78bbc46b Mon Sep 17 00:00:00 2001
From: Robert Resch <robert@resch.dev>
Date: Wed, 16 Dec 2020 16:37:35 +0000
Subject: [PATCH] reload page done on success callback

---
 public/js/grocy.js | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/public/js/grocy.js b/public/js/grocy.js
index a4d001f9..072c87be 100644
--- a/public/js/grocy.js
+++ b/public/js/grocy.js
@@ -454,14 +454,17 @@ Grocy.FrontendHelpers.SaveUserSetting = function(settingsKey, value)
 	);
 }
 
-Grocy.FrontendHelpers.DeleteUserSetting = function(settingsKey)
+Grocy.FrontendHelpers.DeleteUserSetting = function(settingsKey, reloadPageOnSuccess = false)
 {
 	delete Grocy.UserSettings[settingsKey];
 
 	Grocy.Api.Delete('user/settings/' + settingsKey, {},
 		function(result)
 		{
-			// Nothing to do...
+			if (reloadPageOnSuccess)
+			{
+				location.reload();
+			}
 		},
 		function(xhr)
 		{
@@ -739,7 +742,8 @@ $.extend(true, $.fn.dataTable.defaults, {
 		var settingKey = 'datatables_state_' + settings.sTableId;
 		if ($.isEmptyObject(data))
 		{
-			Grocy.FrontendHelpers.DeleteUserSetting(settingKey);
+			//state.clear was called and unfortunately the table is not refresh, so we are reloading the page
+			Grocy.FrontendHelpers.DeleteUserSetting(settingKey, true);
 		} else
 		{
 			var stateData = JSON.stringify(data);
@@ -1017,9 +1021,6 @@ $(".change-table-columns-visibility-button").on("click", function(e)
 
 								//Delete state settings
 								dataTable.state.clear();
-
-								//Reload page as datatable is not reseting itself
-								location.reload();
 							}
 							bootbox.hideAll();
 						}