microsoft / microsoft/TypeScript

Polymorphic `this` breaks when a module exports a variable and interface with the same name

Abierto
#19,397 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Bug Domain: This-Typing
Lenguaje dominante
Go
Estrellas
111k
Forks
14.3k
Merge medio
1 d 19 h
PR fusionados (30 d)
117

Descripción

Importing a variable that has a corresponding interface of the same name from a module causes unexpected behavior.

In the screenshot below, the function, doSomething, should return the type Model<number> due to polymorphic this; however, it returns Model<T>, which is unusable.

screen shot 2017-10-20 at 11 02 03 pm

Here is an easy way to reproduce this behavior:

  1. npm install -g @types/sequelize sequelize
  2. Replace node_modules/@types/sequelize/index.d.ts with the following
declare namespace sequelize {
    interface Model<T>  {
        doSomething(arg: T): this;
    }
}

declare var sequelize: { Model: sequelize.Model<any> };

export = sequelize;
  1. create a file with the following contents:
import { Model } from 'sequelize';

let a: Model<number> = null as any;


a.doSomething(5).doSomething();

Workaround:
Use import * as ... syntax and the names will resolve correctly.

screen shot 2017-10-20 at 11 22 28 pm

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Empieza reproduciendo el problema con la declaración de sequelize proporcionada e importando Model con sintaxis de importación con nombre. Traza cómo el compilador resuelve la variable exportada y la interfaz con el mismo nombre al comprobar this polimórfico. Se considera terminado cuando el ejemplo infiere Model y permite la llamada encadenada sin requerir la solución alternativa de importación de namespace.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
typescript
Área
compilers
Tipo de issue
Error
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.