Fix #103 #105: Defined action update thread for actions

leinardi · Oct 4, 2024 · 80198ec · 80198ec
1 parent 1801d4a
commit 80198ec
Show file tree

Hide file tree

Showing 18 changed files with 80 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,6 @@
 **[0.16.3] 2024-10-04**
 
+- Defined action update thread for actions
 - New: Min IDEA version raised from PC-2022.1.4 to PC-2023.3.6
 - Several dependency updates
 

diff --git a/build.gradle b/build.gradle
@@ -64,8 +64,8 @@ gradle.projectsEvaluated {
 group 'com.leinardi.pycharm'
 version version
 
-sourceCompatibility = JavaVersion.VERSION_11
-targetCompatibility = JavaVersion.VERSION_11
+sourceCompatibility = JavaVersion.VERSION_17
+targetCompatibility = JavaVersion.VERSION_17
 
 intellij {
     version = ideVersion

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/BaseAction.java b/src/main/java/com/leinardi/pycharm/pylint/actions/BaseAction.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.actionSystem.Presentation;
 import com.intellij.openapi.diagnostic.Logger;
@@ -42,6 +43,11 @@ public abstract class BaseAction extends DumbAwareAction {
 
     private static final Logger LOG = Logger.getInstance(BaseAction.class);
 
+    @Override
+    public @NotNull ActionUpdateThread getActionUpdateThread() {
+        return ActionUpdateThread.EDT;
+    }
+
     @Override
     public void update(final @NotNull AnActionEvent event) {
         try {

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/Close.java b/src/main/java/com/leinardi/pycharm/pylint/actions/Close.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import org.jetbrains.annotations.NotNull;
 

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/CollapseAll.java b/src/main/java/com/leinardi/pycharm/pylint/actions/CollapseAll.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.leinardi.pycharm.pylint.toolwindow.PylintToolWindowPanel;
 import org.jetbrains.annotations.NotNull;

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/DisplayConvention.java b/src/main/java/com/leinardi/pycharm/pylint/actions/DisplayConvention.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.project.DumbAwareToggleAction;
 import com.intellij.openapi.project.Project;
@@ -33,6 +34,11 @@
  */
 public class DisplayConvention extends DumbAwareToggleAction {
 
+    @Override
+    public @NotNull ActionUpdateThread getActionUpdateThread() {
+        return ActionUpdateThread.EDT;
+    }
+
     @Override
     public boolean isSelected(final @NotNull AnActionEvent event) {
         final Project project = getEventProject(event);

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/DisplayErrors.java b/src/main/java/com/leinardi/pycharm/pylint/actions/DisplayErrors.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.project.DumbAwareToggleAction;
 import com.intellij.openapi.project.Project;
@@ -33,6 +34,11 @@
  */
 public class DisplayErrors extends DumbAwareToggleAction {
 
+    @Override
+    public @NotNull ActionUpdateThread getActionUpdateThread() {
+        return ActionUpdateThread.EDT;
+    }
+
     @Override
     public boolean isSelected(final @NotNull AnActionEvent event) {
         final Project project = getEventProject(event);

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/DisplayInfo.java b/src/main/java/com/leinardi/pycharm/pylint/actions/DisplayInfo.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.project.DumbAwareToggleAction;
 import com.intellij.openapi.project.Project;
@@ -33,6 +34,11 @@
  */
 public class DisplayInfo extends DumbAwareToggleAction {
 
+    @Override
+    public @NotNull ActionUpdateThread getActionUpdateThread() {
+        return ActionUpdateThread.EDT;
+    }
+
     @Override
     public boolean isSelected(final @NotNull AnActionEvent event) {
         final Project project = getEventProject(event);

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/DisplayRefactor.java b/src/main/java/com/leinardi/pycharm/pylint/actions/DisplayRefactor.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.project.DumbAwareToggleAction;
 import com.intellij.openapi.project.Project;
@@ -33,6 +34,11 @@
  */
 public class DisplayRefactor extends DumbAwareToggleAction {
 
+    @Override
+    public @NotNull ActionUpdateThread getActionUpdateThread() {
+        return ActionUpdateThread.EDT;
+    }
+
     @Override
     public boolean isSelected(final @NotNull AnActionEvent event) {
         final Project project = getEventProject(event);

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/DisplayWarnings.java b/src/main/java/com/leinardi/pycharm/pylint/actions/DisplayWarnings.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.project.DumbAwareToggleAction;
 import com.intellij.openapi.project.Project;
@@ -33,6 +34,11 @@
  */
 public class DisplayWarnings extends DumbAwareToggleAction {
 
+    @Override
+    public @NotNull ActionUpdateThread getActionUpdateThread() {
+        return ActionUpdateThread.EDT;
+    }
+
     @Override
     public boolean isSelected(final @NotNull AnActionEvent event) {
         final Project project = getEventProject(event);

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/ExpandAll.java b/src/main/java/com/leinardi/pycharm/pylint/actions/ExpandAll.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.leinardi.pycharm.pylint.toolwindow.PylintToolWindowPanel;
 import org.jetbrains.annotations.NotNull;

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/ScanCurrentChangeList.java b/src/main/java/com/leinardi/pycharm/pylint/actions/ScanCurrentChangeList.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.actionSystem.Presentation;
 import com.intellij.openapi.diagnostic.Logger;
@@ -39,6 +40,11 @@ public class ScanCurrentChangeList extends BaseAction {
 
     private static final Logger LOG = Logger.getInstance(ScanCurrentChangeList.class);
 
+    @Override
+    public @NotNull ActionUpdateThread getActionUpdateThread() {
+        return ActionUpdateThread.BGT;
+    }
+
     @Override
     public final void actionPerformed(final @NotNull AnActionEvent event) {
         project(event).ifPresent(project -> {

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/ScanCurrentFile.java b/src/main/java/com/leinardi/pycharm/pylint/actions/ScanCurrentFile.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.actionSystem.Presentation;
 import com.intellij.openapi.editor.Editor;
@@ -37,6 +38,11 @@
  */
 public class ScanCurrentFile extends BaseAction {
 
+    @Override
+    public @NotNull ActionUpdateThread getActionUpdateThread() {
+        return ActionUpdateThread.BGT;
+    }
+
     @Override
     public void actionPerformed(final @NotNull AnActionEvent event) {
         project(event).ifPresent(project -> {

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/ScanModifiedFiles.java b/src/main/java/com/leinardi/pycharm/pylint/actions/ScanModifiedFiles.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.actionSystem.Presentation;
 import com.intellij.openapi.diagnostic.Logger;
@@ -36,6 +37,11 @@ public class ScanModifiedFiles extends BaseAction {
 
     private static final Logger LOG = Logger.getInstance(ScanModifiedFiles.class);
 
+    @Override
+    public @NotNull ActionUpdateThread getActionUpdateThread() {
+        return ActionUpdateThread.BGT;
+    }
+
     @Override
     public final void actionPerformed(final @NotNull AnActionEvent event) {
         project(event).ifPresent(project -> {

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/ScanModule.java b/src/main/java/com/leinardi/pycharm/pylint/actions/ScanModule.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.actionSystem.Presentation;
 import com.intellij.openapi.application.ReadAction;
@@ -39,6 +40,11 @@
  */
 public class ScanModule extends BaseAction {
 
+    @Override
+    public @NotNull ActionUpdateThread getActionUpdateThread() {
+        return ActionUpdateThread.BGT;
+    }
+
     @Override
     public final void actionPerformed(final @NotNull AnActionEvent event) {
         project(event).ifPresent(project -> {

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/ScanProject.java b/src/main/java/com/leinardi/pycharm/pylint/actions/ScanProject.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.actionSystem.Presentation;
 import com.intellij.openapi.application.ReadAction;
@@ -33,6 +34,11 @@
  */
 public class ScanProject extends BaseAction {
 
+    @Override
+    public @NotNull ActionUpdateThread getActionUpdateThread() {
+        return ActionUpdateThread.BGT;
+    }
+
     @Override
     public void actionPerformed(final @NotNull AnActionEvent event) {
         project(event).ifPresent(project -> {

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/ScrollToSource.java b/src/main/java/com/leinardi/pycharm/pylint/actions/ScrollToSource.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.actionSystem.PlatformDataKeys;
 import com.intellij.openapi.project.DumbAwareToggleAction;
@@ -25,6 +26,7 @@
 import com.intellij.ui.content.Content;
 import com.leinardi.pycharm.pylint.PylintPlugin;
 import com.leinardi.pycharm.pylint.toolwindow.PylintToolWindowPanel;
+import org.jetbrains.annotations.NotNull;
 
 /**
  * Toggle the scroll to source setting.

diff --git a/src/main/java/com/leinardi/pycharm/pylint/actions/StopCheck.java b/src/main/java/com/leinardi/pycharm/pylint/actions/StopCheck.java
@@ -16,6 +16,7 @@
 
 package com.leinardi.pycharm.pylint.actions;
 
+import com.intellij.openapi.actionSystem.ActionUpdateThread;
 import com.intellij.openapi.actionSystem.AnActionEvent;
 import com.intellij.openapi.actionSystem.Presentation;
 import com.intellij.openapi.wm.ToolWindow;
@@ -29,6 +30,11 @@
  */
 public class StopCheck extends BaseAction {
 
+    @Override
+    public @NotNull ActionUpdateThread getActionUpdateThread() {
+        return ActionUpdateThread.BGT;
+    }
+
     @Override
     public void actionPerformed(final @NotNull AnActionEvent event) {
         project(event).ifPresent(project -> {