additional condition so that a function CommandAst will not be touched if it exists before the functions definition

Razmo99 · JustinGrote · commit ffadfe639942 · 2024-06-07T13:02:35.000+10:00
diff --git a/src/PowerShellEditorServices/Services/PowerShell/Refactoring/IterativeFunctionVistor.cs b/src/PowerShellEditorServices/Services/PowerShell/Refactoring/IterativeFunctionVistor.cs
@@ -171,7 +171,8 @@ public void ProcessNode(Ast node, bool shouldRename)
                     }
                     break;
                 case CommandAst ast:
-                    if (ast.GetCommandName()?.ToLower() == OldName.ToLower())
+                    if (ast.GetCommandName()?.ToLower() == OldName.ToLower() &&
+                        TargetFunctionAst.Extent.StartLineNumber <= ast.Extent.StartLineNumber)
                     {
                         if (shouldRename)
                         {

Original file line number	Diff line number	Diff line change
`@@ -171,7 +171,8 @@ public void ProcessNode(Ast node, bool shouldRename)`
`171`	`171`	`}`
`172`	`172`	`break;`
`173`	`173`	`case CommandAst ast:`
`174`		`- if (ast.GetCommandName()?.ToLower() == OldName.ToLower())`
	`174`	`+ if (ast.GetCommandName()?.ToLower() == OldName.ToLower() &&`
	`175`	`+ TargetFunctionAst.Extent.StartLineNumber <= ast.Extent.StartLineNumber)`
`175`	`176`	`{`
`176`	`177`	`if (shouldRename)`
`177`	`178`	`{`