jsx-eslint/eslint-plugin-react

react/no-direct-mutation-state doesn't catch all cases

Open

#1418 aperta il 8 set 2017

Vedi su GitHub
 (8 commenti) (0 reazioni) (0 assegnatari)JavaScript (2797 fork)batch import
help wanted

Metriche repository

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

Descrizione

  const { model } = this.state;
  model.rating = rate;

In this case eslint won't show any errors.

Guida contributor

Direzione di ricerca
Esamina l'implementazione della regola no direct mutation state per capire perché non riesce a rilevare la mutazione quando lo stato viene destrutturato. Guarda i casi di test per i pattern esistenti e aggiungi un caso di test per l'assegnazione destrutturata. Quindi modifica l'attraversamento AST della regola per rilevare mutazioni sulle proprietà dell'oggetto ottenute tramite destrutturazione.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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
ESLintReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60