jashkenas/coffeescript

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

Open

#5222 aperta il 18 lug 2019

Vedi su GitHub
 (1 commento) (5 reazioni) (0 assegnatari)CoffeeScript (1980 fork)batch import
bughelp wanted

Metriche repository

Star
 (16.577 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina come il compilatore CoffeeScript risolve `this` nelle classi nidificate ispezionando il JavaScript generato o l'AST del parser. Cerca differenze nel binding tra metodi diretti e metodi vincolati.
Tech stack
javascript
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-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
CoffeeScriptJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60