mapbox/mapbox-gl-js

validate property names in getPaintProperty

Ouverte

#6 033 ouverte le 22 janv. 2018

Voir sur GitHubGuide contributeur
 (2 commentaires) (5 réactions) (0 personne assignée)JavaScript (2 203 forks)batch import
api :memo:good first issue

Métriques du dépôt

Stars
 (10 532 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

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