jsx-eslint/eslint-plugin-react

no-array-index-key not triggered when spreading props

Open

#1.625 aberto em 24 de dez. de 2017

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (2.797 forks)batch import
enhancementhelp wanted

Métricas do repositório

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

Description

There's no warning or error when doing something like this:

people.map(({ name, email }, key) => {
  const props = {
    key,
    name,
    email,
  };

  return <Person {...props} />;
});

I realize this is a bit of an edge case, but maybe we can find a way to catch these uses as well.

Guia do colaborador

Direção de pesquisa
O problema é sobre a regra `no array index key` não ser acionada quando a chave é espalhada via props. Para corrigir, examine a implementação da regra em `lib/rules/no array index key.js`. Atualmente, a regra verifica a prop `key` diretamente em elementos JSX, mas não detecta o espalhamento `{...props}`. Investigue como outras regras lidam com o espalhamento de props e considere adicionar um auxiliar para procurar `key` em expressões de objeto usadas em atributos de espalhamento JSX. Também verifique o arquivo de teste da regra (`tests/lib/rules/no array index key.js`) para casos de teste existentes e adicione novos que cubram cenários de espalhamento.
Pilha de tecnologia
javascriptreact
Domain
frontend
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.
3
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
JavaScriptReactESLint
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50