- Bugfix #35028 XML object fails FALSE test

diff --git a/ext/simplexml/simplexml.c b/ext/simplexml/simplexml.c
index f83714c1634c58071ac07d9bd31a3f3534cf19c2..5afbbecdd9a06f11ae137183785b0f138a9a1444 100644 (file)
--- a/ext/simplexml/simplexml.c
+++ b/ext/simplexml/simplexml.c
@@ -1205,6 +1205,13 @@ static int sxe_object_cast(zval *readobj, zval *writeobj, int type TSRMLS_DC)
 
        sxe = php_sxe_fetch_object(readobj TSRMLS_CC);
 
+       if (type == IS_BOOL) {
+               node = php_sxe_get_first_node(sxe, NULL TSRMLS_CC);
+               INIT_PZVAL(writeobj);
+               ZVAL_BOOL(writeobj, node != NULL || zend_hash_num_elements(sxe_properties_get(readobj TSRMLS_CC)) > 0);
+               return SUCCESS;
+       }
+
        if (sxe->iter.type != SXE_ITER_NONE) {
                node = php_sxe_get_first_node(sxe, NULL TSRMLS_CC);
                if (node) {

diff --git a/ext/simplexml/tests/000.phpt b/ext/simplexml/tests/000.phpt
index b83b62af704c136c97632a0271455f2d55580e62..cc8fe54d1298adbfcaf9f6c86d0ef08471e8d47e 100755 (executable)
--- a/ext/simplexml/tests/000.phpt
+++ b/ext/simplexml/tests/000.phpt
@@ -12,6 +12,7 @@ function test($what)
        global $sxe;
        echo "===$what\n";
        eval("var_dump(isset(\$$what));");
+       eval("var_dump((bool)\$$what);");
        eval("var_dump(\$$what);");
 }
 
@@ -20,7 +21,7 @@ test('sxe->elem1');
 test('sxe->elem1[0]');
 test('sxe->elem1[0]->elem2');
 test('sxe->elem1[0]->elem2->bla');
-test('sxe->elem1[0]["attr1"]');
+if (!ini_get("unicode_semantics")) test('sxe->elem1[0]["attr1"]');
 test('sxe->elem1[0]->attr1');
 test('sxe->elem1[1]');
 test('sxe->elem1[2]');
@@ -38,6 +39,7 @@ test('sxe->elem22["attr22"]');
 --EXPECTF--
 ===sxe
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (3) {
   ["@attributes"]=>
   array(1) {
@@ -71,6 +73,7 @@ object(SimpleXMLElement)#%d (3) {
 }
 ===sxe->elem1
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (3) {
   ["@attributes"]=>
   array(2) {
@@ -104,6 +107,7 @@ object(SimpleXMLElement)#%d (3) {
 }
 ===sxe->elem1[0]
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (3) {
   ["@attributes"]=>
   array(2) {
@@ -137,6 +141,7 @@ object(SimpleXMLElement)#%d (3) {
 }
 ===sxe->elem1[0]->elem2
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (2) {
   ["@attributes"]=>
   array(2) {
@@ -157,20 +162,24 @@ object(SimpleXMLElement)#%d (2) {
 }
 ===sxe->elem1[0]->elem2->bla
 bool(false)
+bool(false)
 object(SimpleXMLElement)#%d (0) {
 }
 ===sxe->elem1[0]["attr1"]
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (1) {
   [0]=>
   string(5) "first"
 }
 ===sxe->elem1[0]->attr1
 bool(false)
+bool(false)
 object(SimpleXMLElement)#%d (0) {
 }
 ===sxe->elem1[1]
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (1) {
   ["@attributes"]=>
   array(2) {
@@ -182,9 +191,11 @@ object(SimpleXMLElement)#%d (1) {
 }
 ===sxe->elem1[2]
 bool(false)
+bool(false)
 NULL
 ===sxe->elem11
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (1) {
   ["elem111"]=>
   object(SimpleXMLElement)#%d (1) {
@@ -195,6 +206,7 @@ object(SimpleXMLElement)#%d (1) {
 }
 ===sxe->elem11->elem111
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (1) {
   ["elem1111"]=>
   object(SimpleXMLElement)#%d (0) {
@@ -202,25 +214,31 @@ object(SimpleXMLElement)#%d (1) {
 }
 ===sxe->elem11->elem111->elem1111
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (0) {
 }
 ===sxe->elem22
 bool(false)
+bool(false)
 object(SimpleXMLElement)#%d (0) {
 }
 ===sxe->elem22->elem222
 bool(false)
+bool(false)
 NULL
 ===sxe->elem22->attr22
 bool(false)
+bool(false)
 NULL
 ===sxe->elem22["attr22"]
 bool(false)
+bool(false)
 NULL
 ===DONE===
 --UEXPECTF--
 ===sxe
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (3) {
   [u"@attributes"]=>
   array(1) {
@@ -254,6 +272,7 @@ object(SimpleXMLElement)#%d (3) {
 }
 ===sxe->elem1
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (3) {
   [u"@attributes"]=>
   array(2) {
@@ -287,6 +306,7 @@ object(SimpleXMLElement)#%d (3) {
 }
 ===sxe->elem1[0]
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (3) {
   [u"@attributes"]=>
   array(2) {
@@ -320,6 +340,7 @@ object(SimpleXMLElement)#%d (3) {
 }
 ===sxe->elem1[0]->elem2
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (2) {
   [u"@attributes"]=>
   array(2) {
@@ -340,20 +361,17 @@ object(SimpleXMLElement)#%d (2) {
 }
 ===sxe->elem1[0]->elem2->bla
 bool(false)
-object(SimpleXMLElement)#%d (0) {
-}
-===sxe->elem1[0]["attr1"]
 bool(false)
-object(SimpleXMLElement)#%d (1) {
-  [0]=>
-  unicode(5) "first"
+object(SimpleXMLElement)#%d (0) {
 }
 ===sxe->elem1[0]->attr1
 bool(false)
+bool(false)
 object(SimpleXMLElement)#%d (0) {
 }
 ===sxe->elem1[1]
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (1) {
   [u"@attributes"]=>
   array(2) {
@@ -365,9 +383,11 @@ object(SimpleXMLElement)#%d (1) {
 }
 ===sxe->elem1[2]
 bool(false)
+bool(false)
 NULL
 ===sxe->elem11
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (1) {
   [u"elem111"]=>
   object(SimpleXMLElement)#%d (1) {
@@ -378,6 +398,7 @@ object(SimpleXMLElement)#%d (1) {
 }
 ===sxe->elem11->elem111
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (1) {
   [u"elem1111"]=>
   object(SimpleXMLElement)#%d (0) {
@@ -385,19 +406,24 @@ object(SimpleXMLElement)#%d (1) {
 }
 ===sxe->elem11->elem111->elem1111
 bool(true)
+bool(true)
 object(SimpleXMLElement)#%d (0) {
 }
 ===sxe->elem22
 bool(false)
+bool(false)
 object(SimpleXMLElement)#%d (0) {
 }
 ===sxe->elem22->elem222
 bool(false)
+bool(false)
 NULL
 ===sxe->elem22->attr22
 bool(false)
+bool(false)
 NULL
 ===sxe->elem22["attr22"]
 bool(false)
+bool(false)
 NULL
 ===DONE===