microsoft / microsoft/TypeScript
3.6 regression: unions of callable types involving `this` types
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.4k
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 117
Descripción
TypeScript Version: Version 3.6.3, Version 3.7.0-dev.20190911
Search Terms: intersect function
Code
class A {
public uniqueToA = "a";
public getString(this: A): string {
return "a";
}
}
class B {
public uniqueToB = "b";
public getString(this: B): string {
return "b";
}
}
function f(model: A | B): string {
return model.getString();
}
Expected behavior:
In Typescript 3.5.3, the model.getString() in function f has no error.
Actual behavior:
In Typescript 3.6.3 and 3.7.0-dev.20190911, the code above gets the following errors:
index.ts:18:12 - error TS2684: The 'this' context of type 'A | B' is not assignable to method's 'this' of type 'A & B'.
Type 'A' is not assignable to type 'A & B'.
Property 'uniqueToB' is missing in type 'A' but required in type 'B'.
18 return model.getString();
~~~~~
index.ts:10:12
10 public uniqueToB = "b";
~~~~~~~~~
'uniqueToB' is declared here.
Related Issues: #32506
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Reproduce el ejemplo de index.ts con TypeScript 3.5.3, 3.6.3 y la versión 3.7.0-dev referenciada para confirmar la regresión. Sigue la ruta de comprobación de tipos para model.getString() y las uniones de tipos invocables con parámetros this; se considera terminado cuando el ejemplo compile sin el error TS2684 y se conserven las comprobaciones relevantes de this.
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