jashkenas/coffeescript

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

Open

#5.222 geöffnet am 18. Juli 2019

Auf GitHub ansehen
 (1 Kommentar) (5 Reaktionen) (0 zugewiesene Personen)CoffeeScript (1.980 Forks)batch import
bughelp wanted

Repository-Metriken

Stars
 (16.577 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuchen Sie, wie der CoffeeScript Compiler `this` in verschachtelten Klassen auflöst, indem Sie das generierte JavaScript oder den AST des Parsers untersuchen. Suchen Sie nach Unterschieden in der Bindung zwischen direkten Methoden und gebundenen Methoden.
Tech Stack
javascript
Domain
compilers
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
CoffeeScriptJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60