facebookarchive/draft-js

Why is createEntity mutable?

Open

#1256 aperta il 21 giu 2017

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (2646 fork)batch import
documentationhelp wanted

Metriche repository

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

Descrizione

Do you want to request a feature or report a bug? Question/bug

What is the current behavior? Calling contentState.createEntity actually modifies contentState object instead of just returning a new one. Here is a simple fiddle: https://jsfiddle.net/FredyCr/32vo35tk/2/

What is the expected behavior? I guess it's not a big deal, it's just little bit confusing given that almost every other operation is immutable.

Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js? Working with the latest 0.10.1, not sure about previous versions.

Guida contributor

Direzione di ricerca
Esamina il metodo createEntity nel modulo ContentState di Draft.js per capire perché modifica l'oggetto esistente. Il problema segnala che contentState.createEntity muta l'oggetto originale invece di restituirne uno nuovo. Cerca test o discussioni esistenti sull'immutabilità nella codebase, quindi implementa una correzione che restituisca un nuovo ContentState con l'entità aggiunta, garantendo la compatibilità con le versioni precedenti.
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptReactDraft.js basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50