fixed PY-11654 Move Statement: IOE at com.intellij.openapi.editor.impl.DocumentImpl.d

python/src/com/jetbrains/python/codeInsight/editorActions/moveUpDown/PyStatementMover.java

@@ -103,7 +103,7 @@ public class PyStatementMover extends LineMover {
 
     final int offset = down ? elementToMove.getTextRange().getEndOffset() : elementToMove.getTextRange().getStartOffset();
     int lineNumber = down ? document.getLineNumber(offset) + 1 : document.getLineNumber(offset) - 1;
-    if (moveOutsideFile(elementToMove, document, lineNumber)) return null;
+    if (moveOutsideFile(document, lineNumber)) return null;
     int lineEndOffset = document.getLineEndOffset(lineNumber);
     final int startOffset = document.getLineStartOffset(lineNumber);
     lineEndOffset = startOffset != lineEndOffset ? lineEndOffset - 1 : lineEndOffset;
@@ -141,24 +141,8 @@ public class PyStatementMover extends LineMover {
     return new LineRange(startLine, endLine + 1);
   }
 
-  private static boolean moveOutsideFile(@NotNull final PsiElement elementToMove, @NotNull final Document document, int lineNumber) {
-    if (lineNumber < 0) return true;
-    if (lineNumber >= document.getLineCount()) {
-      final int elementOffset = elementToMove.getTextRange().getStartOffset();
-      final int lineStartOffset = document.getLineStartOffset(document.getLineNumber(elementOffset));
-      final int insertIndex = lineNumber < 0 ? 0 : document.getTextLength();
-      if (elementOffset != lineStartOffset) {
-        ApplicationManager.getApplication().runWriteAction(new Runnable() {
-          @Override
-          public void run() {
-            document.insertString(insertIndex, "\n");
-            PsiDocumentManager.getInstance(elementToMove.getProject()).commitAllDocuments();
-          }
-        });
-      }
-      else return true;
-    }
-    return false;
+  private static boolean moveOutsideFile(@NotNull final Document document, int lineNumber) {
+    return lineNumber < 0 || lineNumber >= document.getLineCount();
   }
 
   private static boolean moveToEmptyLine(@NotNull final PsiElement elementToMove, boolean down) {

python/testData/mover/innerIf.py

@@ -1,3 +1,3 @@
 if value is not None:
     if not False or value <= 2:
-        print "h<caret>ere"
+        print "h<caret>ere"

python/testData/mover/innerIf_afterUp.py

@@ -1,4 +1,4 @@
 if value is not None:
     print "here"
     if not False or value <= 2:
-        pass
+        pass

python/testData/mover/multiCompound.py

@@ -1,3 +1,3 @@
 for item in range(1,
     3):
-    b = 2<caret>
+    b = 2<caret>

python/testData/mover/multiCompound_afterUp.py

@@ -1,4 +1,4 @@
 b = 2<caret>
 for item in range(1,
     3):
-    pass
+    pass

python/testData/mover/multiLineSelection10.py

@@ -1,3 +1,3 @@
 if True:
  <caret><selection>   a = 2
-    b = 3</selection>
+    b = 3</selection>

python/testData/mover/multiLineSelection10_afterUp.py

@@ -1,4 +1,4 @@
 <caret><selection>a = 2
 b = 3</selection>
 if True:
-    pass
+    pass

python/testData/mover/nestedBlock.py

@@ -4,4 +4,4 @@ def test(a, b=1):
     else:
         for n in range(10):
 
-    <caret>        print(b)
+    <caret>        print(b)

python/testData/mover/oneLineCompoundOutside.py

@@ -3,4 +3,4 @@ if condition:
 elif other_condition:
     if another_one:
         if T<caret>rue: a = 1      # <- move statement up here
-        else: b = 2
+        else: b = 2

python/testData/mover/oneLineCompoundOutside_afterUp.py

@@ -4,4 +4,4 @@ elif other_condition:
     if T<caret>rue: a = 1      # <- move statement up here
     else: b = 2
     if another_one:
-        pass
+        pass

python/testData/mover/upInNested.py

@@ -4,4 +4,4 @@ try:
     except:
         print(zoo(0).foo(2))
 except:
-    zoo<caret>(3)
+    zoo<caret>(3)

python/testData/mover/upInNested_afterUp.py

@@ -5,4 +5,4 @@ try:
         print(zoo(0).foo(2))
     zoo(3)
 except:
-    pass
+    pass