mapbox/mapbox-gl-js

validate property names in getPaintProperty

Open

#6.033 aberto em 22 de jan. de 2018

Ver no GitHub
 (2 comments) (5 reactions) (0 assignees)JavaScript (2.203 forks)batch import
api :memo:good first issue

Métricas do repositório

Stars
 (10.532 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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

Guia do colaborador

Direção de pesquisa
Inspecione a função `getPaintProperty` no código fonte (provavelmente em `src/ui/map.js` ou similar). Adicione validação para verificar se a chave da propriedade existe para o tipo de camada fornecido, semelhante à validação em `setPaintProperty`. Retorne uma mensagem de erro descritiva como a do comportamento esperado.
Pilha de tecnologia
javascript
Domain
web dev
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75