jashkenas/coffeescript

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

Open

#5 222 ouverte le 18 juil. 2019

Voir sur GitHub
 (1 commentaire) (5 réactions) (0 assignés)CoffeeScript (1 980 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (16 577 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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.

Guide contributeur

Direction de recherche
Examinez comment le compilateur CoffeeScript résout `this` dans les classes imbriquées en inspectant le JavaScript généré ou l'AST de l'analyseur. Recherchez les différences de liaison entre les méthodes directes et les méthodes liées.
Stack technique
javascript
Domaine
compilers
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
CoffeeScriptJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60