Bug #62639 [Com]: XML structure broken

From: Date: Tue, 11 Feb 2014 11:33:18 +0000
Subject: Bug #62639 [Com]: XML structure broken
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-184261@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=62639&edit=1

 ID:                 62639
 Comment by:         electroglyph at gmail dot com
 Reported by:        alexshock at yandex dot ru
 Summary:            XML structure broken
 Status:             Assigned
 Type:               Bug
 Package:            SimpleXML related
 Operating System:   debian 6.0.5
 PHP Version:        5.4.5
 Assigned To:        acurioso
 Block user comment: N
 Private report:     N

 New Comment:

???


Previous Comments:
------------------------------------------------------------------------
[2013-12-29 20:34:00] knyttl at goout dot cz

Is there something happening with this issue? It is now in stable debian and others :/

------------------------------------------------------------------------
[2013-08-16 01:59:10] sergey at shymko dot net

The following workaround seems to fix both dumping and type casting:

<?php

class SimpleXMLElement_CloneFix extends SimpleXMLElement
{
    public function xpath($path)
    {
        $result = parent::xpath($path) ?: array();
        foreach ($result as $node) {
            $this->_fixNode($node);
        }
        return $result;
    }

    protected function _fixNode(SimpleXMLElement $node)
    {
        if (!$node && (string)$node) {
            $dom = dom_import_simplexml($node);
            $dom->parentNode->replaceChild(clone $dom, $dom);
        }
    }
}

Unfortunately, the same approach does not work for children() method, because of mysterious nature
of its returned value.
Despite returned object manifests itself as an instance of SimpleXMLElement, it does not behave as
such.
For instance, it is incompatible with dom_import_simplexml().
So, what kind of SimpleXMLElement is that?

------------------------------------------------------------------------
[2013-06-14 00:21:27] info at fuktommy dot com

It causes not only wrong var_dump and print_r, but also wrong boolean cast.

Test script:
------------
$xml = <<<XML
<?xml version="1.0"?>
<a>
    <b>Some String</b>
</a>
XML;

$b = simplexml_load_string($xml)->xpath('/a/b');
var_dump((string)$b[0]);
var_dump((bool)$b[0]);

Expected result:
----------------
string(11) "Some String"
bool(true)

Actual result:
--------------
string(11) "Some String"
bool(false)

------------------------------------------------------------------------
[2013-04-24 07:48:36] laruence@php.net

I think we should revert the previous fix at now, since this new bug is more 
critical than the previous one.

and I think the key problem there is that we can not tell iterating from 
convert_to_array in the sxe_get_prop_hash

without that, I don't think we can get a good fix for the 034.phpt

------------------------------------------------------------------------
[2013-02-18 21:21:48] salathe@php.net

Related To: Bug #62203

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=62639


-- 
Edit this bug report at https://bugs.php.net/bug.php?id=62639&edit=1


Thread (21 messages)

« previous php.bugs (#184261) next »