facebookarchive/draft-js

Why is createEntity mutable?

Open

#1.256 aberto em 21 de jun. de 2017

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (22.554 stars) (2.646 forks)batch import
documentationhelp wanted

Description

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.

Guia do colaborador

Pilha de tecnologia
javascriptreact
Domain
frontend
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
JavaScriptReactDraft.js basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50
Direção de pesquisa
Examine the createEntity method in Draft.js's ContentState module to understand why it modifies the existing object. The issue reports that contentState.createEntity mutates the original object instead of returning a new one. Look for existing tests or discussions about immutability in the codebase, then implement a fix that returns a new ContentState with the entity added, ensuring backward compatibility.