zenoamaro/react-quill

False positive "You are passing the delta object from the onChange event back as value"

Open

#498 aberto em 8 de jul. de 2019

Ver no GitHub
 (6 comments) (7 reactions) (0 assignees)JavaScript (860 forks)batch import
good first issuehelp wanted

Métricas do repositório

Stars
 (6.238 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

In my case, when both nextContents and this.lastDeltaChangeSet are undefined, I get this error. It happens in a real scenario when I switch between 2 records, and component.js componentWillReceiveProps gets called

From component.js:

if (nextContents === this.lastDeltaChangeSet) {
        console.error(nextContents, this.lastDeltaChangeSet)
        throw new Error(
          'You are passing the `delta` object from the `onChange` event back ' +
            'as `value`. You most probably want `editor.getContents()` instead. ' +
            'See: https://github.com/zenoamaro/react-quill#using-deltas'
        )
      }

React-Quill version

1.3.3

Guia do colaborador

Direção de pesquisa
Investigue a condição que aciona o erro quando tanto nextContents quanto this.lastDeltaChangeSet são undefined. Determine se a comparação deve tratar valores undefined ou se uma verificação diferente é necessária para evitar falsos positivos.
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
ReactQuill editor
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75