From 8d60d0abba96fa9e461a74403cf0faa4822bdeef Mon Sep 17 00:00:00 2001
From: Naumov Alexey <Iamemptyhuh@gmail.com>
Date: Thu, 12 May 2022 17:11:58 +0200
Subject: [PATCH] fix(plugin-resource-manager): CB-2019 1925 add sync throttle

---
 .../src/SaveScriptDialog.tsx                  |  5 +---
 .../src/SqlEditorTabResourceService.ts        | 25 ++++---------------
 2 files changed, 6 insertions(+), 24 deletions(-)

diff --git a/webapp/packages/plugin-resource-manager/src/SaveScriptDialog.tsx b/webapp/packages/plugin-resource-manager/src/SaveScriptDialog.tsx
index 0c5c17be8e..25299c01aa 100644
--- a/webapp/packages/plugin-resource-manager/src/SaveScriptDialog.tsx
+++ b/webapp/packages/plugin-resource-manager/src/SaveScriptDialog.tsx
@@ -8,7 +8,6 @@
 
 import { observable } from 'mobx';
 import { observer } from 'mobx-react-lite';
-import { useCallback } from 'react';
 import styled, { css } from 'reshadow';
 
 import { BASE_CONTAINERS_STYLES, Button, Container, InputField, SubmittingForm, useFocus, useObservableRef } from '@cloudbeaver/core-blocks';
@@ -39,9 +38,7 @@ export const SaveScriptDialog: DialogComponent<Payload, string> = observer(funct
     value: payload.defaultScriptName,
   }), {
     value: observable.ref,
-  }, {
-    payload,
-  });
+  }, false);
 
   return styled(useStyles(style, BASE_CONTAINERS_STYLES))(
     <CommonDialogWrapper
diff --git a/webapp/packages/plugin-sql-editor-navigation-tab-resource/src/SqlEditorTabResourceService.ts b/webapp/packages/plugin-sql-editor-navigation-tab-resource/src/SqlEditorTabResourceService.ts
index 1f909199a3..b7bb53e5b8 100644
--- a/webapp/packages/plugin-sql-editor-navigation-tab-resource/src/SqlEditorTabResourceService.ts
+++ b/webapp/packages/plugin-sql-editor-navigation-tab-resource/src/SqlEditorTabResourceService.ts
@@ -11,15 +11,15 @@ import { injectable } from '@cloudbeaver/core-di';
 import { NotificationService } from '@cloudbeaver/core-events';
 import { WindowEventsService } from '@cloudbeaver/core-root';
 import { ResourceKey, ResourceKeyUtils } from '@cloudbeaver/core-sdk';
+import { throttleAsync } from '@cloudbeaver/core-utils';
 import { NavResourceNodeService } from '@cloudbeaver/plugin-resource-manager';
 import { ISqlEditorTabState, SqlEditorService } from '@cloudbeaver/plugin-sql-editor';
 import { isSQLEditorTab, SqlEditorTabService } from '@cloudbeaver/plugin-sql-editor-navigation-tab';
 
+const SYNC_DELAY = 5 * 1000;
+
 @injectable()
 export class SqlEditorTabResourceService {
-  private synced = false;
-  private interval: NodeJS.Timer | null = null;
-
   constructor(
     private readonly navigationTabsService: NavigationTabsService,
     private readonly sqlEditorService: SqlEditorService,
@@ -35,26 +35,20 @@ export class SqlEditorTabResourceService {
 
     this.start = this.start.bind(this);
     this.stop = this.stop.bind(this);
+
+    this.syncTab = throttleAsync(this.syncTab, SYNC_DELAY);
   }
 
   start() {
     this.navTreeResource.onItemDelete.addHandler(this.onNodeDeleteHandler);
     this.navigationTabsService.onTabSelect.addHandler(this.onTabSelectHandler);
     this.windowEventsService.onFocusChange.addHandler(this.onFocusChangeHandler);
-
-    this.interval = setInterval(() => {
-      this.setSynced(false);
-    }, 15 * 1000);
   }
 
   stop() {
     this.navTreeResource.onItemDelete.removeHandler(this.onNodeDeleteHandler);
     this.navigationTabsService.onTabSelect.removeHandler(this.onTabSelectHandler);
     this.windowEventsService.onFocusChange.removeHandler(this.onFocusChangeHandler);
-
-    if (this.interval) {
-      clearInterval(this.interval);
-    }
   }
 
   private onFocusChangeHandler(focused: boolean) {
@@ -78,22 +72,13 @@ export class SqlEditorTabResourceService {
     });
   }
 
-  private setSynced(synced: boolean) {
-    this.synced = synced;
-  }
-
   private async syncCurrentTab() {
-    if (this.synced) {
-      return;
-    }
-
     const current = this.sqlEditorTabService.sqlEditorTabs.find(
       tab => tab.id === this.navigationTabsService.currentTabId
     );
 
     if (current) {
       await this.syncTab(current);
-      this.setSynced(true);
     }
   }