mapbox/mapbox-gl-js

validate property names in getPaintProperty

Open

#6.033 geöffnet am 22. Jan. 2018

Auf GitHub ansehen
 (2 Kommentare) (5 Reaktionen) (0 zugewiesene Personen)JavaScript (2.203 Forks)batch import
api :memo:good first issue

Repository-Metriken

Stars
 (10.532 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Prüfe die Funktion `getPaintProperty` im Quellcode (wahrscheinlich in `src/ui/map.js` oder ähnlich). Füge eine Validierung hinzu, die prüft, ob der Property Schlüssel für den angegebenen Layer Typ existiert, ähnlich der Validierung in `setPaintProperty`. Gib eine beschreibende Fehlermeldung wie im erwarteten Verhalten zurück.
Tech Stack
javascript
Domain
web dev
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75