Remove deprecated usage

Author: laech

.gitignore

@@ -8,3 +8,4 @@
 .idea
 .gradle
 .intellijPlatform
+.kotlin

build.gradle.kts

@@ -17,7 +17,7 @@ repositories {
 }
 
 dependencies {
-  intellijPlatform { intellijIdeaCommunity("2024.1") }
+  intellijPlatform { intellijIdeaCommunity("2024.3") }
   testImplementation("org.mockito:mockito-core:3.11.2")
   testImplementation("org.mockito.kotlin:mockito-kotlin:3.2.0")
   testImplementation("nz.lae.stacksrc:stacksrc-junit5:0.5.1")

src/main/kotlin/com/gitlab/lae/intellij/actions/tree/Popup.kt

@@ -7,6 +7,7 @@ import com.intellij.ide.DataManager
 import com.intellij.openapi.actionSystem.ActionGroup
 import com.intellij.openapi.actionSystem.ActionManager
 import com.intellij.openapi.actionSystem.ActionPlaces.ACTION_PLACE_QUICK_LIST_POPUP_ACTION
+import com.intellij.openapi.actionSystem.ActionUiKind
 import com.intellij.openapi.actionSystem.AnAction
 import com.intellij.openapi.actionSystem.AnActionEvent
 import com.intellij.openapi.actionSystem.CommonDataKeys.EDITOR
@@ -129,12 +130,13 @@ internal class Popup(action: ActionNode, e: AnActionEvent) {
   private fun performAction(action: AnAction, modifiers: Int, dataContext: DataContext) {
     val event =
       AnActionEvent(
-        null,
         dataContext,
-        ACTION_PLACE_QUICK_LIST_POPUP_ACTION,
         action.templatePresentation.clone(),
-        actionManager,
+        ACTION_PLACE_QUICK_LIST_POPUP_ACTION,
+        ActionUiKind.POPUP,
+        null,
         modifiers,
+        actionManager,
       )
 
     event.setInjectedContext(action.isInInjectedContext)

src/main/kotlin/com/gitlab/lae/intellij/actions/tree/app/RootAction.kt

@@ -27,10 +27,6 @@ data class RootAction(
     findAction(e)?.actionPerformed(e)
   }
 
-  override fun beforeActionPerformedUpdate(e: AnActionEvent) {
-    findAction(e)?.beforeActionPerformedUpdate(e)
-  }
-
   override fun update(e: AnActionEvent) {
     /* It's important to disable this action when none inner action
      * is available to execute, so that the keystroke will be passed

src/main/kotlin/com/gitlab/lae/intellij/actions/tree/ui/ActionPresentation.kt

@@ -3,6 +3,7 @@ package com.gitlab.lae.intellij.actions.tree.ui
 import com.gitlab.lae.intellij.actions.tree.util.setEnabledModalContext
 import com.intellij.openapi.actionSystem.ActionManager
 import com.intellij.openapi.actionSystem.ActionPlaces.ACTION_PLACE_QUICK_LIST_POPUP_ACTION
+import com.intellij.openapi.actionSystem.ActionUiKind
 import com.intellij.openapi.actionSystem.AnAction
 import com.intellij.openapi.actionSystem.AnActionEvent
 import com.intellij.openapi.actionSystem.AnActionHolder
@@ -75,12 +76,13 @@ data class ActionPresentation(
   fun update(actionManager: ActionManager, dataContext: DataContext) {
     val event =
       AnActionEvent(
-        null,
         dataContext,
-        ACTION_PLACE_QUICK_LIST_POPUP_ACTION,
         presentation,
-        actionManager,
+        ACTION_PLACE_QUICK_LIST_POPUP_ACTION,
+        ActionUiKind.POPUP,
+        null,
         0,
+        actionManager,
       )
     event.setInjectedContext(value.isInInjectedContext)
     performDumbAwareUpdate(value, event, false)

src/test/kotlin/com/gitlab/lae/intellij/actions/tree/Actions.kt

@@ -2,6 +2,7 @@ package com.gitlab.lae.intellij.actions.tree
 
 import com.gitlab.lae.intellij.actions.tree.When.Always
 import com.intellij.openapi.actionSystem.ActionManager
+import com.intellij.openapi.actionSystem.ActionUiKind
 import com.intellij.openapi.actionSystem.AnActionEvent
 import com.intellij.openapi.actionSystem.DataContext
 import com.intellij.openapi.actionSystem.Presentation
@@ -30,10 +31,11 @@ fun actionEvent(
   mask: Int = 0,
 ) =
   AnActionEvent(
-    input,
     context,
-    place,
     presentation,
-    actionManager,
+    place,
+    ActionUiKind.NONE,
+    input,
     mask,
+    actionManager,
   )

src/test/kotlin/com/gitlab/lae/intellij/actions/tree/app/RootActionTest.kt

@@ -92,9 +92,9 @@ class RootActionTest {
 
   @Test
   fun `merges root actions`() {
-    val cut = EmptyAction("Cut", null, null)
-    val copy = EmptyAction("Copy", null, null)
-    val paste = EmptyAction("Paste", null, null)
+    val cut = EmptyAction()
+    val copy = EmptyAction()
+    val paste = EmptyAction()
 
     val actionManager = mock<ActionManager>()
     whenever(actionManager.getAction(ACTION_CUT)).thenReturn(cut)
@@ -140,9 +140,9 @@ class RootActionTest {
 
   @Test
   fun `merging maintains action with no key strokes`() {
-    val cut = EmptyAction("Cut", null, null)
-    val copy = EmptyAction("Copy", null, null)
-    val paste = EmptyAction("Paste", null, null)
+    val cut = EmptyAction()
+    val copy = EmptyAction()
+    val paste = EmptyAction()
     val actionManager = mock<ActionManager>()
     whenever(actionManager.getAction(ACTION_CUT)).thenReturn(cut)
     whenever(actionManager.getAction(ACTION_COPY)).thenReturn(copy)