Bug #77686 [Com]: Removed elements are still returned by getElementById

From: Date: Mon, 04 Mar 2019 06:47:42 +0000
Subject: Bug #77686 [Com]: Removed elements are still returned by getElementById
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-219827@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=77686&edit=1 ID: 77686 Comment by: gtisza at gmail dot com Reported by: gtisza at gmail dot com Summary: Removed elements are still returned by getElementById Status: Open Type: Bug Package: DOM XML related PHP Version: 7.3.2 Block user comment: N Private report: N New Comment: Apparently the id is only unindexed when the element is destroyed, not when it is detached from the document: https://3v4l.org/61Hbv (thanks to Ori Livneh for tracking that down). So not that easy to run into this bug as typically no reference is retained to a detached element, once the variable used for detaching goes out of scope. Previous Comments: ------------------------------------------------------------------------ [2019-03-01 22:14:54] bd808@php.net Corrected test script including the DOM removal: --------------- $doc = new DOMDocument; $doc->loadHTML('<html><body id="x"><div id="y"></div></body></html>'); $body = $doc->getElementById('x'); $div = $doc->getElementById('y'); $body->removeChild($div); $doc->getElementById('y'); // <div id="y"> ------------------------------------------------------------------------ [2019-03-01 07:11:23] gtisza at gmail dot com Description: ------------ Removing a DOMElement from a HTML document does not refresh the id index; getElementById() will still return the removed element. (getElementsByTagName(), in contrast, works as expected.) Live test: https://3v4l.org/UQYTG Calling setIdAttribute() on the removed element apparently updates the index and fixes getElementById() behavior. It does not fix it for any children of the removed node, however. Test script: --------------- $doc = new DOMDocument; $doc->loadHTML('<html><body id="x"><div id="y"></div></body></html>'); $body = $doc->getElementById('x'); $div = $doc->getElementById('y'); $doc->getElementById('y'); // <div id="y"> ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=77686&edit=1

« previous php.bugs (#219827) next »