Merge pull request #133 from kdbinsidebrains/improvements

improvements

Author: smklimenko

build.gradle.kts

@@ -30,7 +30,7 @@ plugins {
     id("maven-publish")
     id("org.jetbrains.changelog") version "2.2.1"
     id("org.jetbrains.grammarkit") version "2022.3.2.2"
-    id("org.jetbrains.intellij.platform") version "2.5.0"
+    id("org.jetbrains.intellij.platform") version "2.7.0"
 }
 
 repositories {
@@ -84,8 +84,8 @@ dependencies {
         testFramework(TestFrameworkType.JUnit5)
     }
 
-    implementation("org.jfree:jfreechart:1.5.5")
-    implementation("org.jfree:org.jfree.svg:5.0.6")
+    implementation("org.jfree:jfreechart:1.5.6")
+    implementation("org.jfree:org.jfree.svg:5.0.7")
     implementation("org.apache.poi:poi:5.4.1")
     implementation("org.apache.poi:poi-ooxml:5.4.1")
 

src/main/java/org/kdb/inside/brains/view/console/table/QSchemaViewPanel.java

@@ -99,6 +99,13 @@ public JComponent getTargetComponent() {
     @Override
     public void dispose() {
         myTable.removePropertyChangeListener("columnModel", listener);
+        final int itemsCount = columnsFilterList.getItemsCount();
+        for (int i = 0; i < itemsCount; i++) {
+            final QColumnView item = columnsFilterList.getItemAt(i);
+            if (item != null) {
+                item.setVisible(true);
+            }
+        }
         columnsFilterList.clear();
     }
 
@@ -120,9 +127,13 @@ public SelectUnselectAction(String text, String description, Icon icon, boolean
 
         @Override
         public void actionPerformed(@NotNull AnActionEvent anActionEvent) {
-            for (QColumnView item : columnsFilterList.getAllItems()) {
-                item.setVisible(selected);
-                columnsFilterList.setItemSelected(item, selected);
+            final int itemsCount = columnsFilterList.getItemsCount();
+            for (int i = 0; i < itemsCount; i++) {
+                final QColumnView item = columnsFilterList.getItemAt(i);
+                if (item != null) {
+                    item.setVisible(selected);
+                    columnsFilterList.setItemSelected(item, selected);
+                }
             }
             columnsFilterList.repaint();
         }