[question] Node removeChild() set to null

Offen
  • Stimmt etwas nicht?
#375 6 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
38/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Tech-Stack
javascript
Bereich
frontend

Rechercherichtung

Beginne in core/Node bei removeChild() und vergleiche die Behandlung von _children mit Dispatch.prototype.dispatch im gemeldeten Snippet. Reproduziere eine Event-Dispatch-Operation nach dem Entfernen eines Kindes und überprüfe anschließend, dass entfernte Kinder keinen Zugriff auf null mehr verursachen, während das beabsichtigte Verhalten der Kinderliste erhalten bleibt; prüfe, ob bereits Tests für Node und dispatch vorhanden sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

question

When Node1.removeChild(node2) is called, the index of node2 in Node1._children array is set to NULL.
This causes an error when, after a removeChild, i use events:

Uncaught TypeError: Cannot read property 'onReceive' of null

here:

Dispatch.prototype.dispatch = function dispatch (event, payload) {
if (!event) throw new Error('dispatch requires an event name as it's first argument');

var queue = this._queue;
var item;
var i;
var len;
var children;

queue.length = 0;
queue.push(this._context);

while (queue.length) {
    item = queue.shift();
    if (item.onReceive) item.onReceive(event, payload);
    children = item.getChildren();
    for (i = 0, len = children.length ; i < len ; i++) queue.push(children[i]);
}

};

I managed to fix this, by changing:

this._children[index] = null;

with:

this._children.splice(index, 1);

in the removeChild() function of core/Node.

Is this intentional?
Does this change affect other things?

Vorherrschende Sprache
JavaScript
Sterne
1.7k
Forks
249
Keine gemergten PRs in 30 T.

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus Famous/engine

Alle Issues in Famous/engine

Ähnliche Issues

Weitere Issues zu JavaScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.