Microsoft/TypeScript

SourceFileObject.getNamedDeclarations is missing declarations before methods

Open

#2.137 geöffnet am 25. Feb. 2015

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (6.726 Forks)batch import
BugDomain: APIDomain: BinderHelp Wanted

Repository-Metriken

Stars
 (48.455 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (9 gemergte PRs in 30 T)

Beschreibung

I'm doing a SourceFileObject.getNamedDeclarations on the following code snippet:

module Mankala {
  export class Rectangle {
    x: number;
    y: number;
    width: number;
    height: number;
    square() {
    }
  }
}

This returns an array that is missing the property 'height'.

The problem is in SourceFileObject.prototype.getNamedDeclarations

     case 125 /* Method */:
             var functionDeclaration = node;
             if (functionDeclaration.name && functionDeclaration.name.getFullWidth() > 0) {
             var lastDeclaration = namedDeclarations.length > 0 ? namedDeclarations[namedDeclarations.length - 1] : undefined;
                                if (lastDeclaration && functionDeclaration.symbol === lastDeclaration.symbol) {
                                    if (functionDeclaration.body && !lastDeclaration.body) {
                                        namedDeclarations[namedDeclarations.length - 1] = functionDeclaration;
                                    }
                                }

functionDeclaration.symbol === lastDeclaration.symbol is true as both symbols are undefined. The property 'height' is then replaced by method 'square'

Contributor Guide

Research-Richtung
Der Fehler liegt in SourceFileObject.prototype.getNamedDeclarations. Das Problem ist, dass bei einer Methode das Symbol der Methode mit dem Symbol der letzten Deklaration verglichen wird. Wenn beide undefined sind, wird die letzte Deklaration fälschlicherweise ersetzt. Die Lösung besteht darin, sicherzustellen, dass der Vergleich nur ausgelöst wird, wenn Symbole definiert sind, oder den Fall zu behandeln, in dem Symbole undefined sind. Schauen Sie sich die Datei an, die SourceFileObject enthält (wahrscheinlich src/compiler/utilities.ts oder ähnlich), und prüfen Sie die Logik um case 125 (Method). Überprüfen Sie die Kommentare im Issue auf Diskussionen oder verlinkte PRs.
Tech Stack
typescript
Domain
api
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
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
TypeScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70