Type definition for TilemapLayer.getTileAt is wrong
Los mantenedores suelen responder en 2 días
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 70/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- javascript, typescript
- Área
- game-dev
Línea de trabajo
El problema es el tipo de retorno declarado de getTileAt() en Phaser.Tilemaps.TilemapLayer, que indica Phaser.Tilemaps.Tile pero puede devolver null cuando las coordenadas x/y quedan fuera de la capa. Busca la declaración de tipos de ese método y comprueba si el tipo de retorno debería incluir null. Está terminado cuando la declaración coincide con el comportamiento en tiempo de ejecución y el código de reproducción del issue ya no compila sin una comprobación de null.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Version
- Phaser Version: 4.2.1
- Operating system: Linux
- Browser: not relevant
Description
When using a Phaser.Tilemaps.TilemapLayer object, it exposes the method getTileAt(). The return type of that method according to the type definition is Phaser.Tilemaps.Tile. However, it could also return null if the given x/y coordinates are not within layer bounds.
Example Test Code
getTileId(tx:number, ty:number, layer: Phaser.Tilemaps.TilemapLayer): number {
return layer.getTileAt(tx, ty).index ?? 0
}
The typescript compiler will happily accept this code but passing in values for tx and ty that are outside the layer boundaries will cause Cannot read properties of null (reading 'index') at runtime.
If I put an optional chaining operator (?) there, I would get the error Unnecessary optional chain on a non-nullish value and only if I manually type coerce it into as Phaser.Tilemaps.Tile | null the compiler does not complain.
Additional Information
- Lenguaje dominante
- JavaScript
- Estrellas
- 40.5k
- Forks
- 7.2k
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la 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.
Más de phaserjs/phaser
-
Shader.preDestroy leaks its ShaderQuad's vertex buffer and VAO on every create/destroy cycle (4.2.1)Posiblemente ocupada @k1epg01ng129-eng la tomó hace 2 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
Los mantenedores suelen responder en 2 días
-
Black opaque background on transparent Phaser.Text textures in Phaser 4.2.1 on older iOS WebGLAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 2 días
-
flipY renders tile incorrectlyAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 2 días
-
WebGL stroke of a Polygon drops a vertex when its origin-shifted position equals the previous raw vertexPosiblemente ocupada @ICOM725 la tomó hace 41 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
phaserjs/phaser#7361 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
MouseManager.stopListeners never removes the wheel listener — destroyed Game stays retained through the canvasPosiblemente ocupada @schulerj89 la tomó hace 80 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
Todos los issues de phaserjs/phaser
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Add: EUROTV NEWSAbiertochannels:add check:passed
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
curriculum documentation quality
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
githubnext/gh-aw-workshop#4638 ·
Los mantenedores suelen responder en 1 día
-
clawsweeper:fix-shape-clear clawsweeper:queueable-fix clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster no-stale P2
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
openclaw/openclaw-enterprise#2571 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día