getFieldsValue(true) allows for direct mutation of the form store
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 45/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- react, typescript
- Ambito
- frontend
Direzione di ricerca
Inizia da src/useForm.ts intorno alle righe 271-273 e riproduci il problema con getFieldsValue(true) e l’eliminazione di un campo. Conferma che il valore restituito condivida uno stato mutabile con il form store; il lavoro è completato quando l’eliminazione di un campo dall’oggetto restituito non modifica più il risultato di una successiva chiamata a getFieldsValue(true), mentre il comportamento esistente per NamePaths specifici rimane invariato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Repro:
Set up any basic form, and give a field an onChange like this:
() => {
const formData = form.getFieldsValue(true)
delete formData.nameOfField
console.log(form.getFieldsValue(true))
}
Normally, one would expect that this should have no effect, as react/state context should be immutable, but in this case you will find that it will log an object missing the deleting field.
The problem comes from the implementation of getFieldsValue here:
https://github.com/react-component/field-form/blob/d7892873e9eee6e4c618d4289fcdbc9ee240f68f/src/useForm.ts#L271-L273
Which just returns the store object directly.
This should be following React conventions and returning a clone, as it does when a specific list of NamePaths is requested.
- Lingua principale
- TypeScript
- Stelle
- 1k
- Fork
- 287
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di react-component/field-form
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
react-component/field-form#768 · 5 commenti ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
react-component/field-form#751 ·
-
Add getFieldsWarning APIAperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
react-component/field-form#749 ·
-
useWatch支持dynamic pathAperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
react-component/field-form#746 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
react-component/field-form#733 · 1 commento ·
Tutte le issue di react-component/field-form
Issue simili
-
Table: Space fires onActivate in single-selection mode — the reference doc and the JSDoc disagreeAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
sidorares/react-x11-components#764 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
backnotprop/plannotator#1840 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
JoviDeCroock/pracht#432 ·
I maintainer di solito rispondono entro 1 giorno
-
Add: CNN en Espanol SDApertaapproved check:passed streams:add
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
Hardware attribute name "app Connection Support" has inconsistent casingForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
walletbeat/walletbeat#1628 ·
I maintainer di solito rispondono entro 1 giorno