ID: 43542
Updated by: [EMAIL PROTECTED]
Reported By: 007not at gmail dot com
-Status: Open
+Status: Assigned
Bug Type: SimpleXML related
Operating System: *
PHP Version: 5.2.5
-Assigned To:
+Assigned To: helly
Previous Comments:
------------------------------------------------------------------------
[2008-01-11 17:51:49] 007NOT at gmail dot com
helly wrote:
>The comment is a node.
SimpleXMLElement->children() do not think so !!! where is the TRUE ?
exsample:
<?php
$string = <<<XML
<?xml version='1.0'?>
<document>
<node>
<!-- comment -->
<a />
</node>
</document>
XML;
$xml = simplexml_load_string($string);
var_dump($xml->node);
$i = 0;
foreach ($xml->node as $node)
{
$i++;
}
echo $i . "\n";
var_dump((array) $xml->node);
$i = 0;
foreach ((array) $xml->node as $node)
{
$i++;
}
echo $i . "\n";
var_dump($xml->node->children());
$i = 0;
foreach ((array) $xml->node->children() as $node)
{
$i++;
}
echo $i . "\n";
------------------------------------------------------------------------
[2007-12-23 20:07:58] 007NOT at gmail dot com
>The comment is a node.
comment isn't node, it's nothing
exsampe:
<node></node> - node with empty value
<node><!-- comment --></node> - node with empty value
<node><!-- comment --> </node> - but not empty node thats has value
"space"
in :
>$i = 0;
>foreach ((array) $xml->node as $node)
>{
> $i++;
>}
>echo $i . "\n";
he we checks child nodes, but not checks that node exists
see one more test:
<?php
$string = <<<XML
<?xml version='1.0'?>
<document>
<node><!-- comment --></node>
<otherNode></otherNode>
<comment>value</comment>
</document>
XML;
$xml = simplexml_load_string($string);
$i = 0;
foreach ((array) $xml->node as $node)
{
$i++;
}
echo $i . "\n";
$dom= new DOMDocument();
$dom->loadXML($string);
$Xpath = new domxpath($dom);
$NodeList = $Xpath->query('/document');
foreach ($NodeList as $Node)
{
foreach ($Node->childNodes as $cNode)
{
echo 'Name: ' . $cNode->nodeName . ' Value: ' . $cNode->nodeValue .
PHP_EOL;
if ($cNode->childNodes->length > 1) echo 'Has childNodes' . PHP_EOL;
}
}
>What we actually need is a way to figure out the xml type of a
SimpleXMLElement instance (Element, Comment,...).
may be you make new metod
------------------------------------------------------------------------
[2007-12-20 14:39:45] [EMAIL PROTECTED]
The comment is a node. What we actually need is a way to figure out the
xml type of a SimpleXMLElement instance (Element, Comment,...). This
will also have to return the internal SXE state (iterator for something
or direct value).
------------------------------------------------------------------------
[2007-12-10 20:29:17] 007not at gmail dot com
hubert, you rigth about first test, i made mistake while i made posting
of this bug.
the right firts test looks such:
//first test
echo "node:";
var_dump($xml->node);
echo "notExitsNode:";
var_dump($xml->notExitsNode);
echo "otherNode:";
var_dump($xml->otherNode);
Expected result:
----------------
node:
object(SimpleXMLElement)[2]
public 'comment' =>
object(SimpleXMLElement)[4]
notExitsNode:
null
otherNode:
object(SimpleXMLElement)[2]
Actual result:
--------------
node:
object(SimpleXMLElement)[2]
public 'comment' =>
object(SimpleXMLElement)[4]
notExitsNode:
object(SimpleXMLElement)[2]
otherNode:
object(SimpleXMLElement)[2]
--------------------------------------------------------
>Actually, that "comment" property might have been intentionally
created
as a way to indicate whether the node has a comment.
But it is illogical, and we will have "comments hell" like:
array(/*'comment' => 'value'*/) === array('comment' => 'value')
--------------------------------------------------------
>As for your second test, I'm afraid it is incorrect
may be (because i tryed to count nodes, and it must be 1 1), but when
we use arrays we have problems with fake comments
$i = 0;
foreach ((array) $xml->node as $node)
{
$i++;
}
echo $i . "\n";
$i = 0;
foreach ((array) $xml->otherNode as $node)
{
$i++;
}
echo $i . "\n";
Expected result:
----------------
0
0
Actual result:
--------------
1
0
#####################################################################
updated test:
<?php
$string = <<<XML
<?xml version='1.0'?>
<document>
<node><!-- comment --></node>
<otherNode></otherNode>
<comment>value</comment>
</document>
XML;
$xml = simplexml_load_string($string);
//first test
echo "node:";
var_dump($xml->node);
echo "notExitsNode:";
var_dump($xml->notExitsNode);
echo "otherNode:";
var_dump($xml->otherNode);
/*
Expected result:
----------------
node:
object(SimpleXMLElement)[2]
public 'comment' =>
object(SimpleXMLElement)[4]
notExitsNode:
null
otherNode:
object(SimpleXMLElement)[2]
Actual result:
--------------
node:
object(SimpleXMLElement)[2]
public 'comment' =>
object(SimpleXMLElement)[4]
notExitsNode:
object(SimpleXMLElement)[2]
otherNode:
object(SimpleXMLElement)[2]
*/
//second test
$i = 0;
foreach ($xml->node as $node)
{
$i++;
}
echo $i . "\n";
$i = 0;
foreach ($xml->otherNode as $node)
{
$i++;
}
echo $i . "\n";
$i = 0;
foreach ((array) $xml->node as $node)
{
$i++;
}
echo $i . "\n";
$i = 0;
foreach ((array) $xml->otherNode as $node)
{
$i++;
}
echo $i . "\n";
/*
Expected result:
----------------
1
1
0
0
Actual result:
--------------
1
1
1
0
*/
//third test
var_dump($xml->node->comment);
var_dump($xml->otherNode->comment);
//check magic
echo "node:\n";
if (is_object($xml->node->comment))
{
echo "is_object === TRUE \n";
}
if (isset($xml->node->comment))
{
echo "isset === TRUE \n";
}
//but
if (strlen($xml->node->comment) > 0)
{
echo "strlen > 0\n";
}
if (strlen($xml->node->comment) == 0)
{
echo "strlen == 0\n";
}
echo "otherNode:\n";
if (is_object($xml->otherNode->comment))
{
echo "is_object === TRUE \n";
}
if (isset($xml->otherNode->comment))
{
echo "isset === TRUE \n";
}
/*
Expected result:
----------------
node:
is_object === TRUE
isset === TRUE
strlen == 0
otherNode:
is_object === TRUE
Actual result:
--------------
node:
is_object === TRUE
strlen == 0
otherNode:
is_object === TRUE
*/
------------------------------------------------------------------------
[2007-12-09 14:39:46] hubert dot roksor at gmail dot com
Regarding your first test, I wouldn't consider that a bug. var_dump()
is a debugging tool, it may expose some of the behind-the-scene magic.
Actually, that "comment" property might have been intentionally created
as a way to indicate whether the node has a comment. That would explain
isset()'s behaviour in your third test, but in this case I would
recommand replacing that magical property with a method such as
$node->hasComment(). I guess Rob Richards will be able to shed some
light here.
As for your second test, I'm afraid it is incorrect: both $xml->node
and $xml->otherNode should return 1 element and I don't see why having a
comment as a child would change that.
------------------------------------------------------------------------
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
http://bugs.php.net/43542
--
Edit this bug report at http://bugs.php.net/?id=43542&edit=1
