zenoamaro/react-quill

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

Open

#498 ouverte le 8 juil. 2019

Voir sur GitHub
 (6 commentaires) (7 réactions) (0 assignés)JavaScript (6 238 stars) (860 forks)batch import
good first issuehelp wanted

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

Guide contributeur

Stack technique
javascriptreact
Domaine
frontend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
React component lifecycleQuill delta understanding
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45
Direction de recherche
The issue is in component.js, where a check throws an error when both nextContents and this.lastDeltaChangeSet are undefined. The condition should handle the case where both are undefined (e.g., first render with no value). Look at the React component's componentWillReceiveProps method and the delta comparison logic. Consider adding a guard clause returning early when both are undefined to avoid the false positive.