jsx-eslint/eslint-plugin-react

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

Open

#1625 aperta il 24 dic 2017

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (2797 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (8630 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Il problema riguarda la regola `no array index key` che non si attiva quando la chiave viene diffusa tramite props. Per risolvere, esamina l'implementazione della regola in `lib/rules/no array index key.js`. Attualmente la regola controlla la prop `key` direttamente sugli elementi JSX, ma non rileva la diffusione `{...props}`. Indaga come altre regole gestiscono la diffusione delle props e considera l'aggiunta di un helper per cercare `key` nelle espressioni oggetto utilizzate negli attributi di diffusione JSX. Controlla anche il file di test della regola (`tests/lib/rules/no array index key.js`) per i casi di test esistenti e aggiungine di nuovi che coprano gli scenari di diffusione.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptReactESLint
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50