Microsoft/TypeScript

Bug: Accessing private statics in a class via its derived class is allowed

Open

#8.624 aberto em 16 de mai. de 2016

Ver no GitHub
 (22 comments) (1 reaction) (0 assignees)TypeScript (6.726 forks)batch import
Breaking ChangeEffort: ModerateHelp WantedSuggestion

Métricas do repositório

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

Description

TypeScript Version:

1.8.X

Code

class FooBase {
    private static privateStatic: string = "";

    testBase(): void {
        console.log(FooBase.privateStatic);

        // Should error, but doesn't
        console.log(Foo.privateStatic);
   }
}

class Foo extends FooBase {
    test() {
        // Should error, and does
        console.log(Foo.privateStatic);
    }
}

Expected behavior: Accessing Foo.privateStatic shouldn't be allowed, regardless of the context it's called in.

Actual behavior: FooBase is allowed to do this.

(thanks @sethbrenith for deducing this)

Guia do colaborador

Direção de pesquisa
Inspecione a lógica do verificador para acesso a membros em tipos de classe, especialmente a resolução de membros estáticos privados quando acessados através de um nome de classe derivada. O problema ocorre na fase de verificação de tipo, onde FooBase acessa Foo.privateStatic, o que deveria ser um erro. Procure o código que verifica a acessibilidade de membros estáticos e rastreie a condição que erroneamente permite esse acesso.
Pilha de tecnologia
typescript
Domain
developer experience
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.
3
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.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
TypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40