jashkenas/coffeescript

Bug: Wrong `this` in bound methods of nested classes

Open

#5.222 aberto em 18 de jul. de 2019

Ver no GitHub
 (1 comment) (5 reactions) (0 assignees)CoffeeScript (1.980 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (16.577 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

While upgrading an application to CS2 we ran into the following issue in an obscure usage of nested classes.

Screenshot 2019-07-18 at 16 33 10

As you can see, the nested bound method thought its this is the parent class. This seems to happen on a "parse" level, because it's replaced in the code. This lead to tricky to catch bug on our side.

Guia do colaborador

Direção de pesquisa
Examine como o compilador CoffeeScript resolve `this` em classes aninhadas inspecionando o JavaScript gerado ou o AST do parser. Procure diferenças na vinculação entre métodos diretos e métodos vinculados.
Pilha de tecnologia
javascript
Domain
compilers
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
CoffeeScriptJavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60