[question] Node removeChild() set to null
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 38/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- frontend
Direzione di ricerca
Inizia in core/Node da removeChild() e confronta la gestione di _children con Dispatch.prototype.dispatch nello snippet segnalato. Riproduci un dispatch di eventi dopo la rimozione di un elemento figlio, quindi verifica che gli elementi figli rimossi non causino più un accesso a null, preservando al contempo il comportamento previsto dell’elenco dei figli; controlla se esistono già test per Node e dispatch.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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?
- Lingua principale
- JavaScript
- Stelle
- 1.7k
- Fork
- 249
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Famous/engine
-
mechronetechAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 5/100
-
car motors for saleAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 1/100
-
k20 engine for saleAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 1/100
-
#THEREISNOSPOONAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 10/100
-
Slack sign up not workingAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 10/100
Tutte le issue di Famous/engine
Issue simili
-
area: docs good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
I maintainer di solito rispondono entro 1 giorno
-
Support
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
openseadragon/openseadragon#2996 ·
I maintainer di solito rispondono entro 3 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
jfromaniello/url-join#134 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100