fix #922

src/actions/actions.ts

@@ -2153,9 +2153,16 @@ export class PutCommandVisual extends BaseCommand {
   canBePrefixedWithDot = true;
 
   public async exec(position: Position, vimState: VimState, after: boolean = false): Promise<VimState> {
-    const result = await new DeleteOperator().run(vimState, vimState.cursorStartPosition, vimState.cursorPosition, false);
+    let start = vimState.cursorStartPosition;
+    let end = vimState.cursorPosition;
 
-    return await new PutCommand().exec(vimState.cursorStartPosition, result, true);
+    if (start.isAfter(end)) {
+      [start, end] = [end, start];
+    }
+
+    const result = await new DeleteOperator().run(vimState, start, end, false);
+
+    return await new PutCommand().exec(start, result, true);
   }
 
   // TODO - execWithCount

test/mode/modeNormal.test.ts

@@ -1095,6 +1095,13 @@ suite("Mode Normal", () => {
       end: ["abc abc |dhi"]
     });
 
+    newTest({
+      title: "can handle p with selection",
+      start: ["one", "two", "|three"],
+      keysPressed: "yykVkp",
+      end: ["|three", "three"]
+    });
+
     newTest({
       title: "can handle P with selection",
       start: ["|abc def ghi"],