Handle strict error properly in adoptNode failure, and add a test

Author: ndossche

ext/dom/document.c

@@ -1064,7 +1064,7 @@ PHP_METHOD(DOMDocument, adoptNode)
 		|| nodep->type == XML_ENTITY_NODE
 		|| nodep->type == XML_NOTATION_NODE)) {
 		php_dom_throw_error(NOT_SUPPORTED_ERR, dom_get_strict_error(dom_object_nodep->document));
-		RETURN_THROWS();
+		RETURN_FALSE;
 	}
 
 	xmlDocPtr new_document;

ext/dom/tests/DOMDocument_adoptNode.phpt

@@ -43,14 +43,24 @@ $adopted = $doc1->adoptNode($doc1->firstChild->firstChild);
 var_dump($adopted->textContent);
 var_dump($doc1->saveXML());
 
-echo "-- Adopt a document --\n";
+echo "-- Adopt a document (strict error on) --\n";
 
 try {
     $doc1->adoptNode($doc1);
 } catch (\DOMException $e) {
     echo $e->getMessage(), "\n";
 }
 
+echo "-- Adopt a document (strict error off) --\n";
+
+$doc1->strictErrorChecking = false;
+try {
+    $doc1->adoptNode($doc1);
+} catch (\DOMException $e) {
+    echo $e->getMessage(), "\n";
+}
+$doc1->strictErrorChecking = true;
+
 echo "-- Adopt an attribute --\n";
 
 $doc3 = new DOMDocument();
@@ -102,7 +112,7 @@ unset($doc1);
 var_dump($child->nodeName);
 
 ?>
---EXPECT--
+--EXPECTF--
 -- Owner document check before adopting --
 bool(true)
 bool(false)
@@ -127,8 +137,11 @@ string(5) "world"
 string(27) "<?xml version="1.0"?>
 <p/>
 "
--- Adopt a document --
+-- Adopt a document (strict error on) --
 Not Supported Error
+-- Adopt a document (strict error off) --
+
+Warning: DOMDocument::adoptNode(): Not Supported Error in %s on line %d
 -- Adopt an attribute --
 bool(true)
 bool(true)