mapbox/mapbox-gl-js

validate property names in getPaintProperty

Open

#6033 aperta il 22 gen 2018

Vedi su GitHub
 (2 commenti) (5 reazioni) (0 assegnatari)JavaScript (2203 fork)batch import
api :memo:good first issue

Metriche repository

Star
 (10.532 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

map.setPaintProperty(layer, key value) validates arguments but map.getPaintProperty does not. Providing a bad property key will trigger an internal error. It should trigger an error with a helpful errror message. Seen here: https://github.com/mapbox/mapbox-gl-js/issues/6030

mapbox-gl-js version: v0.43.0 and master

Steps to Trigger Behavior

  1. map.getPaintProperty("background", "fill-radius");

Expected Behavior

something like: "fill-radius" is not a paint property background layers"

Actual Behavior

Uncaught TypeError: Cannot read property 'value' of undefined

Guida contributor

Direzione di ricerca
Ispeziona la funzione `getPaintProperty` nel codice sorgente (probabilmente in `src/ui/map.js` o simile). Aggiungi una convalida per verificare se la chiave della proprietà esiste per il tipo di layer dato, simile alla convalida in `setPaintProperty`. Restituisci un messaggio di errore descrittivo come quello nel comportamento previsto.
Tech stack
javascript
Dominio
web dev
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75