PHP :: Bug #55538 :: Error when unsetting a child of an SimpleXmlElement

[2011-08-30 11:21 UTC] csnaitsirch at web dot de

Description:
------------
Hi. I want to remove a child element of an SimpleXmlElement. But that produces a strange error and I think it's a bug.

This is the error message: Warning: main() [function.main]: Node no longer exists in C:\xampp\htdocs\test.php on line 5

It is very easy to reproduce. Pleas look at the test script.

Test script:
---------------
<?php
$xml = "<root><a><aa></aa></a></root>";
$xml = simplexml_load_string($xml);

foreach ($xml as $tag => $child) {
	unset($xml->{$tag});
}

Expected result:
----------------
No errors.

Actual result:
--------------
Warning: main() [function.main]: Node no longer exists in C:\xampp\htdocs\test.php on line 5

[2011-08-30 11:26 UTC] csnaitsirch at web dot de

I forgot to mention, that the node is removed correctly. The only problem is the error message.

[2012-10-15 10:20 UTC] opensource at prodigy7 dot de

Can confirm this behaviour. Please fix or tell us, what we do wrong.

[2012-10-17 19:51 UTC] csnaitsirch at web dot de

Sorry, but I am not a core developer. I just made a bug report. I do not know what you made wrong. I just see that there is something wrong...

[2015-05-12 17:28 UTC] cmb@php.net

-Status: Open +Status: Verified

[2015-05-12 17:28 UTC] cmb@php.net

I'm not sure, whether this qualifies as bug in PHP. Actually,
during the iteration over a collection, removing or adding
elements can always have undesired side effects.

What's happening in this case is that after the first and only
element is deleted, the next node is requested
(php_sxe_move_forward_iterator[1]). However, this fails, because
the current element had been removed, and so GET_NODE[2] triggers
the warning.

[1] <https://github.com/php/php-src/blob/55d94a453243cafc2292920e9c91bd2bc551f2be/ext/simplexml/simplexml.c#L2462-L2477>
[2] <https://github.com/php/php-src/blob/55d94a453243cafc2292920e9c91bd2bc551f2be/ext/simplexml/simplexml.c#L89-L96>

[2015-05-12 21:35 UTC] csnaitsirch at web dot de

Maybe one could just remove the line 'php_error_docref(NULL, E_WARNING, "Node no longer exists");', because it is just working fine. Or is there a further purpose for this warning?

HHVM does not output any warning (see http://3v4l.org/edMDh)

[2015-10-14 16:07 UTC] paul at muckypuddle dot com

I think removing the warning is a valid solution here.

Patches

Pull Requests

History

	php.net \| support \| documentation \| report a bug \| advanced search \| search howto \| statistics \| random bug \| login
go to bug id or search bugs for


Copyright © 2001-2023 The PHP Group All rights reserved.	Last updated: Fri Jun 09 02:03:37 2023 UTC