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 (860 forks)batch import
good first issuehelp wanted

Métriques du dépôt

Stars
 (6 238 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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

Direction de recherche
Examinez la condition qui déclenche l'erreur lorsque nextContents et this.lastDeltaChangeSet sont tous deux undefined. Déterminez si la comparaison doit gérer les valeurs undefined ou si une vérification différente est nécessaire pour éviter les faux positifs.
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
ReactQuill editor
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75