#22103 Tab away from datetime field doesn't work as

expected Fixed autosave support
dbeaver · Mar 22, 2024 · e639ba5 · e639ba5
1 parent be1e635
commit e639ba5
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/....data/src/org/jkiss/dbeaver/ui/controls/resultset/panel/valueviewer/ValueViewerPanel.java b/....data/src/org/jkiss/dbeaver/ui/controls/resultset/panel/valueviewer/ValueViewerPanel.java
@@ -281,9 +281,6 @@ private void viewValue(boolean forceRefresh)
 
             viewPlaceholder.layout();
         }
-        if (valueEditor instanceof BaseValueEditor) {
-            ((BaseValueEditor) valueEditor).setAutoSaveEnabled(false);
-        }
         if (valueEditor != null) {
             try {
                 Object newValue = previewController.getValue();
@@ -311,9 +308,6 @@ private void viewValue(boolean forceRefresh)
             }
             valueEditor.setDirty(false);
         }
-        if (valueEditor instanceof BaseValueEditor) {
-            ((BaseValueEditor) valueEditor).setAutoSaveEnabled(true);
-        }
     }
 
     private void handleTraverseEvent(TraverseEvent e) {

diff --git a/....jkiss.dbeaver.ui.editors.data/src/org/jkiss/dbeaver/ui/data/editors/BaseValueEditor.java b/....jkiss.dbeaver.ui.editors.data/src/org/jkiss/dbeaver/ui/data/editors/BaseValueEditor.java
@@ -196,6 +196,7 @@ protected void addInlineListeners(@NotNull Control inlineControl, @NotNull Liste
 
     private void addAutoSaveSupport(final Control inlineControl) {
         inlineControl.addDisposeListener(getAutoSaveListener());
+        this.setAutoSaveEnabled(true);
     }
 
     @NotNull