Fix DOM

Author: Girgias

ext/dom/node.c

@@ -469,7 +469,8 @@ zend_result dom_node_owner_document_read(dom_object *obj, zval *retval)
 
 	xmlDocPtr docp = nodep->doc;
 	if (!docp) {
-		return FAILURE;
+		ZVAL_NULL(retval);
+		return SUCCESS;
 	}
 
 	php_dom_create_object((xmlNodePtr) docp, retval, obj);

ext/dom/tests/DOMEntityReference_predefined_free.phpt

@@ -8,7 +8,7 @@ $ref = new DOMEntityReference("amp");
 var_dump($ref);
 ?>
 --EXPECTF--
-object(DOMEntityReference)#1 (17) {
+object(DOMEntityReference)#1 (18) {
   ["nodeName"]=>
   string(3) "amp"
   ["nodeValue"]=>
@@ -33,6 +33,8 @@ object(DOMEntityReference)#1 (17) {
   NULL
   ["isConnected"]=>
   bool(false)
+  ["ownerDocument"]=>
+  NULL
   ["namespaceURI"]=>
   NULL
   ["prefix"]=>

ext/dom/tests/delayed_freeing/without_contructor.phpt

@@ -31,8 +31,6 @@ try {
 
 ?>
 --EXPECT--
-object(DOMNode)#2 (0) {
-}
 Invalid State Error
 Couldn't fetch DOMNode
 Couldn't fetch DOMNode

ext/dom/tests/gh16316.phpt

@@ -24,9 +24,5 @@ try {
 
 ?>
 --EXPECT--
-object(Demo)#1 (1) {
-  ["registerNodeNamespaces"]=>
-  bool(true)
-}
 Invalid State Error
 Invalid State Error

ext/dom/tests/modern/spec/Document_implementation_createDocumentType.phpt

@@ -25,7 +25,7 @@ foreach ($test_matrix as $test_item) {
 
 ?>
 --EXPECT--
-object(Dom\DocumentType)#3 (19) {
+object(Dom\DocumentType)#3 (20) {
   ["name"]=>
   string(5) "qname"
   ["entities"]=>
@@ -46,6 +46,8 @@ object(Dom\DocumentType)#3 (19) {
   NULL
   ["isConnected"]=>
   bool(false)
+  ["ownerDocument"]=>
+  NULL
   ["parentNode"]=>
   NULL
   ["parentElement"]=>
@@ -68,7 +70,7 @@ object(Dom\DocumentType)#3 (19) {
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE qname PUBLIC "public" "system">
 
-object(Dom\DocumentType)#2 (19) {
+object(Dom\DocumentType)#2 (20) {
   ["name"]=>
   string(5) "qname"
   ["entities"]=>
@@ -89,6 +91,8 @@ object(Dom\DocumentType)#2 (19) {
   NULL
   ["isConnected"]=>
   bool(false)
+  ["ownerDocument"]=>
+  NULL
   ["parentNode"]=>
   NULL
   ["parentElement"]=>
@@ -111,7 +115,7 @@ object(Dom\DocumentType)#2 (19) {
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE qname PUBLIC "public" "">
 
-object(Dom\DocumentType)#1 (19) {
+object(Dom\DocumentType)#1 (20) {
   ["name"]=>
   string(5) "qname"
   ["entities"]=>
@@ -132,6 +136,8 @@ object(Dom\DocumentType)#1 (19) {
   NULL
   ["isConnected"]=>
   bool(false)
+  ["ownerDocument"]=>
+  NULL
   ["parentNode"]=>
   NULL
   ["parentElement"]=>
@@ -154,7 +160,7 @@ object(Dom\DocumentType)#1 (19) {
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE qname SYSTEM "system">
 
-object(Dom\DocumentType)#4 (19) {
+object(Dom\DocumentType)#4 (20) {
   ["name"]=>
   string(5) "qname"
   ["entities"]=>
@@ -175,6 +181,8 @@ object(Dom\DocumentType)#4 (19) {
   NULL
   ["isConnected"]=>
   bool(false)
+  ["ownerDocument"]=>
+  NULL
   ["parentNode"]=>
   NULL
   ["parentElement"]=>

ext/dom/tests/modern/xml/DTDNamedNodeMap.phpt

@@ -142,7 +142,7 @@ object(Dom\Entity)#3 (17) {
   ["textContent"]=>
   NULL
 }
-object(Dom\Notation)#4 (13) {
+object(Dom\Notation)#4 (14) {
   ["nodeType"]=>
   int(12)
   ["nodeName"]=>
@@ -151,6 +151,8 @@ object(Dom\Notation)#4 (13) {
   NULL
   ["isConnected"]=>
   bool(false)
+  ["ownerDocument"]=>
+  NULL
   ["parentNode"]=>
   NULL
   ["parentElement"]=>