Skip to content

Commit 9a857ea

Browse files
committed
Replace VisitCommandParameter with VisitParameter
1 parent 6652213 commit 9a857ea

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

src/PowerShellEditorServices/Services/Symbols/ReferenceTable.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,7 @@ private AstVisitAction AddReference(SymbolReference symbol)
9090
return existing;
9191
});
9292

93-
return AstVisitAction.Continue;
93+
return symbol.SymbolType is SymbolType.Parameter
94+
? AstVisitAction.SkipChildren : AstVisitAction.Continue;
9495
}
9596
}

src/PowerShellEditorServices/Services/Symbols/Vistors/SymbolVisitor.cs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -68,13 +68,14 @@ public override AstVisitAction VisitFunctionDefinition(FunctionDefinitionAst fun
6868
isDeclaration: true));
6969
}
7070

71-
public override AstVisitAction VisitCommandParameter(CommandParameterAst commandParameterAst)
71+
public override AstVisitAction VisitParameter(ParameterAst parameterAst)
7272
{
73+
// TODO: When we add DisplayString, include the default value.
7374
return _action(new SymbolReference(
7475
SymbolType.Parameter,
75-
commandParameterAst.Extent.Text,
76-
commandParameterAst.Extent,
77-
commandParameterAst.Extent,
76+
$"${parameterAst.Name.VariablePath.UserPath}",
77+
parameterAst.Name.Extent,
78+
parameterAst.Extent,
7879
_file,
7980
isDeclaration: true));
8081
}

0 commit comments

Comments
 (0)