Fix(VIM-3569): Pipe-symbol can no longer be used in remaps

JetBrains · Jul 30, 2024 · 5dc860f · 5dc860f
1 parent 956e726
commit 5dc860f
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/test/java/org/jetbrains/plugins/ideavim/ex/implementation/commands/MapCommandTest.kt b/src/test/java/org/jetbrains/plugins/ideavim/ex/implementation/commands/MapCommandTest.kt
@@ -298,6 +298,15 @@ class MapCommandTest : VimTestCase() {
     assertState("I found it in ${c}a legendary land")
   }
 
+  @Test
+  @TestFor(issues = ["VIM-3569"])
+  fun `test bar in mapping`() {
+    configureByText("${c}I found it in a legendary land")
+    typeText(commandToKeys("nmap <leader>\\| dw"))
+    typeText("<leader>|")
+    assertState("${c}found it in a legendary land")
+  }
+
   // VIM-676 |:map|
   @TestWithoutNeovim(reason = SkipNeovimReason.VIM_SCRIPT)
   @Test

diff --git a/...ine/src/main/kotlin/com/maddyhome/idea/vim/vimscript/model/commands/mapping/MapCommand.kt b/...ine/src/main/kotlin/com/maddyhome/idea/vim/vimscript/model/commands/mapping/MapCommand.kt
@@ -169,7 +169,7 @@ data class MapCommand(val range: Range, val argument: String, val cmd: String) :
         if (specialArgument != null) {
           specialArguments.add(specialArgument)
         } else {
-          fromKeys = injector.parser.parseKeys(part)
+          fromKeys = injector.parser.parseKeys(processBars(part))
         }
       }
     }
@@ -190,4 +190,8 @@ data class MapCommand(val range: Range, val argument: String, val cmd: String) :
       CommandArguments(specialArguments, it, toExpr, toKeysBuilder.toString().trimStart())
     }
   }
+
+  private fun processBars(fromString: String): String {
+    return fromString.replace("\\|", "|")
+  }
 }