Skip to content

Commit 14f395e

Browse files
committed
Remove Unnecessary code
1 parent 1a8287f commit 14f395e

File tree

1 file changed

+2
-35
lines changed

1 file changed

+2
-35
lines changed

src/PowerShellEditorServices/Services/TextDocument/Handlers/RenameHandler.cs

Lines changed: 2 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
using System.Management.Automation.Language;
88
using System.Threading;
99
using System.Threading.Tasks;
10-
using MediatR;
1110
using Microsoft.PowerShell.EditorServices.Services;
1211
using Microsoft.PowerShell.EditorServices.Services.TextDocument;
1312
using Microsoft.PowerShell.EditorServices.Refactoring;
@@ -19,7 +18,6 @@
1918

2019
namespace Microsoft.PowerShell.EditorServices.Handlers;
2120

22-
2321
/// <summary>
2422
/// A handler for <a href="https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#textDocument_prepareRename">textDocument/prepareRename</a>
2523
/// LSP Ref: <see cref="PrepareRename()"/>
@@ -83,15 +81,6 @@ private static ScriptExtentAdapter GetFunctionNameExtent(FunctionDefinitionAst a
8381
};
8482
}
8583

86-
/// <summary>
87-
/// Finds a renamable symbol at a given position in a script file using 1-based row/column references
88-
/// <param name="scriptFile"/>
89-
/// <param name="line">1-based line number</param>
90-
/// <param name="column">1-based column number</param>
91-
/// </summary>
92-
internal static Ast FindRenamableSymbol(ScriptFile scriptFile, int line, int column) =>
93-
FindRenamableSymbol(scriptFile, new ScriptPositionAdapter(line, column));
94-
9584
/// <summary>
9685
/// Finds a renamable symbol at a given position in a script file.
9786
/// </summary>
@@ -204,22 +193,15 @@ internal static TextEdit[] RenameFunction(Ast token, Ast scriptAst, RenameParams
204193

205194
internal static TextEdit[] RenameVariable(Ast symbol, Ast scriptAst, RenameParams requestParams)
206195
{
207-
RenameSymbolParams request = new()
208-
{
209-
FileName = requestParams.TextDocument.Uri.ToString(),
210-
Line = requestParams.Position.Line,
211-
Column = requestParams.Position.Character,
212-
RenameTo = requestParams.NewName
213-
};
214196
if (symbol is VariableExpressionAst or ParameterAst or CommandParameterAst or StringConstantExpressionAst)
215197
{
216198

217199
IterativeVariableRename visitor = new(
218-
request.RenameTo,
200+
requestParams.NewName,
219201
symbol.Extent.StartLineNumber,
220202
symbol.Extent.StartColumnNumber,
221203
scriptAst,
222-
request.Options ?? null
204+
null //FIXME: Pass through Alias config
223205
);
224206
visitor.Visit(scriptAst);
225207
return visitor.Modifications.ToArray();
@@ -322,18 +304,3 @@ internal record ScriptExtentAdapter(IScriptExtent extent) : IScriptExtent
322304
public bool Contains(Position position) => ContainsPosition(this, position);
323305
public static bool ContainsPosition(ScriptExtentAdapter range, ScriptPositionAdapter position) => Range.ContainsPosition(range, position);
324306
}
325-
326-
public class RenameSymbolParams : IRequest<RenameSymbolResult>
327-
{
328-
public string? FileName { get; set; }
329-
public int Line { get; set; }
330-
public int Column { get; set; }
331-
public string? RenameTo { get; set; }
332-
public RenameSymbolOptions? Options { get; set; }
333-
}
334-
335-
public class RenameSymbolResult
336-
{
337-
public RenameSymbolResult() => Changes = new List<TextEdit>();
338-
public List<TextEdit> Changes { get; set; }
339-
}

0 commit comments

Comments
 (0)