dotnet/roslyn

Move Static Members to another type overwrites existing file and code

Open

#61.471 aberto em 23 de mai. de 2022

Ver no GitHub
 (1 comment) (0 reactions) (1 assignee)C# (4.257 forks)batch import
Area-IDEBugFeature - Move Static Membershelp wanted

Métricas do repositório

Stars
 (20.414 stars)
Métricas de merge de PR
 (Mesclagem média 6d 17h) (256 fundiu PRs em 30d)

Description

This issue has been moved from a ticket on Developer Community.

hello again :)

I've been developing a few related methods and had them all in one static class file, but then I wanted to refactor them out into their own files.

I had previously done this for a common method so there already existed a file with the standard name given by vs. In this case LayoutHelpers.cs

Anyway this had slipped my mind and in the previous (non preview version) I was used to VS behaving itself and checking if the file already existed.

I have come to rely on this magical refactoring tool and have not had issues before.

Anyway this time the original contents of the file were overwritten.

Hoping to be helpful I have recreated my steps and recorded the issue.

All the best

rob

image

Original Comments

Feedback Bot on 5/22/2022, 07:39 PM:

(private comment, text removed)

Feedback Bot on 5/23/2022, 00:23 PM:

(private comment, text removed)

Original Solutions

(no solutions)

Guia do colaborador

Direção de pesquisa
Localize o código de refatoração MoveStaticMembers no Roslyn e examine a lógica de substituição de arquivos. Identifique onde ele falha ao verificar se o arquivo já existe antes de escrever.
Pilha de tecnologia
csharp
Domain
developer experiencetooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#Visual StudioGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70