zenoamaro/react-quill

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

Open

#498 geöffnet am 8. Juli 2019

Auf GitHub ansehen
 (6 Kommentare) (7 Reaktionen) (0 zugewiesene Personen)JavaScript (6.238 Stars) (860 Forks)batch import
good first issuehelp wanted

Beschreibung

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

Contributor Guide

Tech Stack
javascriptreact
Domain
frontend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
React component lifecycleQuill delta understanding
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
45
Research-Richtung
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.