Reorder params fixed

Hosch250 · Hosch250 · commit eb89c1829a7a · 2015-06-19T10:46:56.000-05:00
diff --git a/RetailCoder.VBE/Refactorings/ReorderParameters/ReorderParametersRefactoring.cs b/RetailCoder.VBE/Refactorings/ReorderParameters/ReorderParametersRefactoring.cs
@@ -140,7 +140,11 @@ private void RewriteCall(VBAParser.ArgsCallContext paramList, CodeModule module)
             }
 
             module.ReplaceLine(paramList.Start.Line, newContent);
-            module.DeleteLines(paramList.Start.Line + 1, lineCount - 1);
+
+            for (var line = paramList.Start.Line + 1; line < paramList.Start.Line + lineCount; line++)
+            {
+                module.ReplaceLine(line, "");
+            }
         }
 
         private void AdjustSignatures()
@@ -239,7 +243,11 @@ private void RewriteSignature(Declaration target, VBAParser.ArgListContext param
             }
 
             module.ReplaceLine(paramList.Start.Line, newContent);
-            module.DeleteLines(paramList.Start.Line + 1, lineNum - 1);
+
+            for (var line = paramList.Start.Line + 1; line < paramList.Start.Line + lineNum; line++)
+            {
+                module.ReplaceLine(line, "");
+            }
         }
 
         private string GetOldSignature(Declaration target)

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,11 @@ private void RewriteCall(VBAParser.ArgsCallContext paramList, CodeModule module)`
`140`	`140`	`}`
`141`	`141`
`142`	`142`	`module.ReplaceLine(paramList.Start.Line, newContent);`
`143`		`- module.DeleteLines(paramList.Start.Line + 1, lineCount - 1);`
	`143`	`+`
	`144`	`+ for (var line = paramList.Start.Line + 1; line < paramList.Start.Line + lineCount; line++)`
	`145`	`+ {`
	`146`	`+ module.ReplaceLine(line, "");`
	`147`	`+ }`
`144`	`148`	`}`
`145`	`149`
`146`	`150`	`private void AdjustSignatures()`
`@@ -239,7 +243,11 @@ private void RewriteSignature(Declaration target, VBAParser.ArgListContext param`
`239`	`243`	`}`
`240`	`244`
`241`	`245`	`module.ReplaceLine(paramList.Start.Line, newContent);`
`242`		`- module.DeleteLines(paramList.Start.Line + 1, lineNum - 1);`
	`246`	`+`
	`247`	`+ for (var line = paramList.Start.Line + 1; line < paramList.Start.Line + lineNum; line++)`
	`248`	`+ {`
	`249`	`+ module.ReplaceLine(line, "");`
	`250`	`+ }`
`243`	`251`	`}`
`244`	`252`
`245`	`253`	`private string GetOldSignature(Declaration target)`