dotnet/roslyn

Move static members to another type does not use the correct visibility

Aperta

#58.502 aperta il 27 dic 2021

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (1 assegnatario)C# (4257 fork)batch import
Area-IDEBughelp wanted

Metriche repository

Star
 (20.414 stelle)
Metriche merge PR
 (Merge medio 6g 17h) (256 PR mergiate in 30 g)

Descrizione

Related to https://github.com/dotnet/roslyn/issues/55127

Version Used: Version 17.1.0 Preview 1.1

Steps to Reproduce:

  1. Create a new class library, paste the following code in:
class Test
{
    private const string TestConstant = "";

    void M()
    {
        _ = TestConstant;
    }
}
  1. Ctrl+. on TestConstant, select "Move static members to another type..."
  2. Hit enter to accept the refactoring

Expected Behavior:

Generated code visibility allows for Test to see the constant.

Actual Behavior:

The following code is generated:

internal static class TestHelpers
{
    private const string TestConstant = "";
}

The constant is private, and compilation errors result as the original location cannot see the moved constant.

Guida contributor

Direzione di ricerca
Investiga il refactoring MoveStaticMembers nel codebase di Roslyn. Trova dove viene determinata la visibilità dei membri generati e assicurati che rispetti i requisiti di accesso del tipo originale.
Tech stack
csharp
Dominio
compilers
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60