Skip to content

Commit

Permalink
fix: update actions to avoid exception "ActionUpdateThread.OLD_EDT is…
Browse files Browse the repository at this point in the history
… deprecated and going to be removed soon." with 2024.1 eap
  • Loading branch information
jansorg authored and ahtrotta committed Mar 6, 2024
1 parent cc54ec4 commit d6d18ca
Show file tree
Hide file tree
Showing 12 changed files with 27 additions and 16 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,13 @@
import com.intellij.ide.BrowserUtil;
import com.intellij.openapi.actionSystem.AnAction;
import com.intellij.openapi.actionSystem.AnActionEvent;
import com.intellij.openapi.actionSystem.UpdateInBackground;
import com.intellij.openapi.project.DumbAware;
import org.jetbrains.annotations.NotNull;

import javax.swing.*;

public class AppMapDocumentationAction extends AnAction implements DumbAware {
public class AppMapDocumentationAction extends AnAction implements DumbAware, UpdateInBackground {
public AppMapDocumentationAction() {
super(Icons.APPMAP_DOCS);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
import com.intellij.ide.actions.OpenInRightSplitAction;
import com.intellij.openapi.actionSystem.AnAction;
import com.intellij.openapi.actionSystem.AnActionEvent;
import com.intellij.openapi.actionSystem.UpdateInBackground;
import com.intellij.openapi.application.WriteAction;
import com.intellij.openapi.diagnostic.Logger;
import com.intellij.openapi.fileEditor.FileEditorManager;
Expand All @@ -32,7 +33,7 @@
/**
* Generates an OpenAPI file using the CLI tool.
*/
public class GenerateOpenApiAction extends AnAction implements DumbAware {
public class GenerateOpenApiAction extends AnAction implements DumbAware, UpdateInBackground {
private static final Logger LOG = Logger.getInstance(GenerateOpenApiAction.class);
private static final String APPMAP_OPENAPI_FILENAME = "appmap-openapi.yml";

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,15 @@
import appland.webviews.navie.NavieEditorProvider;
import com.intellij.openapi.actionSystem.AnAction;
import com.intellij.openapi.actionSystem.AnActionEvent;
import com.intellij.openapi.actionSystem.UpdateInBackground;
import com.intellij.openapi.project.DumbAware;
import org.jetbrains.annotations.NotNull;

/**
* Action to open the AppMap Navie webview.
*/
@SuppressWarnings("ComponentNotRegistered")
public class OpenAppMapNavieAction extends AnAction implements DumbAware {
public class OpenAppMapNavieAction extends AnAction implements DumbAware, UpdateInBackground {
@Override
public void actionPerformed(@NotNull AnActionEvent e) {
var project = e.getProject();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,13 @@
import appland.installGuide.InstallGuideViewPage;
import com.intellij.openapi.actionSystem.AnAction;
import com.intellij.openapi.actionSystem.AnActionEvent;
import com.intellij.openapi.actionSystem.UpdateInBackground;
import com.intellij.openapi.project.DumbAware;
import org.jetbrains.annotations.NotNull;

import java.util.Objects;

public class OpenInstallGuideAction extends AnAction {
public class OpenInstallGuideAction extends AnAction implements DumbAware, UpdateInBackground {
@Override
public void update(@NotNull AnActionEvent e) {
e.getPresentation().setEnabled(InstallGuideEditorProvider.isSupported());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
import appland.index.AppMapSearchScopes;
import com.intellij.openapi.actionSystem.AnAction;
import com.intellij.openapi.actionSystem.AnActionEvent;
import com.intellij.openapi.actionSystem.UpdateInBackground;
import com.intellij.openapi.diagnostic.Logger;
import com.intellij.openapi.fileEditor.FileEditorManager;
import com.intellij.openapi.ui.Messages;
Expand All @@ -18,7 +19,7 @@
* If two files have the same modification timestamp, then the returned file is randomly chosen,
* depending on the order in the index.
*/
public class OpenRecentAppMapAction extends AnAction {
public class OpenRecentAppMapAction extends AnAction implements UpdateInBackground {
private static final Logger LOG = Logger.getInstance("#appmap.action");

static VirtualFile findMostRecentlyModifiedAppMap(com.intellij.openapi.project.Project project) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,14 +8,15 @@
import com.intellij.notification.NotificationType;
import com.intellij.openapi.actionSystem.AnAction;
import com.intellij.openapi.actionSystem.AnActionEvent;
import com.intellij.openapi.actionSystem.UpdateInBackground;
import com.intellij.openapi.progress.ProgressIndicator;
import com.intellij.openapi.progress.Task;
import com.intellij.openapi.project.DumbAware;
import org.jetbrains.annotations.NotNull;

import static appland.AppMapBundle.get;

public class StartAppMapRecordingAction extends AnAction implements DumbAware {
public class StartAppMapRecordingAction extends AnAction implements DumbAware, UpdateInBackground {
public StartAppMapRecordingAction() {
super(Icons.START_RECORDING_ACTION);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
import com.intellij.notification.NotificationType;
import com.intellij.openapi.actionSystem.AnAction;
import com.intellij.openapi.actionSystem.AnActionEvent;
import com.intellij.openapi.actionSystem.UpdateInBackground;
import com.intellij.openapi.application.ApplicationManager;
import com.intellij.openapi.application.ModalityState;
import com.intellij.openapi.application.ReadAction;
Expand All @@ -33,7 +34,7 @@
import java.util.Objects;
import java.util.concurrent.atomic.AtomicReference;

public class StopAppMapRecordingAction extends AnAction implements DumbAware {
public class StopAppMapRecordingAction extends AnAction implements DumbAware, UpdateInBackground {
private static final Logger LOG = Logger.getInstance(StopAppMapRecordingAction.class);

public StopAppMapRecordingAction() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,10 @@

import com.intellij.openapi.actionSystem.AnAction;
import com.intellij.openapi.actionSystem.AnActionEvent;
import com.intellij.openapi.actionSystem.UpdateInBackground;
import org.jetbrains.annotations.NotNull;

public class ThrowTestException extends AnAction {
public class ThrowTestException extends AnAction implements UpdateInBackground {
@Override
public void actionPerformed(@NotNull AnActionEvent e) {
throw new IllegalStateException("AppMap test exception, " + Math.random());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,14 @@
import appland.settings.AppMapApplicationSettingsService;
import com.intellij.openapi.actionSystem.AnAction;
import com.intellij.openapi.actionSystem.AnActionEvent;
import com.intellij.openapi.actionSystem.UpdateInBackground;
import com.intellij.openapi.application.ApplicationManager;
import com.intellij.openapi.diagnostic.Logger;
import com.intellij.openapi.project.DumbAware;
import org.jetbrains.annotations.NotNull;

@SuppressWarnings("ComponentNotRegistered")
public class AppMapLoginAction extends AnAction {
public class AppMapLoginAction extends AnAction implements DumbAware, UpdateInBackground {
@Override
public void actionPerformed(@NotNull AnActionEvent e) {
authenticate();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
import appland.settings.AppMapApplicationSettingsService;
import com.intellij.openapi.actionSystem.AnAction;
import com.intellij.openapi.actionSystem.AnActionEvent;
import com.intellij.openapi.actionSystem.UpdateInBackground;
import com.intellij.openapi.project.DumbAware;
import com.intellij.openapi.project.Project;
import com.intellij.openapi.ui.InputValidator;
Expand All @@ -25,7 +26,7 @@
/**
* Action to sign in to AppMap by providing a license key.
*/
public class AppMapLoginByKeyAction extends AnAction implements DumbAware {
public class AppMapLoginByKeyAction extends AnAction implements DumbAware, UpdateInBackground {
@Override
public void actionPerformed(@NotNull AnActionEvent e) {
var project = Objects.requireNonNull(e.getProject());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,12 @@
import appland.settings.AppMapApplicationSettingsService;
import com.intellij.openapi.actionSystem.AnAction;
import com.intellij.openapi.actionSystem.AnActionEvent;
import com.intellij.openapi.actionSystem.UpdateInBackground;
import com.intellij.openapi.project.DumbAware;
import org.jetbrains.annotations.NotNull;

@SuppressWarnings("ComponentNotRegistered")
public class AppMapLogoutAction extends AnAction {
public class AppMapLogoutAction extends AnAction implements DumbAware, UpdateInBackground {
@Override
public void actionPerformed(@NotNull AnActionEvent e) {
logout();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,18 +3,15 @@
import appland.AppMapBundle;
import com.intellij.icons.AllIcons;
import com.intellij.ide.DeleteProvider;
import com.intellij.openapi.actionSystem.AnAction;
import com.intellij.openapi.actionSystem.AnActionEvent;
import com.intellij.openapi.actionSystem.CommonDataKeys;
import com.intellij.openapi.actionSystem.DataContext;
import com.intellij.openapi.actionSystem.*;
import com.intellij.openapi.fileChooser.actions.VirtualFileDeleteProvider;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;

/**
* Action to delete all AppMaps displayed in the AppMap panel.
*/
final class DeleteAllMapsAction extends AnAction {
final class DeleteAllMapsAction extends AnAction implements UpdateInBackground {
private final DeleteProvider deleteHandler = new VirtualFileDeleteProvider();

public DeleteAllMapsAction() {
Expand Down

0 comments on commit d6d18ca

Please sign in to comment.