Type definition for TilemapLayer.getTileAt is wrong
I maintainer di solito rispondono entro 2 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 70/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- javascript, typescript
- Ambito
- game-dev
Direzione di ricerca
Il problema è il tipo di ritorno dichiarato di getTileAt() su Phaser.Tilemaps.TilemapLayer, che indica Phaser.Tilemaps.Tile ma può restituire null quando le coordinate x/y cadono al di fuori del layer. Trova la dichiarazione di tipo di quel metodo e verifica se il tipo di ritorno dovrebbe includere null. Il lavoro è completato quando la dichiarazione corrisponde al comportamento in fase di esecuzione e il codice di riproduzione nell'issue non compila più senza un controllo su null.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- JavaScript
- Stelle
- 40.4k
- Fork
- 7.2k
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di phaserjs/phaser
-
Shader.preDestroy leaks its ShaderQuad's vertex buffer and VAO on every create/destroy cycle (4.2.1)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
I maintainer di solito rispondono entro 2 giorni
-
Black opaque background on transparent Phaser.Text textures in Phaser 4.2.1 on older iOS WebGLAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 2 giorni
-
WebGL stroke of a Polygon drops a vertex when its origin-shifted position equals the previous raw vertexForse già presa @ICOM725 l’ha presa 35 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
phaserjs/phaser#7361 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
MouseManager.stopListeners never removes the wheel listener — destroyed Game stays retained through the canvasForse già presa @schulerj89 l’ha presa 75 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di phaserjs/phaser
Issue simili
-
Tool errors containing cycles or BigInt crash getErrorMessage and replace the original failureApertafactory-active factory-automatic task-bug-reproduction-success task-identify-harness-labels-done task-identify-issue-type-done
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
vercel/ai#22796 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
documentation
Difficoltà 2/5 Mezza giornata Idoneità per principianti 78/100
facioquo/stock-indicators-dotnet#2300 ·
I maintainer di solito rispondono entro 1 giorno
-
fix(knowledge): list-promotion-candidates prints a stack trace on a malformed claim inventoryApertagood first issue needs-triage priority: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
melodic-software/claude-code-plugins#6982 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
val() setter uses the first control's type for a mixed selectionForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 Mezza giornata Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 75/100
douglascrockford/Misty#35 ·
I maintainer di solito rispondono entro 1 giorno