mapbox/mapbox-gl-js

validate property names in getPaintProperty

Open

#6 033 ouverte le 22 janv. 2018

Voir sur GitHub
 (2 commentaires) (5 réactions) (0 assignés)JavaScript (2 203 forks)batch import
api :memo:good first issue

Métriques du dépôt

Stars
 (10 532 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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

Guide contributeur

Direction de recherche
Inspectez la fonction `getPaintProperty` dans le code source (probablement dans `src/ui/map.js` ou similaire). Ajoutez une validation pour vérifier si la clé de propriété existe pour le type de calque donné, similaire à la validation dans `setPaintProperty`. Renvoyez un message d'erreur descriptif comme celui du comportement attendu.
Stack technique
javascript
Domaine
web dev
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75