jsx-eslint/eslint-plugin-react

Rule proposal: enforce shape of dangerouslySetInnerHTML

Aperta

#711 aperta il 26 lug 2016

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (2731 fork)batch import
help wantednew rulequestion

Metriche repository

Star
 (9293 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

React will issue a warning if you try to use dangerouslySetInnerHTML with a value of the wrong shape.

https://github.com/facebook/react/blob/85dcbf83/src/renderers/dom/shared/ReactDOMComponent.js#L173-L177

This should be easy enough to enforce for most cases with a linter rule.

Bad:

<div dangerouslySetInnerHTML={{ }} />

<div dangerouslySetInnerHTML={{ html: 'HTML' }} />

<div dangerouslySetInnerHTML={true} />

Good:

<div dangerouslySetInnerHTML={{ __html: 'HTML' }} />

Guida contributor

Direzione di ricerca
Esamina le regole ESLint esistenti per React (es. jsx no literals) per comprendere il pattern, poi crea una nuova regola che avvisi quando dangerouslySetInnerHTML non è un oggetto con una chiave html.
Tech stack
javascriptreact
Dominio
frontendtooling
Tipo issue
Funzionalità
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
JavaScriptReactESLint
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80