Merge pull request #957 from lieryan/lieryan-fix-extract-refactor-range

diraol · web-flow · commit 63e3866c4493 · 2019-01-05T20:37:24.000-02:00
Fix off by one selection of extract variable/method
diff --git a/pymode/rope.py b/pymode/rope.py
@@ -576,7 +576,7 @@ def get_refactor(ctx):
         _, offset1 = env.get_offset_params(cursor1)
         _, offset2 = env.get_offset_params(cursor2)
         return extract.ExtractMethod(
-            ctx.project, ctx.resource, offset1, offset2)
+            ctx.project, ctx.resource, offset1, offset2 + 1)
 
 
 class ExtractVariableRefactoring(Refactoring):
@@ -600,7 +600,7 @@ def get_refactor(ctx):
         _, offset1 = env.get_offset_params(cursor1)
         _, offset2 = env.get_offset_params(cursor2)
         return extract.ExtractVariable(
-            ctx.project, ctx.resource, offset1, offset2)
+            ctx.project, ctx.resource, offset1, offset2 + 1)
 
 
 class InlineRefactoring(Refactoring):
